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.

Sign up for GitHub

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

Jump to bottom

Server Side Password Check #601

Merged
merged 3 commits into from
Sep 23, 2019
Merged

Server Side Password Check #601

merged 3 commits into from
Sep 23, 2019

Conversation

Hackmastr
Copy link
Contributor

@Hackmastr Hackmastr commented Sep 14, 2019 •
edited by rumblefrog
Loading

Description

Check if the current password is correct server side.

Motivation and Context

Closes #595

How Has This Been Tested?

Tested in PHP 7.3.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • [] My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.

@rumblefrog rumblefrog requested a review from Groruk September 14, 2019 20:17
@Groruk Groruk merged commit 79f3a5b into sbpp:v1.x Sep 23, 2019
@rumblefrog rumblefrog changed the title Fix #595 Server Side Password Check Sep 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Groruk Groruk Groruk approved these changes

Assignees
No one assigned
Labels
semver: patch t: permission system t: ui/ux type: enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Password change in "your account" page
3 participants
@Hackmastr @Groruk @rumblefrog