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

4.0.11: After retrieval check baggage entry for null before dereferencing it #8975

Merged

Conversation

barchetta
Copy link
Member

Description

Backport of #8885 to 4.0.11.

Resolves #8882

Check the BaggageEntry retrieved from the map for null before attempting to get the entry's value to return.

Add test.

Documentation

Bug fix; no doc impact.

…elidon-io#8885)

* After retrieval check baggage entry for null before dereferencing it

Signed-off-by: Tim Quinn <tim.quinn@oracle.com>

* Make test class pkg private; no need for it to be public

---------

Signed-off-by: Tim Quinn <tim.quinn@oracle.com>
@barchetta barchetta added tracing 4.x Version 4.x labels Jul 12, 2024
@barchetta barchetta added this to the 4.0.11 milestone Jul 12, 2024
@barchetta barchetta requested a review from tjquinno July 12, 2024 18:30
@barchetta barchetta self-assigned this Jul 12, 2024
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Jul 12, 2024
@barchetta barchetta added the backport Issues that are merged into a single branch, but missing in either master or previous release label Jul 12, 2024
@barchetta barchetta merged commit 55b9518 into helidon-io:helidon-4.0.11 Jul 12, 2024
12 checks passed
@barchetta barchetta deleted the 4.0.11-8885-baggage-npe branch November 12, 2024 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4.x Version 4.x backport Issues that are merged into a single branch, but missing in either master or previous release OCA Verified All contributors have signed the Oracle Contributor Agreement. tracing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants