Skip to content

fix(security): Dependabot/CodeQL, appsettings split, SemVer tag align - #24

Merged
0x-mhmdnzri merged 5 commits into
devfrom
fix/security-appsettings-semver
Aug 30, 2026
Merged

0x-mhmdnzri merged 5 commits into
devfrom
fix/security-appsettings-semver

Conversation

@0x-mhmdnzri

Copy link
Copy Markdown
Owner

Why a new PR?

PR #23 was already merged; later commits were pushed to ci/semver-versioning after merge and did not appear on that PR.

Changes

  • appsettings: shared defaults in appsettings.json; env-specific only in Development/Production
  • Dependabot: KubernetesClient 17.0.14; admin postcss 8.5.23
  • CodeQL: LogSanitizer (log forging); SendGrid no longer logs from-address
  • SemVer: version.json aligned to v0.2.0; tag/release naming documented

Test plan

  • CI green
  • After merge, Dependabot/CodeQL re-scan on dev

Keep topology, plugin shape, and defaults in appsettings.json;
environment-specific hosts, limits, CORS, and logging only in
appsettings.Development.json and appsettings.Production.json.
- pin KubernetesClient 17.0.14 (GHSA-w7r3-mgwf-4mqq via Aspire)
- bump admin postcss to 8.5.23 (multiple GHSA)
- sanitize request path/correlation/tenant before logging (cs/log-forging)
- stop logging SendGrid from-address details (cs/exposure-of-sensitive-information)
Match version.json to existing v0.2.0, standardize annotated tag
messages and GitHub Release titles, document valid/invalid tag names.
@cr-gpt

cr-gpt Bot commented Aug 30, 2026

Copy link
Copy Markdown

Seems you are using me but didn't get OPENAI_API_KEY seted in Variables/Secrets for this repo. you could follow readme for more information

Development no longer defaults ConnectionStrings:Redis to localhost.
Integration workflow forces empty Redis; multiplexer uses AbortOnConnectFail=false when Redis is configured.
@cr-gpt

cr-gpt Bot commented Aug 30, 2026

Copy link
Copy Markdown

Seems you are using me but didn't get OPENAI_API_KEY seted in Variables/Secrets for this repo. you could follow readme for more information

@0x-mhmdnzri
0x-mhmdnzri merged commit f9c1664 into dev Aug 30, 2026
13 of 14 checks passed
@0x-mhmdnzri
0x-mhmdnzri deleted the fix/security-appsettings-semver branch August 30, 2026 10:26
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