Skip to content

fix: align API with CAMARA Commonalities r4.3 validation requirements - #42

Merged
maheshc01 merged 1 commit into
mainfrom
fix/camara-validation-warnings-r4.3
Jul 21, 2026
Merged

fix: align API with CAMARA Commonalities r4.3 validation requirements#42
maheshc01 merged 1 commit into
mainfrom
fix/camara-validation-warnings-r4.3

Conversation

@maheshc01

Copy link
Copy Markdown
Contributor

Summary

Fixes all 29 CAMARA validation warnings identified in the r4.3 validation run (issue #41), to unblock the Sync26 meta-release snapshot.

Changes

Mandatory info.description templates (P-026)

  • Added authorization-and-authentication block
  • Added additional-error-responses block
  • Added request-body-strictness block
  • Added identifying-device-from-access-token block

Common schema references (replaces all inline definitions)

  • securitySchemes/openIdCAMARA_common.yaml
  • headers/x-correlatorCAMARA_common.yaml
  • parameters/x-correlatorCAMARA_common.yaml (also updated inline path param to use component ref)
  • Device, DeviceResponse, PhoneNumber, NetworkAccessIdentifier, DeviceIpv4AddrDeviceIpv4Address, SingleIpv4AddrSingleIpv4Address, Port (min:1), SingleIpv6AddrDeviceIpv6Address deleted and replaced with CAMARA_common.yaml refs
  • XCorrelator, ErrorInfo deleted and replaced with CAMARA_common.yaml refs
  • All Generic400/401/403/404/422/429 inline responses replaced with CAMARA_common.yaml refs

Server config

  • Set x-camara-commonalities: wip
  • Fixed apiRoot description to single-line standard CAMARA text (S-023)

String constraints (S-312)

  • maxLength: 36 added to ApplicationEndpointsId, AppId, EdgeCloudZoneId, ApplicationProfileId
  • maxLength: 64 added to EdgeCloudRegion, EdgeCloudProvider, EdgeCloudZoneName
  • format: ipv4, maxLength: 15 added to Ipv4Address (was missing format entirely)
  • format: ipv6, maxLength: 45 added to Ipv6Address (was missing format entirely)
  • maxLength: 253 and FQDN pattern added to Fqdn
  • maxLength: 2048, pattern: ^[^\r\n]*$ added to applicationEndpointDescription
  • maxLength: 256, pattern: ^[^\r\n]*$ added to applicationServerProviderName

Array constraints (S-309)

  • maxItems: 50 added to applicationEndpoints
  • maxItems: 1 added to ipv4Addresses and ipv6Addresses (one address per endpoint)

Placeholder file removal

  • Deleted documentation/API_documentation/Application-Endpoint-Discovery-API-Readiness-Checklist.md (P-032)

Test plan

  • CAMARA validation check passes with 0 errors, 0 warnings
  • All $ref paths resolve correctly against CAMARA_common.yaml
  • API semantics unchanged (optimal endpoint discovery flow intact)

- Add 3 mandatory CAMARA info.description template blocks (authorization-and-authentication, additional-error-responses, request-body-strictness, identifying-device-from-access-token)
- Set x-camara-commonalities: wip
- Fix apiRoot description to single-line standard CAMARA text (S-023)
- Replace inline openId, x-correlator header/parameter with CAMARA_common.yaml refs
- Replace inline Device, DeviceResponse, PhoneNumber, NetworkAccessIdentifier, DeviceIpv4Addr, SingleIpv4Addr, Port, SingleIpv6Addr schemas with CAMARA_common.yaml refs
- Replace inline XCorrelator, ErrorInfo schemas with CAMARA_common.yaml refs
- Replace all Generic400/401/403/404/422/429 inline responses with CAMARA_common.yaml refs
- Update inline x-correlator path parameter to use components/parameters ref
- Add maxLength: 36 to ApplicationEndpointsId, AppId, EdgeCloudZoneId, ApplicationProfileId (S-312)
- Add maxLength: 64 to EdgeCloudRegion, EdgeCloudProvider, EdgeCloudZoneName (S-312)
- Add format: ipv4, maxLength: 15 to Ipv4Address (S-310, S-312)
- Add format: ipv6, maxLength: 45 to Ipv6Address (S-312)
- Add maxLength: 253 and FQDN pattern to Fqdn (S-312, S-313)
- Add maxLength: 2048 and pattern to applicationEndpointDescription (S-312, S-313)
- Add maxLength: 256 and pattern to applicationServerProviderName (S-312, S-313)
- Add maxItems: 50 to applicationEndpoints array (S-309)
- Add maxItems: 1 to ipv4Addresses and ipv6Addresses arrays (S-309)
- Delete API-Readiness-Checklist.md (P-032)
@camara-validation

Copy link
Copy Markdown

CAMARA Validation — PASS

0 errors, 0 warnings, 0 hints | Profile: standard

View full results

@urvika-v urvika-v left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@Kevsy Kevsy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@maheshc01

Copy link
Copy Markdown
Contributor Author

thank you for the review.. will merge it and create the snapshot for the release.

@maheshc01
maheshc01 merged commit 1414b03 into main Jul 21, 2026
3 checks passed
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