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

Jump to bottom

Cache-Control header cleanup and new HeaderParserHelper trait #4545

Merged
merged 9 commits into from Mar 2, 2021
Merged

Cache-Control header cleanup and new HeaderParserHelper trait #4545

merged 9 commits into from Mar 2, 2021

Conversation

VlachJosef
Copy link
Contributor

@VlachJosef VlachJosef commented Feb 28, 2021 •
edited

partially address #4535

@VlachJosef VlachJosef force-pushed the cache-control branch 3 times, most recently from b7173e6 to fc9ebdd Compare February 28, 2021 16:00
@VlachJosef
Copy link
Contributor Author

Only two Recurring headers remains to be migrated:

  • Connection
  • Transfer-Encoding

They are both used by ServerTestRoutes, and since migration of ServerTestRoutes started in #4547 I'll wait with their migration until both branches are merged.

@VlachJosef VlachJosef mentioned this pull request Mar 1, 2021
Closed
24 tasks
@rossabaker rossabaker merged commit e1e1995 into http4s:series/0.22 Mar 2, 2021
@VlachJosef VlachJosef deleted the cache-control branch March 2, 2021 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@VlachJosef @rossabaker