Skip to content

Fixed #3801 - inject cdi into custom validator #4375

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
Merged
merged 2 commits into from
May 11, 2020
Merged

Fixed #3801 - inject cdi into custom validator #4375

merged 2 commits into from
May 11, 2020

Conversation

@dmatej
Copy link
Contributor

@dmatej dmatej commented Jan 28, 2020 •
edited
Loading

This is a fix for #3801

dmatej added 2 commits January 28, 2020 15:05
@dmatej
Issue #3801 Jersey-Bean-Validator-fails-to-inject-Weld-managed-beans
4d881e9 
Signed-off-by: David Matejcek <dmatej@seznam.cz>
@dmatej
Issue #3801 Do not use CDI with bean validation when it is not available
e795abe 
- fixes osgi-functional-tests

Signed-off-by: David Matejcek <dmatej@seznam.cz>
@jansupol
Copy link
Contributor

jansupol commented Apr 3, 2020

Filed CQ 21874

@jansupol
Copy link
Contributor

jansupol commented Apr 3, 2020

@dmatej Thanks for the contribution. I am only curious, why Payara hates tests?

@dmatej
Copy link
Contributor Author

dmatej commented Apr 3, 2020

@dmatej Thanks for the contribution. I am only curious, why Payara hates tests?

Payara loves tests! But this fix is a bit older, I found it while I was updating dependencies.

@senivam senivam added this to the 2.31 milestone May 11, 2020
@senivam senivam merged commit 9d28d1e into eclipse-ee4j:master May 11, 2020
This was referenced Jun 26, 2021
Merged
Open
Open
Merged
Open
This was referenced Jul 6, 2021
Merged
Closed
This was referenced Aug 5, 2021
Closed
Closed
This was referenced Aug 30, 2021
Closed
Merged
Merged
Merged
@renovate renovate bot mentioned this pull request Sep 6, 2021
Merged
1 task
This was referenced Oct 4, 2021
Merged
Merged
This was referenced Oct 18, 2021
Open
Open
@ChristianHoesel ChristianHoesel mentioned this pull request Oct 21, 2021
Merged
1 task
@renovate renovate bot mentioned this pull request Mar 7, 2022
Closed
1 task
This was referenced Mar 15, 2022
Merged
Merged
This was referenced Apr 17, 2022
Closed
Closed
This was referenced May 3, 2022
Open
Open
@renovate renovate bot mentioned this pull request Sep 23, 2022
Merged
1 task
This was referenced Oct 19, 2022
Closed
Closed
@renovate renovate bot mentioned this pull request Feb 11, 2023
Merged
1 task
@mend-for-github-com mend-for-github-com bot mentioned this pull request Mar 26, 2023
Closed
@renovate renovate bot mentioned this pull request Jul 6, 2023
Open
1 task
@renovate renovate bot mentioned this pull request Aug 30, 2023
Merged
1 task
@mend-for-github-com mend-for-github-com bot mentioned this pull request Dec 4, 2023
Closed
@mend-for-github-com mend-for-github-com bot mentioned this pull request Jan 11, 2024
Closed
This was referenced Apr 13, 2024
Merged
Closed
@renovate renovate bot mentioned this pull request Feb 18, 2025
Closed
1 task
This was referenced Feb 23, 2025
Open
Open
Open
@mend-for-github-com mend-for-github-com bot mentioned this pull request Apr 29, 2025
Closed
1 task
This was referenced May 26, 2025
Closed
Closed
Closed
Closed
@renovate-bot renovate-bot mentioned this pull request Nov 18, 2025
Closed
1 task
This was referenced Nov 18, 2025
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jansupol jansupol jansupol approved these changes

@senivam senivam senivam approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

2.31

Development

Successfully merging this pull request may close these issues.

3 participants

@dmatej @jansupol @senivam