Skip to content

Fix data loss when client sends data immediately after HTTP upgrade on Netty - #35748

Open
nidhin-cp wants to merge 3 commits into
OpenLiberty:integrationfrom
nidhin-cp:testUpgradeReadListenerSendImmediateData-EE11-failure-netty
Open

nidhin-cp wants to merge 3 commits into
OpenLiberty:integrationfrom
nidhin-cp:testUpgradeReadListenerSendImmediateData-EE11-failure-netty

Conversation

@nidhin-cp

@nidhin-cp nidhin-cp commented Sep 10, 2026

Copy link
Copy Markdown
Member

@nidhin-cp nidhin-cp self-assigned this Sep 10, 2026
@nidhin-cp

nidhin-cp commented Sep 10, 2026

Copy link
Copy Markdown
Member Author

#build (view Open Liberty Personal Build - ❌ completed with errors/failures.)

Note: Target locations of links might be accessible only to IBM employees.

@nidhin-cp

nidhin-cp commented Sep 10, 2026

Copy link
Copy Markdown
Member Author

!fullpb
(view Open Liberty Personal Build - ❌ completed with errors/failures.)
extra.fat.ant.options=-Dglobal.jvm.args="-Dcom.ibm.ws.beta.edition=true"

Note: Target locations of links might be accessible only to IBM employees.

@LibbyBot

Copy link
Copy Markdown
Member

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 1 product code files were changed.
  • Please describe in a separate comment how you tested your changes.

1 similar comment
@LibbyBot

Copy link
Copy Markdown
Member

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 1 product code files were changed.
  • Please describe in a separate comment how you tested your changes.

@nidhin-cp nidhin-cp changed the title save unread data before upgrade complete in HttpDispatcherLink.java Save unread data before upgrade complete in HttpDispatcherLink.java Sep 14, 2026
@nidhin-cp
nidhin-cp force-pushed the testUpgradeReadListenerSendImmediateData-EE11-failure-netty branch from a245843 to 42d6bf5 Compare September 14, 2026 10:39
@nidhin-cp

nidhin-cp commented Sep 14, 2026

Copy link
Copy Markdown
Member Author

!fullpb
(view Open Liberty Personal Build - ❌ completed with errors/failures.)
extra.fat.ant.options=-Dglobal.jvm.args="-Dcom.ibm.ws.beta.edition=true"

Note: Target locations of links might be accessible only to IBM employees.

@nidhin-cp

nidhin-cp commented Sep 14, 2026

Copy link
Copy Markdown
Member Author

#build (view Open Liberty Personal Build - ⚠️ completed with warnings.)

Note: Target locations of links might be accessible only to IBM employees.

@LibbyBot

Copy link
Copy Markdown
Member

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 2 product code files were changed.
  • Please describe in a separate comment how you tested your changes.

1 similar comment
@LibbyBot

Copy link
Copy Markdown
Member

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 2 product code files were changed.
  • Please describe in a separate comment how you tested your changes.

saving the bytes

saving the bytes

saving the bytes

saving the bytes

saving the bytes

saving the bytes

Addig NettyServletUpgradeHandler

Addig NettyServletUpgradeHandler

Addig NettyServletUpgradeHandler

Addig NettyServletUpgradeHandler
@nidhin-cp
nidhin-cp force-pushed the testUpgradeReadListenerSendImmediateData-EE11-failure-netty branch from 8ecfa98 to dd5eb11 Compare September 14, 2026 11:13
@nidhin-cp nidhin-cp changed the title Save unread data before upgrade complete in HttpDispatcherLink.java Fix data loss when client sends data immediately after HTTP upgrade on Netty Sep 14, 2026
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.

2 participants