Conversation
Restores the ImageMagick preview providers that 33.0.7 disabled upstream, plus the stable33 sharing, encryption and security-hardening backports. Also refreshes the filebrowser-startos#next pin and folds in the TODO item: clear the abandoned nextcloud:create-admin-user task replay key, left behind when that action was renamed to get-admin-credentials. start-sdk is already on 2.0.9, the latest published version.
MattDHill
approved these changes
Aug 15, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bumps the bundled Nextcloud image from 33.0.7 to 33.0.8, releasing as StartOS version 33.0.8:0.
Per
UPDATING.md, the pin is theNEXTCLOUD_VERSIONARGdefault innextcloud.Dockerfile.Upstream
nextcloud:33.0.8-apacheis published (2026-08-14) and resolves for amd64 and arm64.UPDATING.mddescribes this as a patch bump — so this stays on 33.FROM php:8.4-apache-trixiewithimagick-3.8.1; nothing changed in the image beyond the version.postgres:17-alpine) and Valkey (valkey/valkey:9-alpine) track floating majors and are unchanged.The 33.0.7 known issue is resolved. 33.0.7 shipped server#62148, which hard-returned
falsefromIMagickSupport::hasExtension()andsupportsFormat(), disabling ImageMagick previews. Inv33.0.8both early returns are gone, so thumbnails for PDF, SVG, TIFF, HEIC, PSD and the rest work again. Verified by diffinglib/private/Preview/IMagickSupport.phpbetween the two tags.Other notable backports: sharing fixes (missing recipient/owner no longer breaks a share, rejected shares stay rejected), encryption fixes (zero-byte encrypted files reported the wrong size), security hardening (host/IP validation, code signing revocation list, admin permission required for all system tag changes), plus federated calendar accept/decline and a setting to skip password confirmation on selected IP ranges. Full changelog.
Also in this release
nextcloud:create-admin-usertask replay key — theTODO.mditem recorded in 254ec77, folded into this release'smigrations.upas it asked. The action was renamedcreate-admin-user→get-admin-credentialsin479c4e1, orphaning the old replay key; nothing reaps it, and because the id no longer exists the task can never be resolved or run away.sdk.action.clearTaskis a free no-op where the key is absent. Confirmed safe inside a migration:clear_tasksinstart-coreis a pure DB mutation with noservices.get, so it can't hit thecreate_taskinit deadlock.TODO.mdis now empty.filebrowser-startos#nextpineee37f2→c35ff45(File Browser 2.63.23:0, start-sdk 2.0.9). ItsversionRangestays^2.62.2:1— nothing in its contract changed, and raising the floor would needlessly exclude working installs.create-admin-userinstead ofget-admin-credentials— the same rename this release cleans up after.@start9labs/start-sdkis already pinned at2.0.9, the latest on npm. Thepackage-lock.jsongrowth is the SDK's bundled@start9labs/start-coresubtree being recorded (16 additiveinBundleentries, 0 removals) — the slot'snode_moduleshad been stale at 2.0.7.npm ciis green and there is exactly one SDK copy in the tree.Version placement
33.0.7:0is tagged (v33.0.7_0) and is the registry's current best, so33.0.8:0is free.current.tsis edited in place rather than freezing33.0.7:0intoother[]: the addedclearTaskis idempotent and purely additive,migrateFrom035xis retained oncurrentunchanged, and this matches how every prior 33.0.x bump in this repo was shipped.Test plan
npm run checkgreen (33.0.8:0).npm cifrom the committed lockfile succeeds.nextcloud:33.0.8-apacheverified present on Docker Hub for amd64 + arm64.make/ install-and-upgrade smoke test (PR review).