Skip to content
This repository was archived by the owner on May 15, 2024. It is now read-only.

Fix crash when using WebAuthenticator.AuthenticateAsync #1586#1687

Merged
mattleibow merged 1 commit intoxamarin:mainfrom
jgoyvaerts:bugfix-WebAuthenticatorIntermediateActivity
Feb 10, 2021
Merged

Fix crash when using WebAuthenticator.AuthenticateAsync #1586#1687
mattleibow merged 1 commit intoxamarin:mainfrom
jgoyvaerts:bugfix-WebAuthenticatorIntermediateActivity

Conversation

@jgoyvaerts
Copy link
Copy Markdown
Contributor

@jgoyvaerts jgoyvaerts commented Feb 10, 2021

Description of Change

Fix crash when using WebAuthenticator.AuthenticateAsync in latest version. While this might not be the best way to fix it, it fixes the issue (at least on our app). If a different fix is required, can this at least be given some priority? Thanks

No tests are added because I have no idea how I'd even write a test for this case

Bugs Fixed

API Changes

None

Behavioral Changes

None

@jamesmontemagno
Copy link
Copy Markdown
Collaborator

LGTM

@mattleibow mattleibow added this to the 1.6.2 milestone Feb 10, 2021
@mattleibow mattleibow merged commit 1bd8b65 into xamarin:main Feb 10, 2021
dimonovdd added a commit to dimonovdd/Essentials that referenced this pull request Feb 13, 2021
* main:
  Fix crash when using WebAuthenticator.AuthenticateAsync xamarin#1586 (xamarin#1687)
dimonovdd added a commit to dimonovdd/Essentials that referenced this pull request Feb 13, 2021
* main:
  Fix crash when using WebAuthenticator.AuthenticateAsync xamarin#1586 (xamarin#1687)
  Ensure granted (xamarin#1658)
@purpurproger
Copy link
Copy Markdown

purpurproger commented Mar 31, 2021

@mattleibow, could you tell when will this changes be released?

@SebastienForay
Copy link
Copy Markdown

Hi @mattleibow @jamesmontemagno !
Can we have any update on when 1.6.2 (or any version) will be released with this fix ? Thanks !

@jamesmontemagno jamesmontemagno modified the milestones: 1.6.2, 1.7.0 Jul 12, 2021
@SebastienForay
Copy link
Copy Markdown

Woohoo !
1.7.0 has been released with the fix 🎉

Thanks to all the team @jamesmontemagno 💪

CartBlanche pushed a commit to CartBlanche/Essentials that referenced this pull request Sep 21, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] WebAuthenticator - Android - Null Reference Exception WebAuthenticatorIntermediateActivity.android.cs

5 participants