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

emitPropertyChange merges payloads - race condition? #1231

Open
danielpeintner opened this issue Feb 9, 2024 · 1 comment
Open

emitPropertyChange merges payloads - race condition? #1231

danielpeintner opened this issue Feb 9, 2024 · 1 comment
Labels
binding-http Issues related to http protocol binding bug Something isn't working

Comments

@danielpeintner
Copy link
Member

danielpeintner commented Feb 9, 2024

That would mean that when emitPropertyChange is called both if statements are true:

Just to be precise, it is the emitEvent that is the issue.

I am only sending a GET request from the browser to localhost:8081/coffee-machine/events/resourceEmpty and if both conditions are met, I get "water""beans". Both payloads are somehow merged into one since they are emitted very quickly. I should have gotten only "water" since the browser will not send another GET before the other event could be emitted.

Originally posted by @egekorkan in #1229 (comment)

@relu91 relu91 added bug Something isn't working binding-http Issues related to http protocol binding labels Feb 9, 2024
@relu91
Copy link
Member

relu91 commented Feb 9, 2024

I'm taggin this as binding-http bug because I think it is strongly connected with how we implemented long polling

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
binding-http Issues related to http protocol binding bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants