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

Plugin should check the WP Database when user isnt found in Auth0 Database #107

Closed
lots0logs opened this issue Dec 12, 2015 · 3 comments
Closed

Comments

@lots0logs
Copy link

I have it configured to do SSO with our forum. I'm seeing an issue when trying to log in with a user that is in the WP database but not in the forum database. Here is the log entry:

Occurred: a few seconds ago
Type: Failed Login (wrong password)
Description: id is required
Connection: Username-Password-Authentication
Application: removed (removed)
User: removed

{
  "date": "2015-12-12T10:27:11.617Z",
  "type": "fp",
  "description": "id is required",
  "connection": "Username-Password-Authentication",
  "client_id": "removed",
  "client_name": "removed",
  "ip": "removed",
  "user_agent": "unknown",
  "user_id": "auth0|unknown",
  "user_name": "removed",
  "strategy": "auth0",
  "strategy_type": "database"
}
{
  "error": {
    "message": "id is required"
  }
}
@glena
Copy link
Contributor

glena commented Dec 15, 2015

Hi, where do you see this error?

By default, the plugin creates the user if it does not exists in the WP database.

@lots0logs
Copy link
Author

The user does exist in the WP Database though. After I created this issue I went back and re reviewed the readme for this plugin and I realized that the fact that the plugin does not check the WP User Database is intentional. Though I think you guys should reconsider it and here's why:

When you have an existing database of users prior to implementing Auth0, you'll probably want to migrate those users. With custom Database connections this is simple enough from the Auth0 Dashboard. However, since you have an actual plugin that integrates WordPress,why should the user need to create yet another custom database connection in order to migrate the users? IMO, the plugin should check the WP Database automatically without additional setup.

Though I realize that in order to keep things consistent on the Auth0 side the WordPress plugin would probably need to create a new custom Database connection for itself anyway. Obviously, from the user POV its better for the plugin to handle all of this. However, at the very least, the documentation needs to be improved to explain what needs to be done to make it work. The current documentation makes it sound like it will take care of everything when in fact it does not.

@lots0logs lots0logs changed the title Plugin does not appear to be checking the WP User Database Plugin should check the WP Database when user isnt found in Auth0 Database Dec 15, 2015
@glena
Copy link
Contributor

glena commented Dec 16, 2015

Right now, you can use WP RPC api and set up a custom db connection with import mode enabled for that.

The next version (WIP, no ETA) of the plugin handles this out of the box exposing two webservices.

@glena glena closed this as completed Dec 16, 2015
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants