Public Release r2.2 (Fall25 M4) for BlockchainPublicAddress APIs by PedroDiez · Pull Request #109 · camaraproject/BlockchainPublicAddress

PedroDiez · 2025-08-19T13:34:05Z

What type of PR is this?

documentation
subproject management

What this PR does / why we need it:

PR for Public Release r2.2 Generation for BlockChain Public Address APIs

Which issue(s) this PR fixes:

Fixes #79
Fixes #86

Special notes for reviewers:

To review after PR #108 merge

Changelog input

 Public Release r2.2 for Blockchain Public Address APIs

Additional documentation

This section can be blank.

docs

…ddress into r2.2

CHANGELOG.md

Kevsy · 2025-09-04T15:43:25Z

CHANGELOG.md

+- **This version contains changes compared to v0.2.0:**
+  - Removal of `401 - AUTHENTICATION_REQUIRED`
+  - New pattern for `x-correlator`
+  - Addition of a new section `# Additional CAMARA error responses` in `info.description` field
+  - Addition of `422 - BLOCKCHAIN_PUBLIC_ADDRESS.UNSUPPORTED_ENHANCED_VALIDATION`
+  - Addition of `400 - BLOCKCHAIN_PUBLIC_ADDRESS.INVALID_NONCE`
+  - Addition of `400 - BLOCKCHAIN_PUBLIC_ADDRESS.INVALID_SIGNATURE`
+  - Addition of `400 - BLOCKCHAIN_PUBLIC_ADDRESS.BOTH_NONCE_SIGNATURE_REQUIRED`


Please can you group these into

Added

Changed

Fixed

Removed

I prefer removing those Lines 36 - 42 as already mentioned below (so avoiding unnecessary duplication)

Added

Binding enforcement procedure, including addition of new errors 422 - BLOCKCHAIN_PUBLIC_ADDRESS.UNSUPPORTED_ENHANCED_VALIDATION, 400 - BLOCKCHAIN_PUBLIC_ADDRESS.INVALID_NONCE, 400 - BLOCKCHAIN_PUBLIC_ADDRESS.INVALID_SIGNATURE and 400 - BLOCKCHAIN_PUBLIC_ADDRESS.BOTH_NONCE_SIGNATURE_REQUIRED in Enforcement of BlockChain Public Address Ownership #76, Binding Enforcement - Off-Chain Solution #95 and Enforce Binding Off Chain Solution Editorial Enhancements #97.

New section # Additional CAMARA error responses in info.description field in Blockchain Public Address RC Tranversal WG Aligment Fall 25 #99.

Changed

New pattern for x-correlator and model aligment with Commonalities in Blockchain Public Address RC Tranversal WG Aligment Fall 25 #99.

Update User Stories adding a new user story for Binding Enforcement and making some wording alignment in Fall 25 Blockchain Public Address User Stories Update #100.

Update Gherkin feature files, adding a new test for testing invalid x-correlator value in all API endpoints, adding a specific test to check binding enforcement in bindBlockchainPublicAddress endpoint and making some wording alignments with Commonalities in Fall25 Blockchain Public Address Gherkin Update #101 and APIs aligment with Transversal WGs Public Releases Fall25 #108.

Error schema model alignment with Commonalities in APIs aligment with Transversal WGs Public Releases Fall25 #108.

Fixed

N/A

Removed

401 - AUTHENTICATION_REQUIRED error in Blockchain Public Address RC Tranversal WG Aligment Fall 25 #99.

Kevsy · 2025-09-04T15:48:09Z

code/API_definitions/blockchain-public-address.yaml

@@ -58,13 +58,13 @@ info:
  license:


Line 28: Please change the link to the "Binding Enforcement" image : it currently includes /main/ in the path, which must be replace with /r2.2/

README.md

Kevsy

For blockchain-public-address.yaml:

As well as the inline comments, please can you change the the documentation here (additions in bold just for illustration):

L5: "This API allows the API consumer to generate a one-time nonce for Enhanced Binding validation for Blockchain Public Address associated with a mobile phone number."

L12 "This API allows the API consumer to generate a one-time nonce" etc.

Kevsy

For blockchain-public-address-validation.yaml, please can you add the text shown in bold below in the info.description

L5: This API allows the API consumer to generate a one-time nonce for Enhanced Binding validation for Blockchain Public Address associated with a mobile phone number.

Kevsy · 2025-09-04T16:14:44Z

Note there is another open approved PR (adding a codeowner). Please can you either:

merge it into main, and then merge main into the Release PR (rebase)
or mark the codeowner PR as draft?

Co-authored-by: Kevin Smith <Kevsy@users.noreply.github.com>

…ddress into r2.2

PedroDiez · 2025-09-09T17:12:42Z

For blockchain-public-address.yaml:

As well as the inline comments, please can you change the the documentation here (additions in bold just for illustration):

L5: "This API allows the API consumer to generate a one-time nonce for Enhanced Binding validation for Blockchain Public Address associated with a mobile phone number."

L12 "This API allows the API consumer to generate a one-time nonce" etc.

ok, i will re-write in order to not miss the main point of the API

…ddress into r2.2

PedroDiez · 2025-09-09T17:27:49Z

Ready again for review @Kevsy, @grgpapadopoulos, @rartych

Kevsy

Approved on behalf of Release Management 👏

