Skip to content

Update x-correlator pattern#49

Merged
maxl2287 merged 4 commits intocamaraproject:mainfrom
maxl2287:feature/documented-errorresponses-info-section
Jun 25, 2025
Merged

Update x-correlator pattern#49
maxl2287 merged 4 commits intocamaraproject:mainfrom
maxl2287:feature/documented-errorresponses-info-section

Conversation

@maxl2287
Copy link
Contributor

What type of PR is this?

Add one of the following kinds:

  • correction

What this PR does / why we need it:

Updates the pattern of the x-correlator

Which issue(s) this PR fixes:

Fixes #48

@maxl2287 maxl2287 requested a review from sachinvodafone as a code owner June 16, 2025 21:02
@maxl2287 maxl2287 self-assigned this Jun 16, 2025
@maxl2287 maxl2287 added Fall25 Meta-release Fall25 correction Something needs to be corrected labels Jun 16, 2025
@maxl2287 maxl2287 changed the title feat: update x-correlator pattern Update x-correlator pattern Jun 16, 2025
@eric-murray
Copy link
Contributor

Hi @maxl2287

Commonalities have added an XCorrelator schema to CAMARA_common.yaml.

The request is to use this schema in OAS definitions as test cases will be updated to refer explicitly to this schema. Currently, tests tend to set x-correlator to a UUID and the test follows from there, but some companies find this too restrictive, and just want to test that x-correlator complies with the schema.

@rartych will confirm this if you want details.

@maxl2287
Copy link
Contributor Author

Hi @maxl2287

Commonalities have added an XCorrelator schema to CAMARA_common.yaml.

The request is to use this schema in OAS definitions as test cases will be updated to refer explicitly to this schema. Currently, tests tend to set x-correlator to a UUID and the test follows from there, but some companies find this too restrictive, and just want to test that x-correlator complies with the schema.

@rartych will confirm this if you want details.

Thanks @eric-murray,

Yes I saw it as well right now in the DeviceLocation call as well :)
Will update this PR asap.

@maxl2287
Copy link
Contributor Author

@eric-murray please take a look again :)

@maxl2287 maxl2287 merged commit bcce27f into camaraproject:main Jun 25, 2025
2 checks passed
@maxl2287 maxl2287 deleted the feature/documented-errorresponses-info-section branch June 25, 2025 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

correction Something needs to be corrected Fall25 Meta-release Fall25

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Align x-correlator format with guidelines

3 participants