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

RESTEASY-1049 Check availability of CDI classes #511

Closed
wants to merge 1 commit into from

Conversation

kariem
Copy link
Contributor

@kariem kariem commented Jun 23, 2014

This should fix the problem indicated in the comment on RESTEASY-1049 by assuring that the class used in the instanceof check is available.

Check availability of CdiInjectorFactory before performing an instanceof
@ronsigal
Copy link
Collaborator

Actually, I fixed this problem in commit cee85c7#diff-89690a7c062c4ea7862756da4ec5d036. But then, somehow, I deleted the fix in commit fb2b38e#diff-5905e9c29421b53ac04beafa0c104e01. I must have merged an outdated branch.

@kariem
Copy link
Contributor Author

kariem commented Jul 2, 2014

@ronsigal, what do you suggest we do now?

@ronsigal
Copy link
Collaborator

ronsigal commented Jul 3, 2014

Hi Karlem,

I've dug up my latest version of the solution. I believe it's not enough to know that CDI is on the classpath. It's also necessary to know whether or not it's actually being used, i.e., whether there is a beans.xml file. I'm testing the solution right now.

-Ron

@theigl
Copy link

theigl commented Aug 19, 2014

@ronsigal: is there any progress on this issue and is this going to make it into 3.0.9?

@kariem
Copy link
Contributor Author

kariem commented Aug 19, 2014

From what I have seen, this was resolved via #521. Also see RESTEASY-1049.

@kariem kariem closed this Aug 19, 2014
@ronsigal
Copy link
Collaborator

Yes, it will.

On 08/19/2014 07:03 AM, Thomas wrote:

@ronsigal https://github.com/ronsigal: is there any progress on this
issue and is this going to make it into 3.0.9?


Reply to this email directly or view it on GitHub
#511 (comment).

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.

3 participants