Skip to content

Improved event stream checking example#45

Open
slavarazum wants to merge 2 commits intoAzure:mainfrom
slavarazum:doc-update
Open

Improved event stream checking example#45
slavarazum wants to merge 2 commits intoAzure:mainfrom
slavarazum:doc-update

Conversation

@slavarazum
Copy link

@slavarazum slavarazum commented May 28, 2023

A tiny example update replacing equal condition for EventStreamContentType by checking with startsWith function.

- response.headers.get('content-type') === EventStreamContentType
+ response.headers.get('content-type').startsWith('text/event-stream')

covers cases when charset present: Content-Type: text/event-stream; UTF-8.

Added import of EventStreamContentType variable to take it more clear.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant