docs: correct OWNCLOUD_HTTP2_ENABLED default

[DeepDiver1975]

master counterpart of #763. HTTP2 is force-disabled by default in client 7.1 (flag set only when the variable equals "1"); the docs said "depends on Qt version". Source: src/libsync/accessmanager.cpp:95-98.

🤖 Generated with Claude Code

[DeepDiver1975]

Review: correct OWNCLOUD_HTTP2_ENABLED documented default

Verdict: APPROVE (clean). The correction is accurate and verified against source.

Accuracy verified against source

owncloud/client @ src/libsync/accessmanager.cpp:

// http2 seems to cause issues, as with our recommended server setup we don't support http2, disable it by default for now
static const bool http2EnabledEnv = qEnvironmentVariableIntValue("OWNCLOUD_HTTP2_ENABLED") == 1;
newRequest.setAttribute(QNetworkRequest::Http2AllowedAttribute, http2EnabledEnv);

qEnvironmentVariableIntValue() returns 0 when the variable is unset or non-numeric, so the == 1 test is false unless the value is exactly "1". HTTP/2 is therefore force-disabled by default, and only "1" enables it. There is no Qt-version branching anywhere in this path — the old "depends on Qt version" text was simply wrong, and the old "force-disable ("0")" wording was also misleading (any non-"1" value, not specifically "0", leaves it disabled). The new wording (0 (disabled) + "must be force-enabled by setting this variable to "1"; any other value leaves it disabled") matches the code exactly. I confirmed this logic is identical on both master and the 7.1 branch.

Formatting / CI

• AsciiDoc is clean: the edit replaces the Default and Description cells of the 3-column table entry; the 0 (disabled) value matches the style of neighboring Default cells, and the trailing "Note that HTTP2 use also depends on the server" line is preserved.
• CI: lint + Build documentation pass.

Minor (non-blocking)

The doc statement is scoped slightly broader than the code: the env var is only consulted for https URLs in the access manager. This is a reasonable simplification for user-facing docs and does not warrant a change.

Changelog

This repo has no changelog directory or CHANGELOG.* file — n/a.

Auto-merge: N/A (agent-authored PR; GitHub blocks self-approval). This is a review comment only.

🤖 Automated PR review by Claude Code review agent.

[dj4oC]

Corrects the documented default for OWNCLOUD_HTTP2_ENABLED from "depends on Qt version" to "0 (disabled)", matching the master/7.1 pair (#763).

Findings

Nothing blocking. The table edit is AsciiDoc-valid and internally consistent with the paired 7.1 PR. Note: owncloud/client (the cited source for accessmanager.cpp:95-98) is outside this review agent's repo scope, so the source citation itself isn't independently re-verified here — the diff is otherwise clean.

Verdict

Approved

🤖 Automated review by Claude Code

---

Generated by Claude Code