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.

Sign up for GitHub

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

Jump to bottom

Provide access to interceptor bindings from InvocationContext #99

Merged
merged 1 commit into from Jul 11, 2023
Merged

Provide access to interceptor bindings from InvocationContext #99

merged 1 commit into from Jul 11, 2023

Conversation

Ladicek
Copy link
Contributor

@Ladicek Ladicek commented Jun 20, 2023

No description provided.

@Ladicek Ladicek mentioned this pull request Jun 20, 2023
Closed
@Ladicek
Copy link
Contributor Author

Ladicek commented Jun 20, 2023

I plan to add tests for this to the CDI TCK. It is expected that EJB is not affected by this by any means.

@Ladicek Ladicek force-pushed the interceptor-bindings-access branch from b6226f9 to 331b277 Compare July 11, 2023 16:11
@Ladicek
Copy link
Contributor Author

Ladicek commented Jul 11, 2023 •
edited

Thank you! FYI, I looked at the changes again and realized there's an inconsistency between the javadocs of getInterceptorBinding(Class) and getInterceptorBindings(Class), so I pushed a fix.

@arjantijms
Copy link
Contributor

@Ladicek thanks!

@arjantijms arjantijms merged commit cb51fbe into jakartaee:master Jul 11, 2023
2 checks passed
@Ladicek Ladicek deleted the interceptor-bindings-access branch July 11, 2023 18:48
@Ladicek Ladicek mentioned this pull request Jul 17, 2023
Merged
@manovotn manovotn mentioned this pull request Sep 25, 2023
Closed
@mkouba mkouba mentioned this pull request Jan 11, 2024
Closed
@starksm64 starksm64 linked an issue Feb 22, 2024 that may be closed by this pull request
Update InvocationContext to provide a standard accessor for interceptor bindings #100
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@arjantijms arjantijms arjantijms approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update InvocationContext to provide a standard accessor for interceptor bindings
2 participants
@Ladicek @arjantijms