Skip to content

docs(config): clarify overwrite.cli.url description#60185

Merged
Altahrim merged 1 commit intomasterfrom
jtr/docs-config-ovewriteCLIUrl
May 7, 2026
Merged

docs(config): clarify overwrite.cli.url description#60185
Altahrim merged 1 commit intomasterfrom
jtr/docs-config-ovewriteCLIUrl

Conversation

@joshtrichards
Copy link
Copy Markdown
Member

Summary

Clarify the overwrite.cli.url description in config/config.sample.php so it better reflects how Nextcloud uses the value during CLI and background execution.

The updated text:

  • explains that the setting is used when generating URLs outside a normal web request
  • calls out common contexts like background jobs, occ, and notifications
  • notes that installation seeds the value from the installer’s current access context when it is not already preseeded
  • warns that the inferred value may not match the public URL users actually use and thus may need manual adjustment

Notes

This is a documentation-only change. No runtime behavior is changed.

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

…nual

Signed-off-by: Josh <josh.t.richards@gmail.com>
@joshtrichards joshtrichards added this to the Nextcloud 34 milestone May 6, 2026
@joshtrichards joshtrichards marked this pull request as ready for review May 6, 2026 14:15
@joshtrichards joshtrichards requested a review from a team as a code owner May 6, 2026 14:15
@joshtrichards joshtrichards requested review from Altahrim, ArtificialOwl, leftybournes and sorbaugh and removed request for a team May 6, 2026 14:15
@joshtrichards joshtrichards changed the title docs(config): clarify overwrite.cli.url parameter description and guidance docs(config): clarify overwrite.cli.url description May 6, 2026
@joshtrichards joshtrichards requested review from solracsf and tflidd May 6, 2026 14:19
@Altahrim Altahrim merged commit fffba55 into master May 7, 2026
194 of 198 checks passed
@Altahrim Altahrim deleted the jtr/docs-config-ovewriteCLIUrl branch May 7, 2026 09:27
@nextcloud-bot nextcloud-bot mentioned this pull request May 7, 2026
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.

3 participants