Skip to content

Detect unwritable Paper configuration files - #193

Merged
pavog merged 2 commits into
masterfrom
paper-config-not-writable
Nov 13, 2026
Merged

pavog merged 2 commits into
masterfrom
paper-config-not-writable

Conversation

@pavog

@pavog pavog commented Nov 13, 2026

Copy link
Copy Markdown
Member
[16:52:55 WARN]: [Configurations] Could not save config/paper-global.yml: Paper could not persist the full set of configuration settings in the configuration file. Any setting missing from the configuration file will be set with its default value in memory. Admins should make sure to review the configuration documentation at https://docs.papermc.io/paper/configuration for more details.
org.spongepowered.configurate.ConfigurateException: []: java.nio.file.AccessDeniedException: /home/container/config/.118220814970875303942195paper-global.yml.tmp

When Paper cannot save one of its configuration files, it logs this warning and keeps settings that are missing from the file only in memory.
This was not detected before.
This PR adds a problem that names the affected file and suggests checking its permissions.

Paper only logs this warning for an AccessDeniedException; other save errors are thrown, so the pattern anchors on the fixed Paper text and captures only the path.
The exception is about a temporary file next to the configuration file, because Configurate saves atomically, so the solution names the configuration file from the message rather than the temporary file.
The existing PaperCheckPermissionsSolution is reused instead of adding a new solution.
The problem is registered on Paper, so Folia inherits it; Spigot and Bukkit don't have this message.

Changes

  • add ConfigNotWritableProblem for Paper
  • register it in PaperAnalyser
  • add paper-config-not-writable-problem translation
  • add paper-config-not-writable-26-3.log fixture

@pavog
pavog requested a review from JulianVennen November 13, 2026 15:56
…itable

# Conflicts:
#	lang/en.json
#	src/Analyser/PaperAnalyser.php
@pavog
pavog merged commit 765da06 into master Nov 13, 2026
3 checks passed
@pavog
pavog deleted the paper-config-not-writable branch November 13, 2026 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants