Skip to content

Improved event stream checking example #45

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

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.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant