Skip to content

Conversation

@KN4CK3R
Copy link
Member

@KN4CK3R KN4CK3R commented Apr 20, 2021

Backport #15370

  • Add ETag header.

  • Comply with RFC 7232.

  • Moved logic into httpcache.go

  • Changed name.

  • Lint

  • Implemented If-None-Match list.

  • Fixed missing header on *

  • Removed weak etag support.

  • Removed * support.

  • Added unit test.

  • Lint

Co-authored-by: Lunny Xiao [email protected]
Co-authored-by: techknowlogick [email protected]

* Add ETag header.

* Comply with RFC 7232.

* Moved logic into httpcache.go

* Changed name.

* Lint

* Implemented If-None-Match list.

* Fixed missing header on *

* Removed weak etag support.

* Removed * support.

* Added unit test.

* Lint

Co-authored-by: Lunny Xiao <[email protected]>
Co-authored-by: techknowlogick <[email protected]>
@lunny lunny added this to the 1.14.2 milestone Apr 20, 2021
@lunny lunny added the performance/speed performance issues with slow downs label Apr 20, 2021
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Apr 20, 2021
Copy link
Contributor

@kdumontnu kdumontnu left a comment

Choose a reason for hiding this comment

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

💯

@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Apr 20, 2021
@techknowlogick techknowlogick merged commit 5da024a into go-gitea:release/v1.14 Apr 20, 2021
@KN4CK3R KN4CK3R deleted the backport-15370 branch May 7, 2021 06:31
@go-gitea go-gitea locked and limited conversation to collaborators Jun 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. performance/speed performance issues with slow downs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants