replace configuration page with starting the debugger guide - #11
Merged
Conversation
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Removes the Configuration page and adds Starting the Debugger as the first page of the User Guide.
It opens by recommending the defaults, then covers the cases where you want something else:
php_debugger.mode, thestart_with_requestvalues, on-demand debugging, and triggers. It closes with the two habits that keep overhead low — remove breakpoints when done, disconnect when you stop.Verified against the running extension
Every behaviour described was checked on a freshly pulled 0.3.0 image rather than read from source alone:
modeacceptsdebugandoff; a removed mode logs an error and falls back, and PHP still runs.start_with_requestacceptsyes(default),no,trigger.noand on-demand off, connecting mid-request returns false. Enablingon_demand_debugging_enabledmakes it work, and logs that OPcache is bypassed for every request.XDEBUG_TRIGGER→PHP_DEBUGGER_TRIGGER→XDEBUG_SESSION→PHP_DEBUGGER_SESSION, each looked up in the environment, then$_GET,$_POST,$_COOKIE. With notrigger_valueany value activates; with one set, a mismatch is refused and logged. Comma-separated multi-secret confirmed in source.The page it replaces was wrong
The old Configuration page claimed
start_with_requestdefaults totrigger(it isyes) andclient_hostto127.0.0.1(it islocalhost).Known gap
The page names
php_debugger_connect_to_client()andphp_debugger_break(). Neither exists in 0.3.0 — onlyphp_debugger_infois defined as an alias, confirmed at runtime and inphp_xdebug.stub.php. Documented on the understanding that the aliases are coming; until then those two calls would fatal.The three pages that linked to Configuration now point elsewhere. The Reference section still has Configuration File and Environment Variables placeholders.