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

New login #7

Closed
wants to merge 6 commits into from
Closed

Conversation

FelixSFD
Copy link

Updated the library to support login without OpenID (#6). It's basically an implementation of the idea Sam posted in our channel.

API changes

None of the public APIs was changed. Bot owners won't need to update their code. They'll just need to import the new version of the library as soon as it's released.

Internal changes

Deprecated methods

  • StackExchangeClient.SEOpenIdLogin()
  • StackExchangeClient.siteLogin()

New properties

  • StackExchangeClient.email and StackExchangeClient.password
    • Store the user's mail-address and password
    • We need to store this when initializing StackExchangeClient, because we have to use the credentials on every site where we want to join a room
  • StackExchangeClient.loggedInHosts
    • Stores the hosts (the main-sites, not chat.*), where the user is logged in

Compatibility with hosts

At the moment, only sites that are already using the new login-form are compatible. (for example Stack Overflow, AskUbuntu and meta.SE)

new login

stackexchange.com seems not to work yet, since it's still using OpenID.

@Tunaki
Copy link
Owner

Tunaki commented Mar 31, 2018

Hey @FelixSFD. I don't see why loggedInHosts needs to be added. rooms already contains the logged in hosts. Can you check the "new-login-method" branch?
Also, is the automatic account creation lost due to the OpenID drop?

@Tunaki Tunaki self-assigned this Mar 31, 2018
@FelixSFD
Copy link
Author

FelixSFD commented Jul 8, 2018

I've fixed the account creation and logging in to chat.SE in my fork. If you have time, you can copy the code from there. Otherwise we have to release our fork on Maven as org.sobotics.chatexchange in order for our bots to work

@FelixSFD FelixSFD closed this Jul 8, 2018
@FelixSFD FelixSFD deleted the feature/6-replace-openid branch July 8, 2018 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants