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-6670 Upgrade Jersey to 2.37 #6035

Merged
merged 1 commit into from
Nov 15, 2022

Conversation

arieki
Copy link
Contributor

@arieki arieki commented Nov 14, 2022

Description

Upgrade patched Jersey from 2.36.payara-p1 to 2.37.payara-p1

Important Info

Blockers

None

Testing

New tests

None

Testing Performed

Run Payara server, quicklook test and payara-samples

Testing Environment

Windows 11, WSL, OpenJDK8, Maven 3.8.4

Documentation

N/A

Notes for Reviewers

N/A

@kalinchan
Copy link
Member

If you plan to upgrade ASM in this PR, you should include an upgrade of HK2 with ASM 9.4.

In other words, in HK2, upgrade ASM to 9.4 and release next version of HK2 and increment version in Payara.

@arieki
Copy link
Contributor Author

arieki commented Nov 14, 2022

If you plan to upgrade ASM in this PR, you should include an upgrade of HK2 with ASM 9.4.

In other words, in HK2, upgrade ASM to 9.4 and release next version of HK2 and increment version in Payara.

got it. I won't include it here, and will revert the ASM upgrade. Thanks @kalinchan

@arieki arieki changed the title FISH-6670, FISH-6686 Upgrade Jersey to 2.37 and ASM to 9.4 FISH-6670 Upgrade Jersey to 2.37 Nov 14, 2022
@arieki
Copy link
Contributor Author

arieki commented Nov 15, 2022

jenkins test please

@arieki arieki marked this pull request as ready for review November 15, 2022 09:01
@arieki arieki requested review from Pandrex247, pdudits, aubi and jGauravGupta and removed request for aubi November 15, 2022 09:18
@arieki arieki merged commit 3afa160 into payara:master Nov 15, 2022
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