Skip to content

Commit

Permalink
fix(jsonnet): remove client_port=9000 for xdebug3 configuration
Browse files Browse the repository at this point in the history
It now uses default port 9003 for xdebug3 configuration.

IntelliJ and PHPStorm now listens on both ports 9000 and 9003.
  • Loading branch information
Toilal committed Feb 19, 2021
1 parent 689efb0 commit d8ca959
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion ddb/feature/jsonnet/lib/ddb.docker.libjsonnet
Expand Up @@ -346,7 +346,7 @@ local VirtualHost = if (_docker_virtualhost_type == "traefik" && !_docker_virtua
local NoXDebug(version=_docker_xdebug_version, session=_docker_xdebug_session, mode=_docker_xdebug_mode) = {};
local XDebug(version=_docker_xdebug_version, session=_docker_xdebug_session, mode=_docker_xdebug_mode) =
local xdebug2_config = ["remote_enable=" + if mode != "off" then "on" else "off", "remote_autostart=off", "idekey=" + session, "remote_host=" + _docker_xdebug_host];
local xdebug3_config = ["client_host=" + _docker_xdebug_host, "client_port=9000", "log_level=0"];
local xdebug3_config = ["client_host=" + _docker_xdebug_host, "log_level=0"];
{
environment: {
[if version == null || version == 2 then "PHP_IDE_CONFIG"]: "serverName=" + session,
Expand Down
Expand Up @@ -3,7 +3,7 @@ services:
php-default:
environment:
PHP_IDE_CONFIG: serverName=default
XDEBUG_CONFIG: remote_enable=on remote_autostart=off idekey=default remote_host=%docker.debug.host% client_host=%docker.debug.host% client_port=9000 log_level=0
XDEBUG_CONFIG: remote_enable=on remote_autostart=off idekey=default remote_host=%docker.debug.host% client_host=%docker.debug.host% log_level=0
XDEBUG_MODE: debug
XDEBUG_SESSION: default
image: php:7.4-fpm
Expand All @@ -24,7 +24,7 @@ services:
environment:
XDEBUG_MODE: debug
XDEBUG_SESSION: default
XDEBUG_CONFIG: client_host=%docker.debug.host% client_port=9000 log_level=0
XDEBUG_CONFIG: client_host=%docker.debug.host% log_level=0
image: php:7.4-fpm
init: true
restart: "no"
Expand Down

0 comments on commit d8ca959

Please sign in to comment.