[CXF-8347] URI processing/caching/extending changes #698
Conversation
Includes: - URITemplate cache - Extension to allow extenders to wrap runnables from HTTPConduit - Numeric handling in URIs - URITemplate processes parameters - Add checking for form post when content-type includes charsets, etc. Signed-off-by: Andy McCright <[email protected]>
|
Rebased from master - was hoping that the force-push would kick off an integration build, but it doesn't seem to have... |
|
Sorry @andymc12, I haven't had time to review this PR yet (not forgotten), will do that shortly. The integration builds are not triggering right now since the CI server has changed the name (builds.apache.org -> ci-builds.apache.org), I don't have permissions to modify the repository settings to change webhook endpoints, @dkulp / @coheigea do you have admin rights, guys? |
|
@reta - no worries - it's a fairly big change, so I wasn't expecting instant turnaround. :) Please let me know if you have any questions on these changes. Thanks! |
|
@reta I think so - what do I need to change? |
thanks @coheigea , there are instructions on wiki: https://cwiki.apache.org/confluence/display/INFRA/Kicking+off+a+Jenkins+build+with+a+GitHub+PR (please feel free to ping me in private if you need any help). It seems like |
|
@reta For which CXF build on ci-builds should I make those changes in? |
@coheigea https://ci-builds.apache.org/job/CXF/job/CXF-JDK8/, this is out standard one, we could extend it to others later one |
|
@coheigea sorry, just realized we had a dedicated build previously just for PRs, I could clone https://ci-builds.apache.org/job/CXF/job/CXF-JDK8/, and name it like https://ci-builds.apache.org/job/CXF/job/CXF-JDK8-PR/, would be good for you? |
|
Yes please do, I've actually made the change already to "Check "Enable Git validated merge support" and "Build pull requests to the repository" in your build config". Not sure if the other changes are required? Once you copy the changes, I'll revert the changes on the original job... |
Includes:
These are part of the changes made in the Open Liberty "fork" of CXF.