Next steps for the team:

PR merged (by API repository codeowner)
Release created within GitHub (by API repository codeowner)
Release Tracker updated (with creation date of the release and the release tag link)

PedroDiez · 2025-09-11T10:07:09Z

Merging PR and proceeding with release tag generation

rartych

LGTM

PedroDiez · 2025-09-11T10:59:01Z

Release tag generated: https://github.com/camaraproject/BlockchainPublicAddress/releases/tag/r2.2

PedroDiez · 2025-09-11T11:01:42Z

API release trackers updated:
https://lf-camaraproject.atlassian.net/wiki/spaces/CAM/pages/111280345/blockchain-public-address+v0.3.0
https://lf-camaraproject.atlassian.net/wiki/spaces/CAM/pages/111607836/blockchain-public-address-validation+v0.1.0

Public Release r2.2 for BlockchainPublicAddress APIs

ad31d47

PedroDiez self-assigned this Aug 19, 2025

PedroDiez requested a review from a team as a code owner August 19, 2025 13:34

PedroDiez added the documentation Improvements or additions to documentation label Aug 19, 2025

PedroDiez requested review from grgpapadopoulos and ksl4dtit as code owners August 19, 2025 13:34

PedroDiez added the repository management Indicating issues with the repository or release management process label Aug 19, 2025

PedroDiez marked this pull request as draft August 19, 2025 13:34

Kevsy changed the title ~~Public Release r2.2 for BlockchainPublicAddress APIs~~ Public Release r2.2 (Fall'25 M4) for BlockchainPublicAddress APIs Aug 21, 2025

Kevsy mentioned this pull request Aug 21, 2025

BlockchainPublicAddress r2.2 (Fall25 M4) release review camaraproject/ReleaseManagement#291

Closed

34 tasks

PedroDiez changed the title ~~Public Release r2.2 (Fall'25 M4) for BlockchainPublicAddress APIs~~ Public Release r2.2 (Fall25 M4) for BlockchainPublicAddress APIs Aug 22, 2025

grgpapadopoulos approved these changes Aug 26, 2025

View reviewed changes

PedroDiez added 2 commits August 26, 2025 15:11

Merge branch 'main' of https://github.com/PedroDiez/BlockchainPublicA…

bd4500f

…ddress into r2.2

align_with_main

cd304a5

PedroDiez marked this pull request as ready for review August 26, 2025 13:14

grgpapadopoulos previously approved these changes Aug 27, 2025

View reviewed changes

Kevsy reviewed Sep 4, 2025

View reviewed changes

CHANGELOG.md Outdated Show resolved Hide resolved

Kevsy reviewed Sep 4, 2025

View reviewed changes

README.md Outdated Show resolved Hide resolved

Kevsy reviewed Sep 4, 2025

View reviewed changes

Update CHANGELOG.md

1677395

Co-authored-by: Kevin Smith <Kevsy@users.noreply.github.com>

PedroDiez dismissed grgpapadopoulos’s stale review via 1677395 September 9, 2025 16:21

PedroDiez requested a review from rartych as a code owner September 9, 2025 16:21

PedroDiez and others added 2 commits September 9, 2025 18:22

Update README.md

6cd483f

Co-authored-by: Kevin Smith <Kevsy@users.noreply.github.com>

Merge branch 'main' of https://github.com/PedroDiez/BlockchainPublicA…

67cf2e9

…ddress into r2.2

updates_after_1st_rm_review

8f0db36

PedroDiez added 2 commits September 9, 2025 19:23

Merge branch 'r2.2' of https://github.com/PedroDiez/BlockchainPublicA…

7d3331f

…ddress into r2.2

fix_editorial_typo

c74d6cd

PedroDiez requested review from Kevsy and grgpapadopoulos September 9, 2025 17:26

Kevsy approved these changes Sep 10, 2025

View reviewed changes

rartych approved these changes Sep 11, 2025

View reviewed changes

PedroDiez merged commit ff5121c into camaraproject:main Sep 11, 2025
2 checks passed

Conversation

PedroDiez commented Aug 19, 2025

What type of PR is this?

What this PR does / why we need it:

Which issue(s) this PR fixes:

Special notes for reviewers:

Changelog input

Additional documentation

Uh oh!

Uh oh!

Kevsy Sep 4, 2025

Choose a reason for hiding this comment

Added

Changed

Fixed

Removed

Uh oh!

PedroDiez Sep 9, 2025

Choose a reason for hiding this comment

Added

Changed

Fixed

Removed

Uh oh!

Kevsy Sep 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Kevsy left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Kevsy left a comment

Choose a reason for hiding this comment

Uh oh!

Kevsy commented Sep 4, 2025

Uh oh!

PedroDiez commented Sep 9, 2025

Uh oh!

PedroDiez commented Sep 9, 2025

Uh oh!

Kevsy left a comment • edited by PedroDiez Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

PedroDiez commented Sep 11, 2025

Uh oh!

rartych left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

PedroDiez commented Sep 11, 2025

Uh oh!

PedroDiez commented Sep 11, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Kevsy Sep 4, 2025 •

edited

Loading

Kevsy left a comment •

edited

Loading

Kevsy left a comment •

edited by PedroDiez

Loading

PedroDiez commented Sep 11, 2025 •

edited

Loading