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

CXF-7860: Reprocess @FormParams prior to resource method invocation #5281

Merged
merged 1 commit into from
Oct 4, 2018

Conversation

andymc12
Copy link
Contributor

@andymc12 andymc12 commented Oct 3, 2018

This is a temporary fix (until CXF-7860 is pulled into a Liberty release). This fixes the issue documented in JAX-RS API Issue 659 - where a parameter annotated with @FormParam("value") might have an injected value that does not match a Form parameter's value.

This PR includes the same change submitted to the CXF community, and also a test case.

@LibbyBot
Copy link

LibbyBot commented Oct 3, 2018

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 1 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

  • 1 infrastructure code files were changed.

  • 9 FAT files were changed, added, or removed.

  • Check that the build did not break the affected FAT suite(s).

@andymc12
Copy link
Contributor Author

andymc12 commented Oct 3, 2018

#build

@LibbyBot
Copy link

LibbyBot commented Oct 3, 2018

Your personal build request is at https://wasrtc.hursley.ibm.com:9443/jazz/resource/itemOid/com.ibm.team.build.BuildResult/_mXlvsMclEeiA6ZPkBDHUqw

Target locations of links might be accessible only to IBM employees.

@LibbyBot
Copy link

LibbyBot commented Oct 3, 2018

The build andymc12-5281-20181003-1713
https://wasrtc.hursley.ibm.com:9443/jazz/resource/itemOid/com.ibm.team.build.BuildResult/_mXlvsMclEeiA6ZPkBDHUqw
completed and has errors or failures.

@andymc12
Copy link
Contributor Author

andymc12 commented Oct 3, 2018

#build - previous failed with unrelated ANT compilation errors...

@LibbyBot
Copy link

LibbyBot commented Oct 3, 2018

Your personal build request is at https://wasrtc.hursley.ibm.com:9443/jazz/resource/itemOid/com.ibm.team.build.BuildResult/_TYVMYMdREeiA6ZPkBDHUqw

Target locations of links might be accessible only to IBM employees.

@LibbyBot
Copy link

LibbyBot commented Oct 4, 2018

The build andymc12-5281-20181003-2223
https://wasrtc.hursley.ibm.com:9443/jazz/resource/itemOid/com.ibm.team.build.BuildResult/_TYVMYMdREeiA6ZPkBDHUqw
completed and has errors or failures.

@andymc12 andymc12 merged commit 34ee54c into OpenLiberty:integration Oct 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants