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-5705 Yasson 1.0.9 #5458

Merged
merged 1 commit into from Nov 3, 2021
Merged

FISH-5705 Yasson 1.0.9 #5458

merged 1 commit into from Nov 3, 2021

Conversation

mkarg
Copy link
Contributor

@mkarg mkarg commented Oct 18, 2021

Please consider updating Yasson 1.0.6 to 1.0.9 in next release of Payara Community 5.x

Yasson 1.0.6 contains several serialization failures which are fixed in 1.0.9 and it took me days to work around them, so to prevent others to run in the same troubles, this update should get adopted. As it is a pure bug fix it should not imply any new bugs.

Signed-off-by: Markus KARG <markus@headcrashing.eu>
@AlanRoth AlanRoth self-assigned this Oct 18, 2021
@AlanRoth AlanRoth added PR: CLA CLA submitted on PR by the contributor Type: Community Contribution labels Oct 18, 2021
@rdebusscher
Copy link
Contributor

Hi Markus,

Thank you for this PR and we will have a look with our testing if this presents any backward compatibility issues (which was the case when moving from 1.0.4 to 1.0.6).

Rudy

@rdebusscher
Copy link
Contributor

jenkins test please

@mkarg
Copy link
Contributor Author

mkarg commented Oct 21, 2021

Thanks, Rudy. In fact, 1.0.6 is unusable for us, so we had to rework our application to use JSON-P instead of JSON-B. Staying on 1.0.6 forever is not a solution. If 1.0.9 induces a problem, we should file a PR to fix it.

@Pandrex247 Pandrex247 merged commit eb07a85 into payara:master Nov 3, 2021
@AlanRoth AlanRoth changed the title Yasson 1.0.9 FISH-5705 Yasson 1.0.9 Nov 9, 2021
aubi pushed a commit to aubi/Payara that referenced this pull request Jan 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: CLA CLA submitted on PR by the contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants