Ticket #481 (closed Bug: fixed)

Opened 20 months ago

Last modified 13 months ago

username should be enclosed in double quotes in config files, in case the user name contains a =

Reported by: matt Owned by:
Priority: major Milestone: RobotRock
Component: Core Keywords:
Cc: Sensitive:

Description

From bug report:  http://forum.piwik.org/index.php?showtopic=70

In the config file we should have

username = "my=username"

instead of

username = my=username

the same way the password is currently enclosed in double quotes

Change History

Changed 18 months ago by vipsoft

from #570, host should also be enclosed in quotes.

Changed 18 months ago by matt

  • status changed from new to closed
  • resolution set to fixed

fixed in [949]

--> all non-array values in Config/config.ini.php are now automatically enclosed in double quotes, and double quote automatically encoded as " (parse_ini_file doesn't handle escaping the double quote, arg)

cleaning the Piwik_Config code, adding tests, the code using the Config is now more simple, all the logic is inside the class (before it was very hacky). the installer is now using the Config to create the config file rather than duplicating the logic.

Changed 13 months ago by domtop

Note: See TracTickets for help on using tickets.