nextcloud / nextcloud/updater

[Bug]: Webbased updater does not use 'localstorage.umask' settings

Open
#651 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

0. Needs triage enhancement
Dominant language
PHP
Stars
51
Forks
40
Avg merge
2d 19h
Merged PRs (30d)
3

Description

⚠️ This issue respects the following points: ⚠️
Bug description

Using a non-standard umask:

config.php:
.
.
'localstorage.umask' => 0072
.
.

User files are created with permission 604.

Using webbased updater results in permission 640.

File permission 604 ist required at hetzner.com (see https://docs.hetzner.com/konsoleh/general/faq/file-authorization/)

Steps to reproduce
  1. Set non-standard umask in config.php
  2. Perform nexcloud or app upgrade
  3. Check permission
Expected behavior

Correct usage of umask-setting.

Nextcloud Server version

31

Operating system

Debian/Ubuntu

PHP engine version

PHP 8.3

Web server

Apache (supported)

Database engine version

MySQL

Is this bug present after an update or on a fresh install?

None

Are you using the Nextcloud Server Encryption module?

Encryption is Enabled

What user-backends are you using?
  • Default user-backend (database)
  • LDAP/ Active Directory
  • SSO - SAML
  • Other
Configuration report

List of activated Apps

Nextcloud Signing status

Nextcloud Logs

Additional info

No response

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the web-based updater flow and how it reads config.php during a Nextcloud or app upgrade, focusing on the localstorage.umask setting. Reproduce the upgrade on Debian/Ubuntu with PHP 8.3 and verify that created files use the configured 604 permission rather than 640.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
release
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.