Skip to content

Match a Capitalized Secrets Directory in the Runtime-Secrets Check #3842

Open

Description

@ptr727

Problem

The runtime-secrets check in spec/audit.py (from #3719, PR #3838) matches a secrets directory's name case-sensitively. A bare directory spelled with a capital, such as Secrets/, breaks the runtimesecrets.naming rule just as secrets/ does, yet no tracked signal naming it is reported. The tree loop's existing .secrets match is case-sensitive in the same way, so a .Secrets/ directory is not judged either.

Constructed case

.gitignore    Secrets/
compose.yml   secrets: s: file: ./Secrets/Db_Password

Expected: a runtime-secrets finding naming the bare directory. Actual: no finding.

Suggested direction

Match the folder name case-insensitively in the .gitignore reader, the reference patterns, and the tree loop, and report a capitalized spelling as drift from .secrets/.

Activity

  1. ptr727 commented on Nov 3, 2026

    @ptr727
    OwnerAuthor

    A related line form, from the local review of PR #3838: a .gitignore line with a character class in the folder segment, such as [Ss]ecrets/, makes git ignore a lowercase secrets/. ignored_secrets_folder matches only a segment that is exactly secrets, so a tree holding only that .gitignore reports nothing. A case-insensitive match should cover this form too.

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions