Skip to content

[PM-2915] Fix config service null error bug#2778

Merged
andrebispo5 merged 2 commits intomasterfrom
auth/pm-2915/configservice
Oct 2, 2023
Merged

[PM-2915] Fix config service null error bug#2778
andrebispo5 merged 2 commits intomasterfrom
auth/pm-2915/configservice

Conversation

@andrebispo5
Copy link
Contributor

Type of change

  • Bug fix
  • New feature development
  • Tech debt (refactoring, code cleanup, dependency upgrades, etc)
  • Build/deploy pipeline (DevOps)
  • Other

Objective

Fix ConfigService exception bug when StatusCode is null.

Code changes

Add null check

Before you submit

  • Please check for formatting errors (dotnet format --verify-no-changes) (required)
  • Please add unit tests where it makes sense to do so (encouraged but not required)
  • If this change requires a documentation update - notify the documentation team
  • If this change has particular deployment requirements - notify the DevOps team

@andrebispo5 andrebispo5 requested a review from a team September 18, 2023 14:22
@bitwarden-bot
Copy link

bitwarden-bot commented Sep 18, 2023

Logo
Checkmarx One – Scan Summary & Details50784961-fc6b-4a2a-9cef-b7a442b6001d

No New Or Fixed Issues Found

@andrebispo5 andrebispo5 merged commit 685e0f4 into master Oct 2, 2023
@andrebispo5 andrebispo5 deleted the auth/pm-2915/configservice branch October 2, 2023 19:43
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