Skip to content

feat(github_release): support Source code (zip/tar.gz) - #8584

Closed
nibazshab wants to merge 3 commits into
AlistGo:mainfrom
nibazshab:main
Closed

nibazshab wants to merge 3 commits into
AlistGo:mainfrom
nibazshab:main

Conversation

@nibazshab

@nibazshab nibazshab commented May 26, 2025 •

Copy link
Copy Markdown

抵制投毒行为


支持使用开关决定是否显示 Source code (zip/tar.gz)

解决:#8555

另外:alist-web 需要手动添加这个开关嘛?还是自动处理?如果需要手动修改的话,我就在那边也 pr 一下。

@petercat-assistant

Copy link
Copy Markdown

Walkthrough

This pull request introduces a feature that allows users to toggle the display of GitHub Release source code files (zip/tar.gz) in the application. This enhancement is controlled by a new configuration option ShowSourceCode.

Changes

Files Summary
drivers/github_releases/driver.go Added logic to append source code files to the list if ShowSourceCode is enabled.
drivers/github_releases/meta.go Introduced a new boolean field ShowSourceCode in the configuration struct, defaulting to false.
drivers/github_releases/types.go Implemented a new method GetSourceCode to retrieve source code files (zip/tar.gz) and append them to a list.

Comment thread drivers/github_releases/types.go Resolved
@nibazshab

Copy link
Copy Markdown
Author

已修复 TarballUrl 和 ZipballUrl 写反了的问题

@nibazshab nibazshab changed the title feat: support Github Release Source code (zip/tar.gz) feat(github(release): support Source code (zip/tar.gz) May 26, 2025
@nibazshab nibazshab changed the title feat(github(release): support Source code (zip/tar.gz) feat(github_release): support Source code (zip/tar.gz) May 26, 2025
@xrgzs

xrgzs commented May 26, 2025

Copy link
Copy Markdown
Contributor

alist-web 需要手动添加这个开关嘛?还是自动处理?如果需要手动修改的话,我就在那边也 pr 一下。

开关会自动加上,然后等翻译。

@nibazshab

nibazshab commented May 26, 2025 •

Copy link
Copy Markdown
Author

在开启 All Version 前提下显示 Source Code 会出错,新增了一个 GetSourceCodeByTagName(tagName string) 函数解决。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants