Skip to content

Avoid non-posix environment variables - #92

Open
badtst wants to merge 1 commit into
mainfrom
fix/avoid-non-posix-env-vars
Open

Avoid non-posix environment variables#92
badtst wants to merge 1 commit into
mainfrom
fix/avoid-non-posix-env-vars

Conversation

@badtst

@badtst badtst commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Description

The option to replace the _ in configuration fields with - (kebab-case ?) for environment variables was a bad idea because it creates non-posix environment variables, which should only be composed of numbers, uppercase letters and underscores.

This could create issues in the future when working with some shells and already happened during testing while working withmake.

Instead of using __ as a field delimiter for the environment variables and keep the _ in filed names, we can choose to instead simply remove the _ in the fields (thus moving them from snake_case to nocase ?). It might be less visible but more compact.

Example:

Field name: qpu.tls_verify -> WARDEN_QPU_TLSVERIFY

CHANGED

  • Fields of the configuration now automatically nocase aliases instead of kebab-case aliases

@badtst badtst self-assigned this Aug 12, 2026
@badtst
badtst requested a review from MatthieuMoreau0 August 12, 2026 12:43
@github-actions

Copy link
Copy Markdown

⚠️ critical vulnerabilities were detected in generated SBOM artifacts

SBOM vulnerability check

⚠️ Found 1 critical vulnerability entries across SBOM artifacts.

Total vulnerabilities observed: 4

warden-sbom-mariadb.cdx.json

  • Severity breakdown: CRITICAL: 1, HIGH: 1
  • Critical vulnerabilities:
Vulnerability Severity Component(s) Reference
CVE-2025-65896 CRITICAL asyncmy@0.2.11 link

warden-sbom-pg.cdx.json

  • Severity breakdown: HIGH: 1

warden-sbom.cdx.json

  • Severity breakdown: HIGH: 1

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.

1 participant