Skip to content

[int] Check for hardcoded passwords#8565

Merged
fstagni merged 1 commit into
DIRACGrid:integrationfrom
sfayer:check_passwd
Jun 5, 2026
Merged

[int] Check for hardcoded passwords#8565
fstagni merged 1 commit into
DIRACGrid:integrationfrom
sfayer:check_passwd

Conversation

@sfayer
Copy link
Copy Markdown
Member

@sfayer sfayer commented Jun 2, 2026

Hi,

This is a simple check (part of #8547) for hardcoded passwords. In most cases it's just marking variables called "token" as safe as this isn't used for a security token (we could equally set these to None, they're not used). I've removed the hardcoded default password from the database base class: This is always overridden, so doesn't change any behaviour.

Regards,
Simon

BEGINRELEASENOTES
*All
CHANGE: Check for hardcoded passwords
ENDRELEASENOTES

@sfayer sfayer marked this pull request as ready for review June 4, 2026 21:49
@sfayer sfayer requested review from atsareg and fstagni as code owners June 4, 2026 21:49
@fstagni fstagni merged commit cc22f94 into DIRACGrid:integration Jun 5, 2026
23 checks passed
@DIRACGridBot DIRACGridBot added the sweep:ignore Prevent sweeping from being ran for this PR label Jun 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sweep:ignore Prevent sweeping from being ran for this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants