This repository has been archived by the owner on Jan 5, 2022. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 43
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Collaborator
cjstehno
commented
Sep 20, 2017
- Add support for OPTIONS and TRACE request methods #157: Add support for OPTIONS and TRACE request methods
- Parsing of responses with empty body and no Content-Type #149: Parsing of responses with empty body and no content-type
- Add optional shadow jar distributions? #155: Add optional shadow jar distributions
- Problem encoding "/" as %2F for path elements in request URI #162: Problem encoding / as %2F for path elements in URI
- UTF-8 encoding and missing Accept-Charset header #169: UTF-8 encoding and missing Accept-Charset header
- Visibility of request details #163: Visibility of request details
* Added contributing file * Pulled Apache client into separate project (#52) * moved core code into core project - site not working, and both clients still in this code base * Moved out apache client and started splitting tests between the two projects. * Finished splitting up the client tests * pulled common config up into parent build file * Updated documentation and fixed site-building functionality. * Fixed cookie tests * Removed TODO * Fixed release verification tasks. * Updated contribution doc. * Refactored the http method testing into tests kits to share code. * Fixed #51: created shared http method testing kit and updated documentation. * Finishing up the test kit
* Working on #41 - adding okhttp as a client option. Code-complete at this point but needs refactoring and documentation * Working on #41 - mostly done, but needs documentation and site integration updates. * Finished testing the okhttp integration. * Refactored duplicate handler code into a shared function. * Updated documentation and did a bit of cleanup. * Cleaned up some of the build duplication. * Added some docs and testing around some of the "optional" components.
Converted the mock server based testing to use the Ersatz mock server.
…ating/using them.
Refactored and unified cookie handling.
…-ng into development
Added multipart request/response handling.
…-ng into development
Adds the "ignoreSslIssues" support back in.
Fixed test flakiness (Ersatz update) Upgraded Gradle and Groovy
Adds auto-sync from Bintray to Maven Central repo.
…e values properly and fixed the rendering in general. Also updated the license headers for 2017.
…r null when searching queries and the map is never null. Added traverser for finding a non-empty (and non-null) map.
Integrated new version of Ersatz and proxy mocking support.
At least a partial fix for handling encoded parts of a URI path.
Added verb to request config object and tweaked some logging details.
@dwclark give this a look if you want and let me know if you find anything odd. I am planning to cut the release on Friday morning. |
dwclark
approved these changes
Sep 21, 2017
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Thanks for the hard work. I'm excited to see the 1.0 release!
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.