Page MenuHomePhabricator

Quibble should not rely on dynamically detecting the value of $wgServer
Closed, ResolvedPublic


To unblock T232931: Remove support for run-time automatic $wgServer default value, quibble needs to explicitly supply the value for $wgServer to the installer, using the new --server option. From the patch

Looks like the quibble jobs are still depending on this. Their generated settings are now getting "$wgServer = "http://localhost";" which is wrong (should be something like "", which I guess it needs to pass it from its ENV var).

Event Timeline

From the CLI maintenance/shell.php the port is obviously not detected:

>>> $wgServer;
=> "http://localhost"

Via the webserver: http://localhost:9412 (correct).

maintenance/install.php supports passing the server via --server and that seems to work. At least in the master branch.

Change 538583 had a related patch set uploaded (by Hashar; owner: Hashar):
[integration/quibble@master] Set $wgServer when installing

Change 538583 merged by jenkins-bot:
[integration/quibble@master] Set $wgServer when installing

hashar edited projects, added Quibble; removed Patch-For-Review.
hashar moved this task from Backlog to In progress on the Quibble board.

Setting $wgServer leads to a Selenium test to fail :-\ T235023

Change 541811 had a related patch set uploaded (by Hashar; owner: Hashar):
[integration/config@master] docker: bump quibble to 0.0.38

Change 541811 merged by jenkins-bot:
[integration/config@master] docker: bump quibble to 0.0.38

Further fixed up in 0.0.38 due to T235023