-
Notifications
You must be signed in to change notification settings - Fork 827
[#4873] Support for SSE interface RPC calls #4875
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
40d7c41
[#4873] Support for SSE interface RPC calls and register SSE schema
chengyouling 14c05e7
When reconnecting, refresh the eventId based on the last-event-id and…
chengyouling 7eda5d9
fixed test problems
chengyouling 517480b
fixed style problem
chengyouling 500e8b3
add testcase
chengyouling 5f3a593
add testcase
chengyouling a7135bd
only test rest
chengyouling 58050a5
test reactive testcase
chengyouling 45625a0
move testcase to spring-boot-springmvc
chengyouling af92830
for test
chengyouling 0332c62
adjust Flowable result
chengyouling 0cd0c47
fixed testcase
chengyouling 0e3e646
adjust flowable response decode
chengyouling 9f7a9dc
for test
chengyouling 7076c84
add flushBuffer
chengyouling 76da23e
change using flushStreamBuffer for test
chengyouling 2b32da4
fixed testcase
chengyouling 255dee9
add response parsing
chengyouling 7fb5f75
fixed Multiple transfer-encoding=chunked in Rest Over Servlet
chengyouling cfbe610
fixed review options,compatible use \r\n \n \r,add PublisherProduce…
chengyouling 10255be
delete not use EVENTS_MEDIA_TYPE
chengyouling 1adfc0b
add warn for encoding
chengyouling 4c2c34e
adjust eventId to id, resolution concurrency using writeIndex problem
chengyouling 63e6c1a
Compatibility settings for multiple data
chengyouling 0812b8b
Compatible with both client and server, addressing scenarios of incon…
chengyouling e38bb52
add catch for subscribe
chengyouling 0da8280
Compatible with both packet aggregation and packet fragmentation scen…
chengyouling File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe change
setChunkedimplementation to first check if header exists and then add it?Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The chunked header is empty when constructing the response in Rest Over Servlet model, so it be always add chunked header information, and tomcat itself will actively construct the chunked response header in the streaming scenario.