Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feature: user edit form #163

Merged
merged 4 commits into from
Feb 16, 2023
Merged

feature: user edit form #163

merged 4 commits into from
Feb 16, 2023

Conversation

ByteCarlos
Copy link
Contributor

name, username, password, title and school editing functionality

@ByteCarlos ByteCarlos temporarily deployed to QA February 15, 2023 19:02 — with GitHub Actions Inactive
@ByteCarlos ByteCarlos temporarily deployed to QA February 15, 2023 19:02 — with GitHub Actions Inactive
if (isset($_POST['Users'], $_POST['Confirm'])) {
$model->attributes = $_POST['Users'];
if ($model->validate()) {
$password = md5($_POST['Users']['password']);

Check warning

Code scanning / SnykCode

Use of Password Hash With Insufficient Computational Effort

MD5 hash (used in md5) is insecure. Consider changing it to a secure hashing algorithm.
$model->attributes = $_POST['Users'];
if ($model->validate()) {
$password = md5($_POST['Users']['password']);
$confirm = md5($_POST['Confirm']);

Check warning

Code scanning / SnykCode

Use of Password Hash With Insufficient Computational Effort

MD5 hash (used in md5) is insecure. Consider changing it to a secure hashing algorithm.
@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot E 2 Security Hotspots
Code Smell A 103 Code Smells

0.0% 0.0% Coverage
13.8% 13.8% Duplication

@igorgoncalves igorgoncalves merged commit 6606b8c into master Feb 16, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants