Skip to content

Remove AUTHENTICATION_REQUIRED error code#12

Merged
akoshunyadi merged 6 commits intocamaraproject:mainfrom
eric-murray:eric-murray-patch-1
Jun 10, 2025
Merged

Remove AUTHENTICATION_REQUIRED error code#12
akoshunyadi merged 6 commits intocamaraproject:mainfrom
eric-murray:eric-murray-patch-1

Conversation

@eric-murray
Copy link
Contributor

@eric-murray eric-murray commented Jun 4, 2025

What type of PR is this?

  • correction

What this PR does / why we need it:

Commonalities pre-release r3.1 removes the AUTHENTICATION_REQUIRED error code, leaving UNAUTHENTICATED as the only valid error code when the HTTP status code is 401.

Which issue(s) this PR fixes:

Fixes #6
Fixes #7

Special notes for reviewers:

This is not a breaking change

Issue #7 only required a 401 test case naming correction for these APIs

Changelog input

 release-note
 - Remove AUTHENTICATION_REQUIRED error code

Additional documentation

None

sachinvodafone
sachinvodafone previously approved these changes Jun 4, 2025
bigludo7
bigludo7 previously approved these changes Jun 4, 2025
@eric-murray eric-murray dismissed stale reviews from bigludo7 and sachinvodafone via f2c6eb6 June 6, 2025 10:07
@akoshunyadi akoshunyadi merged commit 7788571 into camaraproject:main Jun 10, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Incorrect type-values in the test-definitions Remove AUTHENTICATION_REQUIRED error code

4 participants