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

Passing more parameters in the reset request. #138

Open
Karlinator opened this issue Jul 8, 2021 · 1 comment
Open

Passing more parameters in the reset request. #138

Karlinator opened this issue Jul 8, 2021 · 1 comment

Comments

@Karlinator
Copy link

Is it possible to pass more parameters in the reset password request? For instance to support different front ends that use the same API, a parameter could be used to switch url bases in the link sent out to the user.

I suppose I could extend the model class and add this myself; is that the easiest way? Or would a PR with this functionality be welcome?

@Karlinator
Copy link
Author

Just to clarify: I don't intend to pass along the url provided in the request, as that would be an obvious security hole (allowing anyone to send users emails linking to nefariousdomain.com/reset_password?t=real_token). Rather, the parameter would act as an enum.

E.g with the parameter url: 'default' I would send the email with a link to 'mainapp.company.com/reset_password' and with url: 'otherapp' the email links to 'different-app.samecompany.com/password_reset'

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

1 participant