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

Improved library API and added automatic HttpClient disposal #5

Closed
wants to merge 4 commits into from
Closed

Conversation

snake-4
Copy link

@snake-4 snake-4 commented Apr 17, 2021

This PR adds the first three of the features suggested here: #4

  • Added automatic decompression of Brotli.
  • Added HttpVersion field to HttpRequest class to allow changing the HTTP protocol version.
  • Renamed ContentType field in HttpRequest to PayloadSerializationType to avoid confusion between ``Content-Type` header.
  • Removed Newtonsoft.Json dependency, used System.Text.Json instead.
  • Updated Flurl dependency.
  • Added automatic client disposal to HttpClientFactory.
  • Changed the Proxy field's type in HttpRequest to IWebProxy to allow user-defined classes to be used.
  • Updated README.md accordingly.

added Brotli automatic decompression and added a new HttpRequest field called HttpVersion to set the HTTP protocol version.
to avoid confusion between this field and the Content-Type HTTP header.
@matteocontrini
Copy link
Owner

Hey, you probably don't even remember this. Sorry for the delay. I'm going to release v1.3.0 with some of these changes:

https://github.com/matteocontrini/PlainHttp/releases/tag/v1.3.0

These are the non-breaking ones which I'm comfortable with releasing right now. The other changes need a bit more thought IMO, or they are breaking changes that require a 2.0 to be tagged in order to follow semantic versioning.

I'm going to close this for now but thanks for your helpful contributions!

@matteocontrini matteocontrini mentioned this pull request Nov 10, 2023
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.

None yet

2 participants