Skip to content

Conversation

@AlexzPurewoko
Copy link
Member

@AlexzPurewoko AlexzPurewoko commented Apr 7, 2025

Pull Request ini ditujukan untuk memperbaiki konfigurasi dari XDebug pada Dockerfile yang sebelumnya ada trouble di connection, menjadi bisa

@abdulhp abdulhp self-requested a review April 7, 2025 07:59
Dockerfile Outdated
rm -rf /tmp/php-spx

# Install and configure Xdebug
RUN pecl install xdebug && docker-php-ext-enable xdebug && \
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bang @AlexzPurewoko, mintol command enable ini dihapus soale redundan sama yang di deck.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Jadi tinggal pecl install xdebug gitu?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yoii, betul

Copy link
Member Author

@AlexzPurewoko AlexzPurewoko Apr 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nah aku butuhnya buat create file ini :
/usr/local/etc/php/conf.d/docker-php-ext-xdebug.ini

Kalau dilangsung seperti ini aman ngga ya @abdulhp ?

# Install and configure Xdebug
RUN pecl install xdebug && \
    touch /usr/local/etc/php/conf.d/docker-php-ext-xdebug.ini && \
    echo "zend_extension=xdebug" >> /usr/local/etc/php/conf.d/docker-php-ext-xdebug.ini && \
    echo "xdebug.mode=debug" >> /usr/local/etc/php/conf.d/docker-php-ext-xdebug.ini && \
    echo "xdebug.client_host=172.20.0.1" >> /usr/local/etc/php/conf.d/docker-php-ext-xdebug.ini && \
    echo "xdebug.idekey=dicoding-debug" >> /usr/local/etc/php/conf.d/docker-php-ext-xdebug.ini && \
    rm -rf /tmp/pear

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done bang

@abdulhp
Copy link
Collaborator

abdulhp commented Apr 7, 2025

LGTM ❤️

@abdulhp abdulhp merged commit 44a03ff into dicoding-dev:main Apr 7, 2025
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.

2 participants