Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Fix: RPM package download routing & missing package version count#34909

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

@ExplodingDragon
Copy link
Contributor

@ExplodingDragonExplodingDragon commentedJun 30, 2025
edited
Loading

Fix RPM package download routing (This is actually a regression bug from#27056)

Href:fmt.Sprintf("package/%s/%s/%s/%s-%s.%s.rpm",pd.Package.Name,pd.Version.Version,pd.FileMetadata.Architecture,pd.Package.Name,pd.Version.Version,pd.FileMetadata.Architecture),

Fix missing package version count

wxiaoguang and SuInk reacted with thumbs up emoji
@GiteaBotGiteaBot added the lgtm/need 2This PR needs two approvals by maintainers to be considered for merging. labelJun 30, 2025
@github-actionsgithub-actionsbot added modifies/apiThis PR adds API routes or modifies them modifies/goPull requests that update Go code labelsJun 30, 2025
@ExplodingDragonExplodingDragon changed the titleFix RPM package download routing & Fix missing package version countFix: RPM package download routing & missing package version countJun 30, 2025
ExplodingDragonand others added5 commitsJune 30, 2025 16:36
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>Signed-off-by: Exploding Dragon <explodingfkl@gmail.com>
@wxiaoguang
Copy link
Contributor

I think we can clarify the download behavior by1377b65 , does it look good to you?

ExplodingDragon reacted with thumbs up emoji

@GiteaBotGiteaBot added lgtm/need 1This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2This PR needs two approvals by maintainers to be considered for merging. labelsJun 30, 2025
@wxiaoguang
Copy link
Contributor

Hmm, the regression is related to Refactor packages (#34777).

Old code used regexp(...|) to allow empty filename. New code missed the empty part.

Thank you very much for figuring it out and add the test.

@GiteaBotGiteaBot added lgtm/doneThis PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1This PR needs approval from one additional maintainer to be merged. labelsJun 30, 2025
@wxiaoguangwxiaoguang merged commit09bb19a intogo-gitea:mainJun 30, 2025
26 checks passed
@GiteaBotGiteaBot added this to the1.25.0 milestoneJun 30, 2025
@ExplodingDragonExplodingDragon deleted the bugfix-rpm-and-pkg-versions branchJune 30, 2025 16:29
zjjhot added a commit to zjjhot/gitea that referenced this pull requestJul 1, 2025
* giteaofficial/main:  Fix modal + form abuse (go-gitea#34921)  [skip ci] Updated translations via Crowdin  Follow file symlinks in the UI to their target (go-gitea#28835)  Fix issue filter (go-gitea#34914)  Fix: RPM package download routing & missing package version count (go-gitea#34909)  Add support for 3D/CAD file formats preview (go-gitea#34794)
bdruth added a commit to bdruth/gitea that referenced this pull requestJul 4, 2025
…h/gitea into feature/enhanced-workflow-runs-api* 'feature/enhanced-workflow-runs-api' of github.com:bdruth/gitea:  [skip ci] Updated translations via Crowdin  Follow file symlinks in the UI to their target (go-gitea#28835)  Fix issue filter (go-gitea#34914)  Fix: RPM package download routing & missing package version count (go-gitea#34909)  Add support for 3D/CAD file formats preview (go-gitea#34794)  Add a `login`/`login-name`/`username` disambiguation to affected endpoint parameters and response/request models (go-gitea#34901)  Improve tags list page (go-gitea#34898)  [skip ci] Updated translations via Crowdin  docs: fix typo in pull request merge warning message text (go-gitea#34899)  Refactor container package (go-gitea#34877)  [skip ci] Updated translations via Crowdin
@go-giteago-gitea locked asresolvedand limited conversation to collaboratorsSep 29, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Reviewers

@wxiaoguangwxiaoguangwxiaoguang approved these changes

+1 more reviewer

@hiifonghiifonghiifong approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

lgtm/doneThis PR has enough approvals to get merged. There are no important open reservations anymore.modifies/apiThis PR adds API routes or modifies themmodifies/goPull requests that update Go code

Projects

None yet

Milestone

1.25.0

Development

Successfully merging this pull request may close these issues.

4 participants

@ExplodingDragon@wxiaoguang@hiifong@GiteaBot

[8]ページ先頭

©2009-2025 Movatter.jp