Navigation Menu

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

Jump to bottom

trunk: update demo to 1.7.2-rc1 fails with missing superuser->salt config entry #3040

Closed
mattab opened this issue Mar 14, 2012 · 3 comments
Closed

trunk: update demo to 1.7.2-rc1 fails with missing superuser->salt config entry #3040

mattab opened this issue Mar 14, 2012 · 3 comments
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone
1.12.x

Comments

@mattab
Copy link
Member

mattab commented Mar 14, 2012

Error message: Notice salt in Config.php

I added a salt = 32 chars hex
and it asked to relogin (normal since cookie hash changed)
and then works. But I dont think it should fail at all, mayb a regression? Did I really not have a salt in the config file on the demo? (I don't know for sure if it was removed or if it wasn't there before the update...)
@robocoder
Copy link
Contributor

it probably wasn't there

we can add a placeholder in global.ini.php to avoid the notice

@robocoder
Copy link
Contributor

(In [6055]) fixes #3040 - salt was added in 0.5.4; Zend_Config masked its absence

@mattab
Copy link
Member Author

mattab commented Mar 19, 2012

I removed salt from config.ini.php, upgraded to 1.7.2-rc2 and the error message did not show up. Thanks! :)

@mattab mattab added this to the 1.12.x - Piwik 1.12.x milestone Jul 8, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Milestone
1.12.x
Development

No branches or pull requests
2 participants
@mattab @robocoder