Skip to content

fetchers: Restrict OBS log filename - #212

Merged
RytoEX merged 1 commit into
obsproject:masterfrom
RytoEX:restrict-obs-log-filenames
Oct 22, 2026
Merged

RytoEX merged 1 commit into
obsproject:masterfrom
RytoEX:restrict-obs-log-filenames

Conversation

@RytoEX

@RytoEX RytoEX commented Oct 22, 2026

Copy link
Copy Markdown
Member

Description

fetchers: Restrict OBS log filename

When uploaded to our server, OBS logs use a base64 alphabet for the filename. Reflect that here instead of allowing any characters to match.

Motivation and Context

No need to allow the regex to be so unrestricted.

How Has This Been Tested?

Tested fetching logs at obsproject.com with now valid characters and logs with invalid characters.

Types of changes

  • Tweak (non-breaking change to improve existing functionality)

Checklist:

  • I have read the contributing document.
  • My code has been run through clang-format.
  • My code follows the project's style guidelines
  • My code is not on the master branch.
  • My code has been tested.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • I have included updates to all appropriate documentation.

When uploaded to our server, OBS logs use a base64 alphabet for the
filename. Reflect that here instead of allowing any characters to match.
@RytoEX
RytoEX requested review from notr1ch and prgmitchell October 22, 2026 01:54
@RytoEX
RytoEX merged commit 904b1d1 into obsproject:master Oct 22, 2026
2 checks passed
@RytoEX
RytoEX deleted the restrict-obs-log-filenames branch October 22, 2026 15:06
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