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

Refactor eTag formatting logic into a static utility method #33412

Conversation

hyunmin0317
Copy link
Contributor

This pull request refactors the eTag formatting logic by consolidating it into a static format method within the ETag record class.
The changes aim to eliminate duplicated code and improve maintainability.

Summary of Changes:

  • Added a static format method to the ETag record class to handle eTag formatting.
  • Refactored various classes to use the new format method for eTag formatting.

Thank you for reviewing this pull request!

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Aug 20, 2024
@bclozel bclozel added the in: web Issues in web modules (web, webmvc, webflux, websocket) label Aug 20, 2024
@bclozel
Copy link
Member

bclozel commented Aug 20, 2024

To be considered when we'll work on #33385

@rstoyanchev rstoyanchev self-assigned this Sep 6, 2024
@rstoyanchev rstoyanchev added this to the 6.2.0-RC1 milestone Sep 6, 2024
@rstoyanchev rstoyanchev added type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Sep 6, 2024
rstoyanchev pushed a commit that referenced this pull request Sep 10, 2024
rstoyanchev added a commit that referenced this pull request Sep 10, 2024
Update checks whether quoting is needed to be more complete
than what we've used so far, making sure the there is both
opening and closing quotes independent of each other.

See gh-33412
rstoyanchev added a commit that referenced this pull request Sep 10, 2024
This aligns HttpHeaders with other places like ServletWebRequest and
DefaultWebExchange where an ETag is accepted as input.

It also allows us to remove quoting from places that delegate to
HttpHeaders#setETag since it now does that internally.

Closes gh-33412
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: web Issues in web modules (web, webmvc, webflux, websocket) type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants