Guests whitelist revision
Open
Nobody has claimed this yet.
0. Needs triage
enhancement
feature: whitelist
- Dominant language
- JavaScript
- Stars
- 61
- Forks
- 34
- Avg merge
- 3d 19h
- Merged PRs (30d)
- 22
Description
WHITELIST_ALWAYS and DEFAULT_WHITELIST could be updated to include some more of the apps shipped (and enabled by default) with Nextcloud core, to avoid confusion like "Guest account can't preview PDF files", as files_pdfviewer is not included in the default list.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Read lib/AppWhitelist.php around lines 52-54, then compare the default-enabled applications listed in Nextcloud core's core/shipped.json around line 56. Update WHITELIST_ALWAYS and DEFAULT_WHITELIST to include the appropriate shipped apps, including files_pdfviewer, and verify the whitelist reflects the selected core apps.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- authorization
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100