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

Fix problem introduced in JAVASERVERFACES_SPEC_PUBLIC-1329, also cons… #46

Merged
merged 1 commit into from
Feb 27, 2020

Conversation

soul2zimate
Copy link

@soul2zimate soul2zimate commented Oct 12, 2019

…ider Submitted value "" when EMPTY_STRING_AS_NULL_PARAM_NAME is enabled.

(cherry picked from commit b1111429d0d66352ea8911c7abc210f1fec5684c)
(cherry picked from commit f3507c74b6fbb37797f3da235d8c0a88588b2e06)

issue: https://issues.jboss.org/browse/WFLY-12491

upstream issue: eclipse-ee4j/mojarra#4550
PR: eclipse-ee4j/mojarra#4643

…ider Submitted value "" when EMPTY_STRING_AS_NULL_PARAM_NAME is enabled.

Signed-off-by: Chao Wang <[email protected]>
@soul2zimate
Copy link
Author

Hi @fjuma would you mind to review this for https://issues.redhat.com/browse/JBEAP-17499 at your convenience time ? the details report and analysis is in upstream issue: eclipse-ee4j/mojarra#4550

@fjuma
Copy link

fjuma commented Feb 10, 2020

@soul2zimate I added another comment on the upstream PR to try to get a review from them. My concern with this PR is that it is changing the expected values for test cases so ideally we should make sure that upstream is ok with this.

@spyrkob spyrkob merged commit f9bc119 into jboss:2.3.9.SP Feb 27, 2020
@spyrkob
Copy link

spyrkob commented Feb 27, 2020

Merged since upstream was approved and merged

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