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

Fixed the implementation of IntoResponse of (HeaderMap, T) and (StatusCode, HeaderMap, T) would ignore headers from T. #137

Merged
merged 2 commits into from
Aug 6, 2021

Conversation

sunli829
Copy link
Contributor

@sunli829 sunli829 commented Aug 6, 2021

Fixes: #136

sunli829 added a commit to sunli829/axum that referenced this pull request Aug 6, 2021
…erMap, T)` and `(StatusCode, HeaderMap, T)` would ignore headers from T. (tokio-rs#137)
sunli829 added a commit to sunli829/axum that referenced this pull request Aug 6, 2021
…erMap, T)` and `(StatusCode, HeaderMap, T)` would ignore headers from T. (tokio-rs#137)
@sunli829 sunli829 marked this pull request as ready for review August 6, 2021 07:11
Copy link
Member

@davidpdrsn davidpdrsn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

Wanna add a line to the changelog as well?

@sunli829 sunli829 changed the title Fixed the problem that the implementation of IntoResponse of (HeaderMap, T) and (StatusCode, HeaderMap, T) would ignore headers from T. Fixed the implementation of IntoResponse of (HeaderMap, T) and (StatusCode, HeaderMap, T) would ignore headers from T. Aug 6, 2021
…StatusCode, HeaderMap, T)` would ignore headers from T. (tokio-rs#137)
@sunli829
Copy link
Contributor Author

sunli829 commented Aug 6, 2021

Haha, I forgot it. 🙂

@davidpdrsn
Copy link
Member

Thanks! I'm gonna resolve the conflicts directly in github.

@davidpdrsn davidpdrsn merged commit a0ac8a5 into tokio-rs:main Aug 6, 2021
davidpdrsn added a commit that referenced this pull request Aug 6, 2021
0.1.3 (06. August, 2021)

- Fix stripping prefix when nesting services at `/` ([#91](#91))
- Add support for WebSocket protocol negotiation ([#83](#83))
- Use `pin-project-lite` instead of `pin-project` ([#95](#95))
- Re-export `http` crate and `hyper::Server` ([#110](#110))
- Fix `Query` and `Form` extractors giving bad request error when query string is empty. ([#117](#117))
- Add `Path` extractor. ([#124](#124))
- Fixed the implementation of `IntoResponse` of `(HeaderMap, T)` and `(StatusCode, HeaderMap, T)` would ignore headers from `T` ([#137](#137))
- Deprecate `extract::UrlParams` and `extract::UrlParamsMap`. Use `extract::Path` instead ([#138](#138))
@davidpdrsn davidpdrsn mentioned this pull request Aug 6, 2021
davidpdrsn added a commit that referenced this pull request Aug 6, 2021
- Fix stripping prefix when nesting services at `/` ([#91](#91))
- Add support for WebSocket protocol negotiation ([#83](#83))
- Use `pin-project-lite` instead of `pin-project` ([#95](#95))
- Re-export `http` crate and `hyper::Server` ([#110](#110))
- Fix `Query` and `Form` extractors giving bad request error when query string is empty. ([#117](#117))
- Add `Path` extractor. ([#124](#124))
- Fixed the implementation of `IntoResponse` of `(HeaderMap, T)` and `(StatusCode, HeaderMap, T)` would ignore headers from `T` ([#137](#137))
- Deprecate `extract::UrlParams` and `extract::UrlParamsMap`. Use `extract::Path` instead ([#138](#138))
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.

Returning (HeaderMap, T) removes headers from T's IntoResponse implementation
2 participants