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

Fix uploaded artifacts should be overwritten (#28726) #28821

Closed

Conversation

GiteaBot
Copy link
Contributor

Backport #28726 by @fuxiaohei

Fix Uploaded artifacts should be overwritten #28549

When upload different content to uploaded artifact, it checks that content size is not match in db record with previous artifact size, then the new artifact is refused.

Now if it finds uploading content size is not matching db record when receiving chunks, it updates db records to follow the latest size value.

Fix `Uploaded artifacts should be overwritten`
go-gitea#28549

When upload different content to uploaded artifact, it checks that
content size is not match in db record with previous artifact size, then
the new artifact is refused.

Now if it finds uploading content size is not matching db record when
receiving chunks, it updates db records to follow the latest size value.
@GiteaBot GiteaBot added modifies/api This PR adds API routes or modifies them type/bug labels Jan 17, 2024
@GiteaBot GiteaBot added this to the 1.21.5 milestone Jan 17, 2024
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jan 17, 2024
@pull-request-size pull-request-size bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jan 17, 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 Jan 17, 2024
@a1012112796
Copy link
Member

@fuxiaohei please have a look, looks it can't be built in v1.21.

@lunny
Copy link
Member

lunny commented Jan 18, 2024

replaced by #28832

@lunny lunny closed this Jan 18, 2024
@GiteaBot GiteaBot removed this from the 1.21.5 milestone Jan 18, 2024
@techknowlogick techknowlogick deleted the backport-28726-v1.21 branch January 18, 2024 01:43
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/need 1 This PR needs approval from one additional maintainer to be merged. modifies/api This PR adds API routes or modifies them size/L Denotes a PR that changes 100-499 lines, ignoring generated files. type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants