-
Notifications
You must be signed in to change notification settings - Fork 9
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
windows asking for an app to open au.gov.my #2
Comments
Hmm, the whole point of having an Electron app is to be able to handle the au.gov.my:// URL scheme. I think they must have changed something in the auth flow, I'll take a look. |
(This is very probably the cause of #1 too) |
No worries. let me know If I can assist |
Have you discovered what the problem is? |
Experiencing the same issue here. |
OK, I have pushed a change that should improve the logging (or even fix the issue, who knows?). If you run it with environment variable The auth flow still works for me, so hopefully it's just some simple (?) thing about Electron's behaviour... |
Nothing happens when pressing submit, but the terminal window displays this stuff. The registration does not complete because the submit button does nothing. au.gov.my://app?state=63065293C60AD6A479B67F1DFC79BBC75DEF04C3&code=C3Mab1fzVcEPsoMrC3WhxHhhwEE2UK |
Thanks @danielgruber8 . I think the issue is that myGov uses the SHA512 TOTP algorithm, but Authy doesn't support it (see eg. speakeasyjs/speakeasy#95 (comment)). I've added better logging so now it actually tells you if myGov thinks your code is wrong. Can you confirm that's what it is? |
When using Authy, a pop up came up saying something about an invalid SHA1 key. But the enrolment now completed successfully when using andOTP..... and I could login to myGov perfectly. You did fix something though as I did try using andOTP previously and had the same issue. |
Is there any way to convert the AndOTP code back in to Authy? Or not possible? Also Electron logging shows a lot of security warnings user@tux:~/Downloads/mygov-totp-enroll-master$ ELECTRON_ENABLE_LOGGING=1 npm start
[7960:0729/112016.334569:INFO:CONSOLE(259)] "%cElectron Deprecation Warning (nodeIntegration default change)", source: /home/user/Downloads/mygov-totp-enroll-master/node_modules/electron/dist/resources/electron.asar/renderer/security-warnings.js (259) |
I'm guessing if Authy haven't fixed it by now they aren't going to bother. It's an unfortunate bit of needless bureaucratic paranoia on the part of the myGov implementors... |
Just tried the latest master in case but still it is asking for an app and not working when i hit submit |
The most recent commit (4cce822) works for me on Windows 10 (1909). Successfully enrolled a TOTP authenticator; login tested after initial verification. Built and run directly from the Windows host. All operations were executed from an unprivileged PowerShell shell. PS C:\path\to\mygov-totp-enroll> git clone
...
PS C:\path\to\mygov-totp-enroll> npm install
...
PS C:\path\to\mygov-totp-enroll> npm start VersionsSummaryCommit: 4cce822 Detailed informationNodeJS PackagesPlease see this gist NodeJS version
Windows OS Version
PowerShell Version
|
This worked for me just now, with similar settings to the above, though I had to set different CSP values: <meta http-equiv="Content-Security-Policy" content="script-src 'unsafe-inline'; form-action https: au.gov.my:;"> on <meta http-equiv="Content-Security-Policy" content="script-src 'self' 'unsafe-inline';"> on |
Okay so this seems to be resolved as mentioned for me too, however didnt realise that Authy is not supported so I cannot use it anyway. not conveniently anyhow. |
Success using bitwarden. |
Awesome! Cheers @SgtBatten |
After scanning the QR code into authy and hitting submit this window appears:
The text was updated successfully, but these errors were encountered: