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

Allow Google Signin #508

Closed

Conversation

WhiteHusky
Copy link

This doesn't truely fix the problem. A weird combination of refreshing several times and...generally spastastic navigation behavior seems to get around the signin. This is mostly here to show what I tried.

This doesn't truely fix the problem. A weird combination of refreshing several times and...generally spastastic navigation behavior seems to get around the signin.
@WhiteHusky
Copy link
Author

Related to #507

@WhiteHusky
Copy link
Author

Gets weirder:

So after doing the weird navigation dance it kinda...doesn't sign in correctly. It appears if you are, but you get a signed out front page. Removing the patch, signing in, then signing back in by clicking your signed out account it works.

Obviously this is broken.

@WhiteHusky
Copy link
Author

More testing and trying to find a concise path to it working finds it's just a coinflip. Trying going back, forward, and several other navigations seems to be random when it'll just work or not. So I'm stumped.

I also tried just updating the user agent but no dice.

@Skaifer
Copy link

Skaifer commented Dec 3, 2021

Hello!
So I'm facing the same issue on Fedora using appimage. Just in case this info would help, I could not sign in to google account on Vivaldi until I turned on google's crypto token extension in privacy settings.

Cheers!

P.S.
Navigating back two times and forwards two times got me further trough login process. Good workaround, thanks )

@Araxeus
Copy link
Collaborator

Araxeus commented Dec 13, 2021

Hey could you try my PR? #515

I think I might have fixed the problem 🙂

@th-ch th-ch added the fix-available A fix to the issue is available in a new version label Dec 30, 2021
@Araxeus
Copy link
Collaborator

Araxeus commented Dec 30, 2021

@WhiteHusky #515 was merged and should have fixed the problem

So this PR could be closed

@th-ch
Copy link
Owner

th-ch commented Apr 8, 2022

Closing as it should be fixed and the PR is stale - feel free to re-open a PR if needed!

@th-ch th-ch closed this Apr 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix-available A fix to the issue is available in a new version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants