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

MSAL:SubmoduleUpdate for Added null check for additional json info in DeviceInfo in Common Core #1664

Merged
merged 3 commits into from
Mar 3, 2023

Conversation

swasti29
Copy link
Contributor

@swasti29 swasti29 commented Mar 3, 2023

Proposed changes

Updated submodule for: Fix a crash when no additional info found in the device info json.
PR: https://github.com/AzureAD/microsoft-authentication-library-common-for-objc/pull/1203/files#diff-b40cd67182487ef24807d9c9268329d35fbd96aa2b0a9cae69e2e0d746b1c666

Type of change

  • Feature work
  • Bug fix
  • Documentation
  • Engineering change
  • Test
  • Logging/Telemetry

Risk

  • High – Errors could cause MAJOR regression of many scenarios. (Example: new large features or high level infrastructure changes)
  • Medium – Errors could cause regression of 1 or more scenarios. (Example: somewhat complex bug fixes, small new features)
  • Small – No issues are expected. (Example: Very small bug fixes, string changes, or configuration settings changes)

Additional information

@swasti29 swasti29 requested a review from a team as a code owner March 3, 2023 20:08
@@ -1 +1 @@
Subproject commit 1b1d3edb835993418329c87784de251eba99a22b

Choose a reason for hiding this comment

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

This pull request does not update CHANGELOG.md.

Please consider if this change would be noticeable to a partner or user and either update CHANGELOG.md or resolve this conversation.

@swasti29 swasti29 merged commit cacbbb8 into dev Mar 3, 2023
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.

None yet

2 participants