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

Admin-Configurable Salt #3

Open
jtickle opened this issue Jan 17, 2014 · 0 comments
Open

Admin-Configurable Salt #3

jtickle opened this issue Jan 17, 2014 · 0 comments
Milestone

Comments

@jtickle
Copy link

jtickle commented Jan 17, 2014

Banner IDs in SLC are hashed before insertion into the database in order to prevent anyone from being able to track an issue back to a particular person. There is a little bit of security through obscurity going on here and it is not a perfect solution, but in this case we believe it is sufficient.

When this was a private project, this hash was simply defined at the top of slc/inc/defines.php, like so:

define('ENCODE_SALT', 'super duper secret');

Although having the salt doesn't make it a lot easier to reverse these hashes, we prefer to keep it secret and changeable.

Please use PHPWS_Settings to create a setting for this, call it "encode_salt", and make sure to use it in the encode() function in inc/functions.php. Also, there will need to be an addition to the settings interface for SLC where this can be set, only by a deity.

┆Issue is synchronized with this Asana task

@jlbooker jlbooker modified the milestone: Fall 2016 Jul 15, 2016
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

2 participants