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

Various inconsistencies related to passwords #6219

Closed
Butterflysaway opened this issue Aug 4, 2020 · 4 comments · Fixed by #6220
Closed

Various inconsistencies related to passwords #6219

Butterflysaway opened this issue Aug 4, 2020 · 4 comments · Fixed by #6220
Milestone

Comments

@Butterflysaway
Copy link
Contributor

Butterflysaway commented Aug 4, 2020

Description

A few inconsistencies that relate to just password resetting, validation handling & hashing that I've found while looking through the files.

Bugfix 1: Fixes a problem which prevented the user from changing e-mail address on the activate action if they entered a non-valid e-mail on sign up and your board requires activation.
Bugfix 2: Added proper titles to some of the message pages the user sees when requesting a new verification code.
Bugfix 3: No longer gives 404 html header when user requests a new verification code.
Bugfix 4: Added un_htmlspecialchars to missing password post from verifyPassword calls where needed.
Bugfix 5: Removed strip_slashes on password which will prevent any user from having a backslash in their password from saving their profile. Also removed the extra un_htmlspecialchars which was already handled when setting $password.
Bugfix 6: Removed unneeded un_htmlspecialchars from upgrade hash password which will prevent an error if your password contains html special chars by random chance.

Environment (complete as necessary)

  • Version/Git revision: 2.1 Latest
  • Database Type: Mysql
  • Database Version: Mariadb
  • PHP Version: 7.4 Latest

Additional information/references

@jdarwood007
Copy link
Member

@Butterflysaway Thanks for all the recent help. If you are going to submit a PR, you don't need to open a separate issue on it. The PR can describe the issue and contain the fix all in one.

@live627
Copy link
Contributor

live627 commented Aug 6, 2020 via email

@Butterflysaway
Copy link
Contributor Author

@jdarwood007 Np. Sounds good. Just trying to fix all the problems I find before using this on my live site. Hopefully it helps others that are using SMF as well. I've used it on my site for 10+ years.

Can someone answer a question about modifications I have

I've been wanting to update the modifications I used on SMF 1.1.21 which are no longer supported for SMF 2.1. Is it possible to just send an email to admins or w/e with the modification fixed and you can update it on the SMF website or how does that work?

Mods in particular:
Member Color Link - https://custom.simplemachines.org/mods/index.php?mod=111
Simple Portal - https://custom.simplemachines.org/mods/index.php?mod=1104

What I would like to do is just modify the existing packages and make them compatible. I don't want any credits/etc. Just would like it to be available to everyone once I do so since they are great mods that should be continued on 2.1.

@frandominguezl
Copy link
Member

You can get in touch with the Customization Team over https://simplemachines.org (contact SychO or Gary), and they'll guide you on how you can do that.

@MissAllSunday MissAllSunday added this to the Final milestone Aug 22, 2020
@live627 live627 removed the Has fix label Nov 24, 2020
live627 pushed a commit to Butterflysaway/SMF2.1 that referenced this issue Nov 24, 2020
live627 added a commit that referenced this issue Nov 24, 2020
…blems

Fixes various password/activate related issues. Fixes #6219
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants