Skip to content

Conversation

@neoeinstein
Copy link
Contributor

This should be compared to the implementation in #60.

@neoeinstein
Copy link
Contributor Author

I removed the const generic from my implementation, but of note it does mean that Default can no longer be implemented for this body type like it is for both Full and Empty. I'm not 100% sure of the value of the Default implementation, but if it's not necessary, then the non-generic version should be fine.

@LucioFranco
Copy link
Member

@davidpdrsn @seanmonstar is this something we'd like to release in 0.4? aka should we merge this before #56

@davidpdrsn
Copy link
Member

davidpdrsn commented May 20, 2022

I would appreciate that yes. Wanna make a middleware in tower-http to go along with this.

Co-authored-by: Programatik <programatik29@gmail.com>
@davidpdrsn davidpdrsn merged commit e17465c into hyperium:master May 20, 2022
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