Skip to content

PO-2337#163

Open
max-holland wants to merge 11 commits intomainfrom
PO-2337
Open

PO-2337#163
max-holland wants to merge 11 commits intomainfrom
PO-2337

Conversation

@max-holland
Copy link
Copy Markdown

@max-holland max-holland commented Oct 27, 2025

Jira link

See PO-2337

Change description

  • Introduce content digest

Testing done

Security Vulnerability Assessment

CVE Suppression: Are there any CVEs present in the codebase (either newly introduced or pre-existing) that are being intentionally suppressed or ignored by this commit?

  • Yes
  • No

Checklist

  • commit messages are meaningful and follow good commit message guidelines
  • README and other documentation has been updated / added (if needed)
  • tests have been updated / new tests has been added (if needed)
  • Does this PR introduce a breaking change

Copy link
Copy Markdown
Contributor

@iamfrankiemoran iamfrankiemoran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You'll also need to bump the version number in the package.json file
@max-holland

@iamfrankiemoran iamfrankiemoran changed the title Add digest verification middleware and integrate with Opal API proxy PO-2337 Oct 30, 2025
@iamfrankiemoran iamfrankiemoran marked this pull request as ready for review October 30, 2025 14:56
Copy link
Copy Markdown
Contributor

@iamfrankiemoran iamfrankiemoran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown

@louisbriggs louisbriggs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sonarqubecloud
Copy link
Copy Markdown

@sonarqubecloud
Copy link
Copy Markdown

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 1, 2026

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.

3 participants