From inception of Discourse social logins on desktop would always use a “popup” window for logging in.
The intent was to be “less” distracting keeping you on the actual site while the “logging in” act was happening.
I think it makes no sense to carry this for a few reasons.
It is already inconsistent with the way “discourse SSO” works, it can be flaky on browsers that don’t allow popups (my Firefox complains and requires a click to enable it), the code for posting messages is complex and hard to maintain plus we already are requiring an extra click on the popup now.
How would you feel about throwing away the “pop up a window for social login” code and only having 1 consistent way of logging on be it SSO/social login or what not?
This is changing some fairly critical parts of our authentication code, so the branch is now deployed to Meta for testing.
If anyone has some time to spare, please try logging in using a few different providers, and disconnecting/reconnecting accounts in your preferences. If you find any issues, let me know and I’ll investigate
Also a great chance to try out our new Discord Authentication, which is configured here on Meta
Tested logging in with Facebook, Google, Twitter, and Discord, all successful. Also tested disconnected and re-connecting the same accounts from user preferences. Both processes smooth.
I cannot test GitHub as I have 2FA on the Discourse account linked to my GitHub.
Did some further testing (above tests were all in Chrome on Windows). I confirmed login works with IE, Edge, and Firefox on Windows as well. Discovered 2 minor bugs, one related to logout, and one with account connection in IE, but both have been confirmed to exist prior to this change.
