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

Rule Errors do not propagate #28

Closed
darkyen opened this issue Jul 16, 2016 · 3 comments
Closed

Rule Errors do not propagate #28

darkyen opened this issue Jul 16, 2016 · 3 comments
Milestone

Comments

@darkyen
Copy link

darkyen commented Jul 16, 2016

The following will not respect pass through any errors that come from Rules as error_description

Strategy.prototype.authenticate = function (req, options) {
  if (req.query && req.query.error) {
    return this.fail(req.query.error);
  }
  this._base.authenticate.call(this, req, options);
};
@techpool
Copy link

Yeah, actually what's happening is the fail function automatically redirects you to the failureRedirect path in the passport setting. I think this is a special case of auth0 and thus needs to be handled differently rather than calling the fail function.

@dshorowitz
Copy link

I have the same issue. Errors from rules get sent to the failureRedirect path. I want to show those errors in the lock widget instead (where the lock widget UI would update with the error text, similar to 'Wrong email or password'

image

How can you do this using auth0 passport?

@joshcanhelp
Copy link
Contributor

We recommend using the Universal Login Page for authentication. See our docs for a comparison. Currently, Rules do not bubble the error text up to Lock but that may be added in the future.

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

No branches or pull requests

4 participants