Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix wrong last modify time #32102

Merged
merged 3 commits into from
Sep 21, 2024
Merged

Fix wrong last modify time #32102

merged 3 commits into from
Sep 21, 2024

Conversation

lunny
Copy link
Member

@lunny lunny commented Sep 21, 2024

Fix #31930 and more places which use http.TimeFormat wrongly. http.TimeFormat requires a UTC time. refer to https://pkg.go.dev/net/http#TimeFormat

@lunny lunny added type/bug backport/v1.22 This PR should be backported to Gitea 1.22 labels Sep 21, 2024
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Sep 21, 2024
@pull-request-size pull-request-size bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Sep 21, 2024
@github-actions github-actions bot added the modifies/go Pull requests that update Go code label Sep 21, 2024
@pull-request-size pull-request-size bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Sep 21, 2024
@github-actions github-actions bot added the modifies/api This PR adds API routes or modifies them label Sep 21, 2024
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Sep 21, 2024
@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 Sep 21, 2024
@lunny lunny enabled auto-merge (squash) September 21, 2024 21:27
@lunny lunny merged commit a802508 into go-gitea:main Sep 21, 2024
26 checks passed
@GiteaBot GiteaBot added this to the 1.24.0 milestone Sep 21, 2024
GiteaBot pushed a commit to GiteaBot/gitea that referenced this pull request Sep 21, 2024
@GiteaBot GiteaBot added the backport/done All backports for this PR have been created label Sep 21, 2024
@lunny lunny modified the milestones: 1.24.0, 1.23.0 Sep 21, 2024
lunny added a commit that referenced this pull request Sep 22, 2024
Backport #32102 by @lunny

Fix #31930 and more places which use `http.TimeFormat` wrongly.
`http.TimeFormat` requires a UTC time. refer to
https://pkg.go.dev/net/http#TimeFormat

Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
zjjhot added a commit to zjjhot/gitea that referenced this pull request Sep 23, 2024
* giteaofficial/main:
  [skip ci] Updated licenses and gitignores
  Fix rename branch permission bug (go-gitea#32066)
  Fix artifact v4 upload above 8MB (go-gitea#31664)
  [skip ci] Updated translations via Crowdin
  Add bin to Composer Metadata (go-gitea#32099)
  Fix wrong last modify time (go-gitea#32102)
  Fix upload maven pacakge parallelly (go-gitea#31851)
  Repo Activity: count new issues that were closed (go-gitea#31776)
  Count typescript files as frontend for labeling (go-gitea#32088)
  Use camo.Always instead of camo.Allways (go-gitea#32097)
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Dec 21, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
backport/done All backports for this PR have been created backport/v1.22 This PR should be backported to Gitea 1.22 lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/api This PR adds API routes or modifies them modifies/go Pull requests that update Go code size/S Denotes a PR that changes 10-29 lines, ignoring generated files. type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The timezone in the last-modified header is specified incorrectly.
4 participants