Skip to content
New issue

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

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

Already on GitHub? # to your account

Ensure request is set for ContextScopeListener in EE10 #11518

Merged
merged 4 commits into from
Mar 25, 2024

Conversation

lachlan-roberts
Copy link
Contributor

This is needed for GoogleCloudPlatform/appengine-java-standard#96

Currently on async requests the ContextScopeListener will have a null request. This PR ensures that the request is set.

Signed-off-by: Lachlan Roberts <lachlan@webtide.com>
@lachlan-roberts lachlan-roberts added the Sponsored This issue affects a user with a commercial support agreement label Mar 14, 2024
@lachlan-roberts lachlan-roberts self-assigned this Mar 14, 2024
Signed-off-by: Lachlan Roberts <lachlan@webtide.com>
Signed-off-by: Lachlan Roberts <lachlan@webtide.com>
Signed-off-by: Lachlan Roberts <lachlan@webtide.com>
Copy link
Contributor

@gregw gregw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Sponsored This issue affects a user with a commercial support agreement
Projects
No open projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

3 participants