docs: correct OWNCLOUD_HTTP2_ENABLED default (7.1)

[DeepDiver1975]

Summary

The documented default for OWNCLOUD_HTTP2_ENABLED was "depends on Qt version". In client 7.1 HTTP2 is force-disabled by default: the attribute is set to qEnvironmentVariableIntValue("OWNCLOUD_HTTP2_ENABLED") == 1, so anything other than "1" (including unset) leaves HTTP2 disabled. There is no Qt-version branching.

Updates the Default column to 0 (disabled) and clarifies it must be force-enabled with "1" (and still requires server support).

Source of truth

owncloud/client @ 7.1: 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]

7.1 counterpart of #764 — corrects the documented default for OWNCLOUD_HTTP2_ENABLED from "depends on Qt version" to "0 (disabled)".

Findings

Nothing blocking. Diff is byte-identical to the master PR (#764), targets the 7.1 base correctly, and is AsciiDoc-valid. Same scope caveat as #764: the owncloud/client source citation isn't independently re-verified here (out of this agent's repo scope).

Verdict

Approved

🤖 Automated review by Claude Code

---

Generated by Claude Code