Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Use empty string in Sse event when there is no data #37036

Merged
merged 1 commit into from
Nov 13, 2023

Conversation

geoand
Copy link
Contributor

@geoand geoand commented Nov 13, 2023

Using an empty string instead of null
is what the classic rest client does,
so let's align with it

Closes: #37033

This comment has been minimized.

Using an empty string instead of null
is what the classic rest client does,
so let's align with it

Closes: quarkusio#37033
Copy link

quarkus-bot bot commented Nov 13, 2023

✔️ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

@geoand geoand merged commit 27f27e9 into quarkusio:main Nov 13, 2023
31 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.6 - main milestone Nov 13, 2023
@geoand geoand deleted the #37033 branch November 14, 2023 07:28
@michalvavrik
Copy link
Member

#35990 was backported, so I think this one should be as well

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

Successfully merging this pull request may close these issues.

Sse difference in empty event between non-reactive and reactive output
4 participants