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

FISH-5832 FISH-5833 Weld 3.1.8 and Weld SPI 3.1.SP4 #5484

Merged
merged 2 commits into from
Nov 4, 2021

Conversation

Pandrex247
Copy link
Member

@Pandrex247 Pandrex247 commented Nov 3, 2021

Description

Title.
Updates Weld to a version that works with JDK 17.

PR does not constitute support for CDI under JDK 17 - there are failures in Payara Samples that indicate an issue in Weld with the resolution of the RolesPermitted annotation. Java reflect can't parse the signature of the semantics enum, specifically when it's the Weld Proxy being parsed (it's using JDK 1.5 type signatures which support for was removed in JDK 17):
fish.payara.cdi.auth.roles.LogicalOperator -- Signature Parse error: Expected Field Type Signature

This does however only occur when the server is running under JDK 17 since it's a change in the JDK itself that breaks things.

Important Info

Blockers

None.

Testing

New tests

None

Testing Performed

Started the server and ran Payara Samples without issue - JDK 8
Started the server and ran Payara Samples without issue - JDK 11

Testing Environment

WSL OpenSUSE 15.3, Zulu JDK 8u312 and 11.0.13

Documentation

N/A

Notes for Reviewers

None

Pandrex247 and others added 2 commits November 3, 2021 16:03
- reimplemented ProxyServicesImpl, because Weld changed behavior and added
  methods to the interface and previous implementation caused failure of
  cdi_all tests

Signed-off-by: Andrew Pielage <[email protected]>
@Pandrex247
Copy link
Member Author

Jenkins test please

@Pandrex247 Pandrex247 marked this pull request as ready for review November 3, 2021 17:05
Copy link
Contributor

@breakponchito breakponchito left a comment

Choose a reason for hiding this comment

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

LGTM

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.

4 participants