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

Attempt to use CDI InterceptionFactory #181

Closed
wants to merge 2 commits into from

Conversation

manovotn
Copy link
Contributor

This PR attempts to remove the need to handle interception chains for @RegisterRestClient proxies manually.
Instead it registers the proxies as CDI beans and uses InterceptionFactory to let CDI fully handle interception.

I've ran the tests with mvn clean verify -Dversion.org.wildfly=29.0.0.Beta1-SNAPSHOT using WFLY with latest Weld (5.1.1.Final) in it.
There are also some notes in the code regarding what could be improved but if it worked this way up until now, it should be the same with these changes.

@jamezp please review carefully as this seems to have poor test coverage overall and it might be that I missed some RE parts that I am blind to :)

@jamezp
Copy link
Contributor

jamezp commented Jun 20, 2023

Replaced by #182 to fix the security manager test failures.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants