Skip to content

Conversation

@HenrikHL
Copy link
Contributor

@HenrikHL HenrikHL commented Oct 20, 2025

SD-2523: Add auditReference to Surrender and PINT APIs

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds an auditReference field to endorsement chain transactions in both the PINT and Surrender APIs to support audit trail tracking. The field allows eBL Solution Providers to record audit identifiers for verification purposes.

  • Added auditReference property to the EndorsementChainLink schema in the PINT API
  • Added auditReference property to the Transaction schema in the Surrender API

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
pint/v3/EBL_PINT_v3.0.0.yaml Added optional auditReference field to endorsement chain link with 100 character limit
ebl/v3/surrender/EBL_SUR_v3.0.2.yaml Added optional auditReference field to transaction with 100 character limit

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@qodo-code-review
Copy link

PR Compliance Guide 🔍

Below is a summary of compliance checks for this PR:

Security Compliance
🟢
No security concerns identified No security vulnerabilities detected by AI analysis. Human verification advised for critical code.
Ticket Compliance
🟡
🎫 #SD-2523
🟢 Add a new optional attribute named 'Audit reference' to the EndorsementChainLink (EBL SUR
API).
The attribute must be a string with a maximum length of 100 characters.
Provide a description stating it is an identifier issued by the eBL Solution Provider for
auditing to verify endorsement chain actions are securely recorded; format may vary by
provider technology.
Ensure the attribute is optional (not part of the required properties).
Include the new attribute in the relevant PINT API schema as well (Transaction), aligning
with EBL 3.x specifications.
Replace the previously added and removed 'signatureHash' attribute with this new
attribute.
Codebase Duplication Compliance
Codebase context is not defined

Follow the guide to enable codebase context checks.

Custom Compliance
No custom compliance provided

Follow the guide to enable custom compliance check.

Compliance status legend 🟢 - Fully Compliant
🟡 - Partial Compliant
🔴 - Not Compliant
⚪ - Requires Further Human Verification
🏷️ - Compliance label

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 2 out of 3 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@qodo-code-review
Copy link

PR Code Suggestions ✨

No code suggestions found for the PR.

@HenrikHL HenrikHL merged commit f54db46 into master Oct 21, 2025
1 check passed
@HenrikHL HenrikHL deleted the SD-2523_Add-auditReference branch October 21, 2025 05:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants