Skip to content

Set HTTP request timeout on the Graph API httpclient - #3407

Open
Lai Wei (weilai-irl) wants to merge 1 commit into
MOODLE_405_STABLEfrom
wip-130866-m405
Open

Set HTTP request timeout on the Graph API httpclient#3407
Lai Wei (weilai-irl) wants to merge 1 commit into
MOODLE_405_STABLEfrom
wip-130866-m405

Conversation

@weilai-irl

Copy link
Copy Markdown
Collaborator

Set HTTP request timeout on the Graph API httpclient to prevent scheduled tasks hanging indefinitely

Copilot AI lite review requested due to automatic review settings August 14, 2026 15:30
@weilai-irl Lai Wei (weilai-irl) self-assigned this Aug 14, 2026
@weilai-irl Lai Wei (weilai-irl) added this to the 2026-03 milestone Aug 14, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a default overall HTTP request timeout to the Microsoft Graph API HTTP client to prevent long-running processes (notably scheduled tasks) from hanging indefinitely on stalled/unreachable remote endpoints.

Changes:

  • Add an httpclient constructor that sets CURLOPT_TIMEOUT to 300 seconds for all requests.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread local/o365/classes/httpclient.php
Set HTTP request timeout on the Graph API httpclient to prevent scheduled tasks hanging indefinitely
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