Skip to content

Commit 168cf85

Browse files
Cover form-feed baseURL whitespace validation
Co-authored-by: Eric Allam <eric@trigger.dev>
1 parent b3bd45c commit 168cf85

File tree

3 files changed

+24
-0
lines changed

3 files changed

+24
-0
lines changed

docs/tasks/streams.mdx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -677,6 +677,7 @@ Examples:
677677
-`https://api.trigger.dev/\ninternal`
678678
-`https://api.trigger.dev/\tinternal`
679679
-`https://api.trigger.dev/\vinternal`
680+
-`https://api.trigger.dev/\finternal`
680681
-`https://api.trigger.dev/\rinternal`
681682
-`https://api.trigger.dev/\u200Binternal`
682683
-`https://api.trigger.dev/\uFEFFinternal`

packages/ai/README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -184,6 +184,7 @@ Examples:
184184
-`https://api.trigger.dev/\ninternal` (internal whitespace characters)
185185
-`https://api.trigger.dev/\tinternal` (internal tab characters)
186186
-`https://api.trigger.dev/\vinternal` (internal vertical-tab characters)
187+
-`https://api.trigger.dev/\finternal` (internal form-feed characters)
187188
-`https://api.trigger.dev/\rinternal` (internal carriage-return characters)
188189
-`https://api.trigger.dev/\u200Binternal` (internal zero-width-space characters)
189190
-`https://api.trigger.dev/\uFEFFinternal` (internal BOM characters)

packages/ai/src/chatTransport.test.ts

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -753,6 +753,17 @@ describe("TriggerChatTransport", function () {
753753
}).toThrowError("baseURL must not contain internal whitespace characters");
754754
});
755755

756+
it("throws when baseURL contains internal form-feed characters", function () {
757+
expect(function () {
758+
new TriggerChatTransport({
759+
task: "chat-task",
760+
accessToken: "pk_trigger",
761+
baseURL: "https://api.trigger.dev/\finternal",
762+
stream: "chat-stream",
763+
});
764+
}).toThrowError("baseURL must not contain internal whitespace characters");
765+
});
766+
756767
it("throws when baseURL contains internal carriage-return characters", function () {
757768
expect(function () {
758769
new TriggerChatTransport({
@@ -3415,6 +3426,17 @@ describe("TriggerChatTransport", function () {
34153426
}).toThrowError("baseURL must not contain internal whitespace characters");
34163427
});
34173428

3429+
it("throws from factory when baseURL contains internal form-feed characters", function () {
3430+
expect(function () {
3431+
createTriggerChatTransport({
3432+
task: "chat-task",
3433+
accessToken: "pk_trigger",
3434+
baseURL: "https://api.trigger.dev/\finternal",
3435+
stream: "chat-stream",
3436+
});
3437+
}).toThrowError("baseURL must not contain internal whitespace characters");
3438+
});
3439+
34183440
it("throws from factory when baseURL contains internal carriage-return characters", function () {
34193441
expect(function () {
34203442
createTriggerChatTransport({

0 commit comments

Comments
 (0)