Skip to content

fix(forward): move secret item path to /secrets/{name} (INT-1666)#623

Merged
armando-rodriguez-cko merged 2 commits into
masterfrom
fix/INT-1666-forward-secrets-path
Jul 24, 2026
Merged

fix(forward): move secret item path to /secrets/{name} (INT-1666)#623
armando-rodriguez-cko merged 2 commits into
masterfrom
fix/INT-1666-forward-secrets-path

Conversation

@armando-rodriguez-cko

Copy link
Copy Markdown
Contributor

Completes the Forward secrets path rename (swagger 2026-07-20): the item endpoints PATCH/DELETE moved from /forward/secrets/{name} to /secrets/{name}, dropping the /forward prefix so the item matches the collection already on /secrets.

Jira: INT-1666
Tests: green (forward suite).

🤖 Generated with Claude Code

The forward secrets collection endpoints (create, list) moved from
/forward/secrets to /secrets, while the per-item endpoint remains at
/forward/secrets/{name}. Reflects the incomplete rename in the swagger spec.
Swagger 2026-07-20 completed the rename: PATCH/DELETE moved from
/forward/secrets/{name} to /secrets/{name}. Update updateSecret/deleteSecret
(sync + async) and their unit tests to drop the /forward prefix.
@agent-wall-e

agent-wall-e Bot commented Jul 23, 2026

Copy link
Copy Markdown

🔴 Risk Classification: MAJOR

Approval route: AI Review + Human Approval Required
Rollback controls: Change-freeze window + documented rollback plan

Classification reasons

  • security_sensitive_path:src/test/java/com/checkout/forward/requests/SecretRequestSerializationTest.java
  • security_sensitive_path:src/test/java/com/checkout/forward/responses/SecretResponseSerializationTest.java

Operational gates

  • ✅ jira_ticket (INT-1666)
  • ✅ independent_review

Files analysed: 4


wall-e 2026.06.19-02 · policy 376219bc71e6…

@agent-wall-e

agent-wall-e Bot commented Jul 23, 2026

Copy link
Copy Markdown
🔬 Debug — why this classification?

Each reason code emitted by the classifier, its source clause in the AI in SDLC Control Framework, and what it means.

Reason code Kind Clause Meaning
security_sensitive_pathsrc/test/java/com/checkout/forward/requests/SecretRequestSerializationTest.java classifying §2.1 M4/M5 Path matched a sensitive pattern (auth, secrets, crypto, PCI, migrations, network IaC).
security_sensitive_pathsrc/test/java/com/checkout/forward/responses/SecretResponseSerializationTest.java classifying §2.1 M4/M5 Path matched a sensitive pattern (auth, secrets, crypto, PCI, migrations, network IaC).

Kinds:

  • classifying — this rule contributed to the chosen tier.
  • informational — context only; did not by itself decide the tier.

See issue #3 for the proposal to formalise this map as Appendix A of the standards doc.

wall-e 2026.06.19-02 · debug

@sonarqubecloud

Copy link
Copy Markdown

@armando-rodriguez-cko
armando-rodriguez-cko merged commit 1a9163a into master Jul 24, 2026
6 of 7 checks passed
@armando-rodriguez-cko
armando-rodriguez-cko deleted the fix/INT-1666-forward-secrets-path branch July 24, 2026 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants