Skip to content

Document GHES pre-receive hook networking restrictions in README - #818

Merged
jamesooo merged 2 commits into
masterfrom
copilot/githubenterprise250910
Sep 1, 2026
Merged

Document GHES pre-receive hook networking restrictions in README#818
jamesooo merged 2 commits into
masterfrom
copilot/githubenterprise250910

Conversation

Copilot AI commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

The pre-receive hooks README lists network as a resource hook scripts may use, but doesn't mention that GitHub Enterprise Server now restricts it. GHES 3.18+ logs hook network calls and lets administrators block them; GHES 3.22+ blocks them by default.

Changes

  • pre-receive-hooks/README.md: added a short paragraph in the "Performance, stability and workflow implications" section — immediately after the discussion of resources contended by parallel hook runs — covering:
    • 3.18+: network calls are logged and can be blocked via the appliance setting core.pre-receive-hook-networking-enabled.
    • 3.22+: network calls are disabled by default and must be explicitly enabled via that same setting.
    • A pointer to review externally-dependent scripts with a site administrator before upgrading.

Documentation only — the example hook scripts don't make network calls, so no script changes were needed.

Co-authored-by: jamesooo <8672454+jamesooo@users.noreply.github.com>
Copilot AI changed the title [WIP] Update README to document GHES networking behavior for pre-receive hooks Document GHES pre-receive hook networking restrictions in README Aug 24, 2026
Copilot AI requested a review from jamesooo August 24, 2026 22:12
@jamesooo
jamesooo requested a review from tyliec August 24, 2026 22:15
@jamesooo
jamesooo marked this pull request as ready for review September 1, 2026 17:08
Copilot AI balanced review requested due to automatic review settings September 1, 2026 17:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@jamesooo
jamesooo merged commit 356b5d4 into master Sep 1, 2026
5 of 6 checks passed
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.

4 participants