Skip to content

Conversation

pvgoran
Copy link
Contributor

@pvgoran pvgoran commented Jul 15, 2025

Resolves #35083

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jul 15, 2025
@github-actions github-actions bot added the modifies/go Pull requests that update Go code label Jul 15, 2025
@pvgoran
Copy link
Contributor Author

pvgoran commented Jul 15, 2025

It just occurred to be that the PR/commit title should say "repository archive" rather than just "archive". Not changing it now since https://github.com/go-gitea/gitea/blob/main/CONTRIBUTING.md#maintaining-open-prs possibly implies that I shouldn't. (Unless someone says it's OK to do.)

@lunny
Copy link
Member

lunny commented Jul 15, 2025

It's better to have a test for that.

@pvgoran
Copy link
Contributor Author

pvgoran commented Jul 15, 2025

It's better to have a test for that.

Where could a test be added, what would it test, and how would I run such a test? (I don't know a thing about testing facilities of Gitea - or of Go in general, for that matter.)

When I look at #33371 for inspiration, I only find something that looks like a test for the isFeedRequest() function. Here, do I need to test just the isArchivePath() function in a similar way, too?

@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 Jul 16, 2025
@wxiaoguang wxiaoguang added this to the 1.25.0 milestone Jul 16, 2025
@wxiaoguang wxiaoguang added the type/enhancement An improvement of existing functionality label Jul 16, 2025
@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 Jul 16, 2025
@lunny lunny added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Jul 16, 2025
@lunny lunny merged commit 891a827 into go-gitea:main Jul 16, 2025
26 checks passed
@GiteaBot GiteaBot removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Jul 16, 2025
zjjhot added a commit to zjjhot/gitea that referenced this pull request Jul 18, 2025
* giteaofficial/main:
  [skip ci] Updated translations via Crowdin
  Increase gap on latest commit (go-gitea#35104)
  Fix job status aggregation logic (go-gitea#35000)
  Fix some missed GitHeadRefName when renaming (go-gitea#35102)
  Fix error logs and improve some comments/messages (go-gitea#35105)
  Support Basic Authentication for archive downloads (go-gitea#35087)
  Run `uv run` with `--frozen` (go-gitea#35097)
  Improve package API log handling (go-gitea#35100)
  Rename pull request GetGitRefName to GetGitHeadRefName (go-gitea#35093)
  Fix review comment/dimiss comment x reference can be refereced back (go-gitea#35094)
  Fix submodule nil check (go-gitea#35096)
  Redirect to a presigned URL of HEAD for HEAD requests (go-gitea#35088)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/go Pull requests that update Go code type/enhancement An improvement of existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Basic authentication doesn't work for archive downloads
5 participants