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

revert to reading config for superuser token_auth #252

Closed
wants to merge 1 commit into from

Conversation

tomval2k
Copy link

import_logs.py --help ... --config ... Piwik will read the configuration file...to fetch the Super User token_auth

A prior change had import_logs.py launching a php command on local file system to query piwik database, however piwik may not be installed on local system. And for me it didn't work. And why run a python script, to launch a php script, to query a database, to then run the http post to the (possibly remove) piwik http server.

import_logs.py --help ... --config ... Piwik will read the configuration file...to fetch the Super User token_auth

A prior change had import_logs.py launching a php command on local file system to query piwik database, however piwik may not be installed on local system. And for me it didn't work. And why run a python script, to launch a php script, to query a database, to then run the http post to the (possibly remove) piwik http server.
@mattab
Copy link
Member

mattab commented Mar 29, 2014

the token_auth is not in the config file as of 2.1.0 (but stored in the DB)
if piwik is not installed on the system please use --token-auth to specify the token.

this solution is the best we could think of, even though it can seem a bit convoluted.
the error reporting can i'm sure be improved to point users in the right direction...

@mattab mattab closed this Mar 29, 2014
@tomval2k
Copy link
Author

given that the config file contains the username and password for the database, why is a token_auth not allowed there? Or is this an effort to keep server config seperate to client/end-user config?

@mattab
Copy link
Member

mattab commented Mar 30, 2014

Yes an effort to keep server config in the DB as much as possible

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

Successfully merging this pull request may close these issues.

None yet

2 participants