Skip to content

Commit 10ddd92

Browse files
Remove port 80 from tests config for local DDEV environments (#23678)
This breaks system tests where it's added to the hostname when configured even when port 80 is the default, breaking the mechanism in `TestRequest\Response::replacePiwikUrl` when it's searching for matomo.ddev.site:80 and response contains just matomo.ddev.site.
1 parent e8d9f39 commit 10ddd92

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

.ddev/commands/host/matomo_init_tests

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,7 @@ ddev matomo:console config:set \
1010
'database_tests.username="db"' \
1111
'database_tests.password="db"' \
1212
'tests.http_host="matomo.ddev.site"' \
13-
'tests.request_uri="/"' \
14-
'tests.port=80'
13+
'tests.request_uri="/"'
1514

1615
echo "Copy UI tests config file ..."
1716
echo "@see https://developer.matomo.org/guides/tests-ui#configuring-ui-tests"

0 commit comments

Comments
 (0)