Skip to content

fix: errorMapping in ResponseCodes with "4XX" and "5XX" Pattern#1735

Merged
baywet merged 2 commits into
microsoftgraph:mainfrom
EnsnerT:patch-1
Sep 9, 2024
Merged

fix: errorMapping in ResponseCodes with "4XX" and "5XX" Pattern#1735
baywet merged 2 commits into
microsoftgraph:mainfrom
EnsnerT:patch-1

Conversation

@EnsnerT

@EnsnerT EnsnerT commented Sep 9, 2024

Copy link
Copy Markdown
Contributor

fix: Failed Responses now work with errorMapping with Key "4XX" and "5XX".

Issues

  • Code from the msgraph SDK for Java use for the errorMap Keys a Default of "4XX" and "5XX" HashMap Key. Since this is not fully supported by core, (but almost implemented) i tried my best at it to implement this 'feature'.

Changes proposed in this pull request

  • Support for Map Keys "4XX" and "5XX" as fallback Patterns for Statuscodes in the 400s and 500s, if the Specific Code is not a Key in the Error Map.

PS: Im new here in such Big Projects, so i dont know if im at the right address for this issue. I'm open for feedback tho.

fix: Failed Responses now work with errorMapping with Key "4XX" and "5XX".

It looks like, there should be code here to satisfy the "4XX" and "5XX" keys to match 400-499 and 500-599 StatusCodes. But it does not work yet, so i implemented it quickly. 

Im new here in such big Projects, so i dont know if im at the right address for this issue. I'm open for feedback tho.
@EnsnerT EnsnerT requested a review from a team as a code owner September 9, 2024 15:33

@baywet baywet left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the contribution!

baywet
baywet previously approved these changes Sep 9, 2024
@baywet baywet enabled auto-merge (squash) September 9, 2024 16:17
@baywet baywet merged commit 150ff3d into microsoftgraph:main Sep 9, 2024
@EnsnerT EnsnerT deleted the patch-1 branch September 9, 2024 16:18
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.

2 participants