Skip to content

scripts/remoteassetify.py: Change the user agent#248

Merged
Z572 merged 1 commit intoopenRuyi-Project:mainfrom
dramforever:remoteasset-user-agent
Apr 28, 2026
Merged

scripts/remoteassetify.py: Change the user agent#248
Z572 merged 1 commit intoopenRuyi-Project:mainfrom
dramforever:remoteasset-user-agent

Conversation

@dramforever
Copy link
Copy Markdown
Contributor

I really do mean just "change". Apparently some servers don't like the original one because of some keyword matches. I'm not sure which ones, but hopefully this shorter one is less likely to trip one of those checks.


This unblocks #241. https://github.com/openRuyi-Project/openRuyi/actions/runs/25001574753/job/73334527795?pr=241

Poking around suggests that this server blocks the keyword "git" in user agent, but I don't think it's possible to know which ones exactly.

I really do mean just "change". Apparently some servers don't like the
original one because of some keyword matches. I'm not sure which ones,
but hopefully this shorter one is less likely to trip one of those
checks.

Signed-off-by: Vivian Wang <wangruikang@iscas.ac.cn>
@openruyi-bot openruyi-bot Bot added the Target: Rolling Targeting rolling/current branch. label Apr 28, 2026
@dramforever dramforever requested a review from Z572 April 28, 2026 08:05
@Z572 Z572 merged commit a756cf4 into openRuyi-Project:main Apr 28, 2026
1 check passed
@misaka00251 misaka00251 added Workflow: Complete Applied after all MRs have merged and the issue has been verified if necessary. CI Issue/PR related to CI. labels Apr 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Issue/PR related to CI. Target: Rolling Targeting rolling/current branch. Workflow: Complete Applied after all MRs have merged and the issue has been verified if necessary.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants