-
Notifications
You must be signed in to change notification settings - Fork 11
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
Can't Sign In - App classed as 'not secure' by Google #11
Comments
I've not worked with Electron before, but I am excited enough about this (Keep is one of my main go-to apps on Android for so many reasons!) that I hope to tinker and see if I can solve this issue. I imagine it's a common theme for Electron/Cordova/etc so I think it shouldn't be too hard to track down. Cheers on the app altogether though, @tmcinerney! If I can integrate Keep with OS X keyboard shortcuts I'll be over the moon. Yes, I am a 200-year-old hipster who uses those hep cat thoughtwords, you crazy kids, you! (actually, I just never sleep and so am deliriously firing off messages into the void, but it's six of one and half a dozen of the other, right?) |
I believe this issue is still ongoing - I am not able to log in and get the same message as in the screenshot above from @glowmango. |
Hey @carmensandoval I don't actually use Google Keep anymore, but I thought a dependency bump might help. I wasn't able to reproduce the issue y'all had, but would be able to test this beta release before I merge the code and release the updated application? |
FYI ^^ @glowmango |
@tmcinerney tried your beta release as I'm seeing the same issue - no luck unfortunately. Same error as above. |
Is it related too this? |
works now - not sure what changed! |
Doesn't work for me, I still get the above error on a brand new install of 2.0.0 beta |
Same here! |
Why am I in this thread?
…On Tue, Jun 23, 2020 at 6:12 PM Carmen Sandoval ***@***.***> wrote:
Same here!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#11 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AOXHQHA5ZZN66M7RTY47U53RYFHJRANCNFSM4KCGXYVA>
.
--
William M. Legrain
|
I've still not been able to replicate this. The Stack Overflow post linked above indicates an issue with the user agent. Looking at the docs from menubar, it seems I can set the user agent for the load options, as described here. I've updated the dependency bump PR to include the setting of the user agent. I've built an updated version of the app and updated the beta release I linked before: https://github.com/tmcinerney/keep/releases/tag/v2.0.0beta I tested the change using this site which displays the user agent. I can see the following when simply setting it to The current version of Chrome on MacOS looks more like:
When navigating to the user agent test page in Keep I can see mentions of Electron and the like: |
Iunno anything about electron or javascript tho so ymmv. |
Just added the useragent attribute in Line 10: |
Added the useragent attribute in index.html webview tag to fix the issue tmcinerney#11
@tmcinerney And rebuild the app. Everything should work now. I have changed 'frame' option to true for ease in movability of application. |
Incredibly excited by the existence of this app. However, the app will not sign in as Google is classing it as 'insecure'.
Screenshot here:
The text was updated successfully, but these errors were encountered: