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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix security issue with reset password #4443

Merged
merged 1 commit into from Nov 4, 2019

Conversation

@alexandrebodin
Copy link
Collaborator

alexandrebodin commented Nov 4, 2019

Description of what you did:

My PR is a:

  • 馃挜 Breaking change
  • 馃悰 Bug fix
  • 馃拝 Enhancement
  • 馃殌 New feature

Main update on the:

  • Admin
  • Documentation
  • Framework
  • Plugin

Manual testing done on the following databases:

  • Not applicable
  • MongoDB
  • MySQL
  • Postgres
  • SQLite
Copy link
Member

lauriejim left a comment

LGTM

@alexandrebodin alexandrebodin merged commit 1162698 into master Nov 4, 2019
1 check was pending
1 check was pending
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
@alexandrebodin alexandrebodin deleted the fix/security branch Nov 4, 2019
@brianadeloye

This comment has been minimized.

Copy link

brianadeloye commented Nov 4, 2019

@alexandrebodin @lauriejim thanks for the quick fix.

do you have a rough estimate of when this will be included in a release? At some point in the future I'm planning on requesting a CVE identifier for this bug.

@derrickmehaffy

This comment has been minimized.

Copy link
Contributor

derrickmehaffy commented Nov 4, 2019

@brianadeloye can you jump on the strapi slack please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can鈥檛 perform that action at this time.