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 for Xcode 12.5 [SDK-2545] #369

Merged
merged 1 commit into from
Apr 29, 2021
Merged

Fix for Xcode 12.5 [SDK-2545] #369

merged 1 commit into from
Apr 29, 2021

Conversation

Widcket
Copy link
Contributor

@Widcket Widcket commented Apr 29, 2021

Changes

Since version 12.5 (released on 26 Apr 2021), Xcode no longer accepts an entire URL for the ASWebAuthenticationSession scheme value. This PR extracts the scheme from the URL and uses just that, instead of the whole /authorize URL.

References

Fixes #360

Testing

  • This change adds unit test coverage
  • This change has been tested on the latest version of the platform/language or why not

Checklist

@norbertschuler
Copy link

norbertschuler commented Apr 29, 2021

I just verified that using this pull request solved my crash mention in #371. Would be great if a new release with the fix could be done to help people with iOS 14.5.

@idrisssakhi
Copy link

It would be great to have a new release. Because old version doesn’t work on IOS 14.5.

@Widcket Widcket merged commit bccee2e into master Apr 29, 2021
@Widcket Widcket deleted the fix/xcode-12.5 branch April 29, 2021 12:40
@adammcarth
Copy link

Thanks for getting this one in so quickly @Widcket. When can we expect this to be available as a release?

@Widcket Widcket added this to the v2-Next milestone Apr 29, 2021
@Widcket Widcket mentioned this pull request Apr 29, 2021
@Widcket
Copy link
Contributor Author

Widcket commented Apr 29, 2021

@adammcarth I expect the release PR to be merged tomorrow, and then I'll push to npm.

@steven-pearson
Copy link

Will the publish also include an update to Nuget for Auth0.OidcClient.iOS?

@Widcket
Copy link
Contributor Author

Widcket commented Apr 30, 2021

The release has been pushed to npm.

@steven-pearson this is only for react-native-auth0.

@smenke
Copy link

smenke commented Apr 30, 2021

Thanks so much! This fixed the issue.

@Widcket
Copy link
Contributor Author

Widcket commented Apr 30, 2021

@steven-pearson the update for Auth0.OidcClient.iOS is being tracked in auth0/auth0-oidc-client-net#189.

@prangel-credera
Copy link

Hi there,

I am encountering this issue but only seem to be able to replicate it on Emulator devices, not on deployed devices. Is that expected?

Thanks,
-Peter

@Widcket
Copy link
Contributor Author

Widcket commented May 4, 2021

Hi @prangel-credera,
I'm able to replicate it on a physical device running iOS 14.5.

IMG_9DC0669D5F7F-1

@sinclas
Copy link

sinclas commented May 14, 2021

Same error here. What is the easiest way to fix this on a developer machine? What are some workarounds? Thanks!!

@marf
Copy link

marf commented May 14, 2021

Same error here.

@sinclas
Copy link

sinclas commented May 14, 2021

HI,

This just for development, but got everything working and productive again:

I opened Xcode, located the A0Auth0.m file. Should be located in development pods.
locate the method - (void)presentAuthenticationSession:(NSURL *)URL
change: NSString *callbackURLScheme = queryItem.value;
to: NSString *callbackURLScheme = [[NSURL URLWithString: queryItem.value] scheme];

I had updated Xcode and iOS and had to make this change to keep productive. Will wait for an NPM update prior to doing anything in test or release. ;)

S

@relm923
Copy link

relm923 commented May 14, 2021

The fix was released in 2.8.2. Are you still seeing issues with the latest version? Its working for our team with 14.5

@sinclas
Copy link

sinclas commented May 14, 2021 via email

@fzaffarana
Copy link

I have the same problem using version 1.6.0 (react-native-auth0).

My problem is that I can't update it because the project is using version 0.59.9 of rn. (version 1.6.0 is a fixed version for 0.59.9 based on the documentation)

is there any fix for this version (1.6.0)?

jinpark added a commit to omt-tech/react-native-auth0 that referenced this pull request Jul 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Auth0 authorize stuck only on the very first try