sahat / satellizer

Token-based AngularJS Authentication
https://satellizer-sahat.rhcloud.com
MIT License
7.85k stars 1.13k forks source link

Impossible to login on iOS Safari/Chrome with 0.15.5, keeps redirecting to domain.com/login. #956

Open aligajani opened 8 years ago

aligajani commented 8 years ago

This has become worse as I upgraded from 0.13.5 to 0.15.5.

Error found using Web Inspector:

Error: OAuth redirect has occurred but no query or hash parameters were found. They were either not set during the redirect, or were removed—typically by a routing library—before Satellizer could read it.

Update

Setting the timer to 200[1] milliseconds from 500 millionseconds makes it work on iOS Safari 100%. But not on Google Chrome , although success rate increases.

[1] Line 587 of satellizer.js.

Okay, guys, now even I am confused. What exactly is happening here and what is the solution @sahat .

Something somewhere is polling every 500 milli, causing the problem, but setting it to 200 milli makes a whole lot of a difference. I am curious. I think we might be close to fixing this...Let's investigate....

omayhemo commented 8 years ago

Changing the polling interval did not help me at all on either Safari or Chrome on iOS

omayhemo commented 8 years ago

I got a hold of an android and a kindle. Both experience similar problems as iOS although the error message is different. (unfortunately haven't been able to capture the error message on android and kindle yet). So this is apparently ubiquitous to mobile.

aligajani commented 8 years ago

Yes it is only on mobile. Setting interval to 0 has helped me to a degree.

On Wednesday, September 28, 2016, Doug Beard notifications@github.com wrote:

I got a hold of an android and a kindle. Both experience similar problems as iOS although the error message is different. (unfortunately haven't been able to capture the error message on android and kindle yet). So this is apparently ubiquitous to mobile.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/sahat/satellizer/issues/956#issuecomment-250205672, or mute the thread https://github.com/notifications/unsubscribe-auth/ABpo2gHg_-cDeD42w3w7zZTkqyhdBy7oks5quoovgaJpZM4J54n4 .

Ali Gajani Founder at Mr. Geek www.mrgeek.me www.aligajani.com

omayhemo commented 8 years ago

Unfortunately, changing the interval (to any number of values, including 0) hasn't helped at all on either iOS or Android.

viktorhunko commented 8 years ago

Hey guys, got same error in web app on pc. And interesting fact, that one day before i had no this error with same settings. any ideas how to deal with it ?

aligajani commented 8 years ago

Rare to see on web app. What's the log saying ?

On Saturday, October 8, 2016, Viktor Hunko notifications@github.com wrote:

Hey guys, got same error in web app on pc. any ideas how to deal with it?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/sahat/satellizer/issues/956#issuecomment-252421126, or mute the thread https://github.com/notifications/unsubscribe-auth/ABpo2p2oPISF8Sload6HEKiwi8C-n8dfks5qx4d8gaJpZM4J54n4 .

Ali Gajani Founder at Mr. Geek www.mrgeek.me www.aligajani.com

srinivasaltair commented 7 years ago

Hi guys, i am facing the same issue. Loging/signup works for first three attempts and fails there after and vice versa. I tried reducing the intervals as mentioned by @aligajani but it dint fix my issue and is still the same. Have anyone fixed this?