Skip to content

docs(admin): correct the antivirus Stream Length default - #15539

Merged
miaulalala merged 1 commit into
masterfrom
fix/antivirus-stream-length-default
Sep 1, 2026
Merged

docs(admin): correct the antivirus Stream Length default#15539
miaulalala merged 1 commit into
masterfrom
fix/antivirus-stream-length-default

Conversation

@miaulalala

@miaulalala miaulalala commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

☑️ Resolves

The "Daemon (Socket)" section of the antivirus page gives the wrong default for Stream Length.

It states 10485760 bytes, "ten megabytes". The actual default in nextcloud/files_antivirus is
26214400 (25 MiB), set as AV_STREAM_MAX_LENGTH in lib/AppInfo/ConfigLexicon.php:

key: self::AV_STREAM_MAX_LENGTH,
defaultRaw: 26214400,

So the documented figure is out by a factor of 2.5. It matters because the same paragraph advises
keeping the value below PHP's memory_limit — an admin sizing memory_limit against the documented
10 MiB would under-provision for the real 25 MiB default.

Found while documenting ICAP mode (#15538), which needed the same setting's real value.

🖼️ Screenshots

Text-only change to one sentence; no visual change and no new images.

✅ Checklist

  • I have built the documentation locally and reviewed the output
  • sphinx-lint is clean
  • Screenshots are included for visual changes (not applicable, text only)
  • I have not moved or renamed pages
  • I have run codespell — not installed locally, please let CI confirm

The Daemon (Socket) section states 10485760 bytes, ten megabytes. The actual
default in files_antivirus is 26214400 (25 MiB), per AV_STREAM_MAX_LENGTH in
lib/AppInfo/ConfigLexicon.php.

AI-Assisted-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Signed-off-by: Anna Larch <anna@nextcloud.com>
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

📖 Documentation Preview

🔍 Open preview →

📄 1 changed documentation page

Last updated: Tue, 01 Sep 2026 20:07:13 GMT

@miaulalala

Copy link
Copy Markdown
Contributor Author

/backport to stable35

@miaulalala

Copy link
Copy Markdown
Contributor Author

/backport to stable34

@miaulalala

Copy link
Copy Markdown
Contributor Author

/backport to stable33

@miaulalala miaulalala self-assigned this Sep 1, 2026
@miaulalala
miaulalala enabled auto-merge September 1, 2026 20:32
@miaulalala
miaulalala merged commit 04caf13 into master Sep 1, 2026
26 checks passed
@miaulalala
miaulalala deleted the fix/antivirus-stream-length-default branch September 1, 2026 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants