Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 2 commits into from
Sep 9, 2024
Merged

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

merged 2 commits into from
Sep 9, 2024

Conversation

EnsnerT
Copy link
Contributor

@EnsnerT EnsnerT commented Sep 9, 2024

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
Copy link
Member

@baywet baywet left a comment

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
10 checks passed
@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