Merge pull request 'Repin UPSTREAM_REF to jellyfin-ha-src main' (#13) from benvin/jellyfin-v0.3.2 into main

Reviewed-on: #13

.woodpecker/build.yaml

@@ -5,7 +5,7 @@ steps:
   # Clone the pinned upstream jellyfin-ha source and publish the .NET server
   # into ./publish-output (consumed by Dockerfile.runtime).
   - name: publish
-    image: mcr.microsoft.com/dotnet/sdk:9.0
+    image: mcr.microsoft.com/dotnet/sdk:10.0
     commands:
       - |
         REF=$$(cat UPSTREAM_REF)
@@ -26,7 +26,7 @@ steps:
 
   # Validate the runtime image builds (no push on PRs).
   - name: docker-build
-    image: woodpeckerci/plugin-docker-buildx
+    image: artifactapi.k8s.syd1.au.unkin.net/docker-internal/plugin-docker-buildx:latest
     settings:
       repo: artifactapi.k8s.syd1.au.unkin.net/docker-internal/jellyfin-ha
       dockerfile: Dockerfile.runtime

.woodpecker/docker.yaml

@@ -5,7 +5,7 @@ when:
 steps:
   # Same publish step as the PR pipeline: clone pinned upstream + dotnet publish.
   - name: publish
-    image: mcr.microsoft.com/dotnet/sdk:9.0
+    image: mcr.microsoft.com/dotnet/sdk:10.0
     commands:
       - |
         REF=$$(cat UPSTREAM_REF)
@@ -24,44 +24,25 @@ steps:
             memory: 6Gi
             cpu: 4
 
-  # Stage the internal (Vault) CA into the shared workspace so the buildkit push
-  # below can verify artifactapi's TLS cert. almalinux9-base already trusts the
-  # unkin CA (it is the image the RPM release pipelines use to reach artifactapi
-  # over HTTPS), so its consolidated trust bundle contains the chain we need.
-  - name: ca-trust
-    image: git.unkin.net/unkin/almalinux9-base:20260606
-    commands:
-      - cp /etc/pki/tls/certs/ca-bundle.crt "$${CI_WORKSPACE}/artifactapi-ca.crt"
-    depends_on: [publish]
-    backend_options:
-      kubernetes:
-        serviceAccountName: default
-        resources:
-          requests:
-            memory: 256Mi
-            cpu: 250m
-          limits:
-            memory: 512Mi
-            cpu: 1
-
   # Build the runtime image and push it to the artifactapi local docker registry.
-  # buildkit_config points buildkit at the staged CA so the TLS handshake with
-  # artifactapi (Vault-signed cert) verifies; buildx copies the referenced CA
-  # into the buildkitd container under /etc/buildkit/certs when it creates the
-  # builder. CI_WORKSPACE is runtime-only so the path is the fixed workspace path.
+  # The plugin image bakes artifactapi's internal (Vault) CA at
+  # /etc/docker/certs.d/<registry>/ca.crt; buildkit_config points the buildx
+  # docker-container builder at that in-image CA. buildkitd runs in its own
+  # container and performs the push, so it needs the CA via --config even though
+  # the plugin image already trusts it — buildx copies the referenced file in.
   - name: docker
-    image: woodpeckerci/plugin-docker-buildx
+    image: artifactapi.k8s.syd1.au.unkin.net/docker-internal/plugin-docker-buildx:latest
     settings:
       registry: artifactapi.k8s.syd1.au.unkin.net
       repo: artifactapi.k8s.syd1.au.unkin.net/docker-internal/jellyfin-ha
       dockerfile: Dockerfile.runtime
       buildkit_config: |
         [registry."artifactapi.k8s.syd1.au.unkin.net"]
-          ca = ["/woodpecker/src/git.unkin.net/unkin/jellyfin-ha/artifactapi-ca.crt"]
+          ca = ["/etc/docker/certs.d/artifactapi.k8s.syd1.au.unkin.net/ca.crt"]
       tags:
         - ${CI_COMMIT_TAG}
         - latest
-    depends_on: [ca-trust]
+    depends_on: [publish]
     backend_options:
       kubernetes:
         serviceAccountName: default

Dockerfile.runtime

@@ -18,10 +18,46 @@ RUN apt-get update \
  && echo "deb [arch=amd64 signed-by=/usr/share/keyrings/jellyfin.gpg] https://repo.jellyfin.org/debian bookworm main" \
     > /etc/apt/sources.list.d/jellyfin.list \
  && apt-get update \
- && apt-get install -y --no-install-recommends "jellyfin-web=10.11.6+deb12" \
+ && apt-get install -y --no-install-recommends "jellyfin-web=12.0+deb12" \
  && rm -rf /var/lib/apt/lists/*
 
+# ── Plugin stage ──────────────────────────────────────────────────────────────
+# Download and verify the auth plugins, unpacked into versioned dirs baked into
+# the image and synced into /config/plugins at start (docker-entrypoint.sh).
+# Versions are the newest each plugin publishes whose targetAbi <= the pinned
+# Jellyfin server version (12.0.0):
+#   LDAP Authentication 24.0.0.0  targetAbi 12.0.0.0
+#   SSO Authentication  4.0.0.4   targetAbi 10.11.0.0 (newest release; loads on 12.0)
+# sha256 pins make each fetch reproducible.
+FROM --platform=linux/amd64 debian:bookworm-slim AS plugins
+
+RUN apt-get update \
+ && apt-get install -y --no-install-recommends curl ca-certificates unzip \
+ && rm -rf /var/lib/apt/lists/*
+
+ARG LDAP_URL=http://artifactapi.k8s.syd1.au.unkin.net/api/v1/remote/jellyfin/files/plugin/ldap-authentication/ldap-authentication_24.0.0.0.zip
+ARG LDAP_SHA256=3be1f9d6a6ce9ea375e556dd30136d178a8dbe35cbe866d30d3451dc3ff7e804
+# LDAP is served through artifactapi remote. SSO is served through the artifactapi
+# github proxy, which the CI build network can reach (github is not directly reachable).
+ARG SSO_URL=http://artifactapi.k8s.syd1.au.unkin.net/api/v1/remote/github/9p4/jellyfin-plugin-sso/releases/download/v4.0.0.4/sso-authentication_4.0.0.4.zip
+ARG SSO_SHA256=c09f16ba31059a434ddd7f811e4f9608d4b4c4514cc80a5bf1ca33bee61e1107
+
+WORKDIR /plugins
+RUN set -eu; \
+    curl -fsSL "$LDAP_URL" -o ldap.zip; \
+    echo "$LDAP_SHA256  ldap.zip" | sha256sum -c -; \
+    mkdir -p "LDAP Authentication_24.0.0.0"; \
+    unzip -oq ldap.zip -d "LDAP Authentication_24.0.0.0"; \
+    curl -fsSL "$SSO_URL" -o sso.zip; \
+    echo "$SSO_SHA256  sso.zip" | sha256sum -c -; \
+    mkdir -p "SSO Authentication_4.0.0.4"; \
+    unzip -oq sso.zip -d "SSO Authentication_4.0.0.4"; \
+    rm -f ldap.zip sso.zip
+
 # ── Runtime stage ─────────────────────────────────────────────────────────────
+# .NET 10 runtime: matches the SDK 10.0 publish step (framework-dependent) and the
+# fork's net10.0 TFM, so the app's required Microsoft.NETCore.App 10.0 is present.
+# Keep in lockstep with the `mcr.microsoft.com/dotnet/sdk` major in .woodpecker/*.yaml.
 FROM --platform=linux/amd64 mcr.microsoft.com/dotnet/aspnet:10.0
 
 # FFmpeg and the native deps required by SkiaSharp and fontconfig.
@@ -39,6 +75,9 @@ WORKDIR /jellyfin
 COPY publish-output/ .
 # jellyfin-web client assets from the webclient stage.
 COPY --from=webclient /usr/share/jellyfin/web ./jellyfin-web/
+# Baked auth plugins; docker-entrypoint.sh syncs these into /config/plugins.
+COPY --from=plugins /plugins /usr/share/jellyfin/plugins-baked
+COPY --chmod=0755 docker-entrypoint.sh /usr/local/bin/docker-entrypoint.sh
 
 # Jellyfin default ports
 EXPOSE 8096
@@ -51,7 +90,4 @@ ENV JELLYFIN_DATA_DIR=/config \
     JELLYFIN_CACHE_DIR=/cache \
     JELLYFIN_LOG_DIR=/config/log
 
-ENTRYPOINT ["./jellyfin", \
-            "--datadir", "/config", \
-            "--cachedir", "/cache", \
-            "--webdir", "/jellyfin/jellyfin-web"]
+ENTRYPOINT ["/usr/local/bin/docker-entrypoint.sh"]

UPSTREAM_REF

@@ -1 +1 @@
-2e1e445e470c2f2c1520f66678a73faa226c2058
+ec581b5e5f156edb862d01f4bd07d9ba51903ed6

docker-entrypoint.sh

@@ -0,0 +1,68 @@
+#!/bin/sh
+# Sync image-baked plugins into the /config (datadir) plugins directory on
+# start. /config is an RWX volume shared across replicas, so plugins baked
+# into the image are invisible until copied in here -- and this must be safe
+# when several replicas start (or restart) at the same instant:
+#   - skip entirely once the correct version is already in place, so the
+#     steady state (almost every start) never touches the shared volume;
+#   - install a new/changed version via copy-to-staging + atomic rename, so
+#     no reader (another replica, or this container's own jellyfin process)
+#     ever observes a partially-written plugin directory. A replica that
+#     loses the rename race just discards its own copy -- that's success,
+#     not an error;
+#   - drop stale, differently-versioned copies of the same plugin afterwards
+#     so they don't shadow the current one. Best-effort: another replica may
+#     already be doing, or have finished, the same cleanup.
+# Deliberately no locking: a lock held by a replica that dies mid-sync would
+# wedge every future start on this volume, which is worse than the race it
+# would prevent.
+set -eu
+
+BAKED_DIR=/usr/share/jellyfin/plugins-baked
+PLUGIN_DIR=/config/plugins
+STAGING_DIR="$PLUGIN_DIR/.sync-tmp"
+REPLICA=$(hostname)
+
+if [ -d "$BAKED_DIR" ]; then
+  mkdir -p "$PLUGIN_DIR" "$STAGING_DIR"
+  for src in "$BAKED_DIR"/*; do
+    [ -d "$src" ] || continue
+    name=$(basename "$src")   # e.g. "LDAP Authentication_24.0.0.0"
+    base=${name%_*}           # plugin name without the trailing _<version>
+    target="$PLUGIN_DIR/$name"
+
+    if [ ! -d "$target" ]; then
+      # Build the new version privately (keyed by this replica's own
+      # hostname, so concurrent replicas never share a staging path), then
+      # move it into place in one atomic rename. mv -T fails with
+      # "Directory not empty" if another replica's rename already won --
+      # that's fine, our copy just becomes garbage we discard.
+      staging="$STAGING_DIR/$REPLICA.$name"
+      rm -rf "$staging"
+      cp -a "$src" "$staging"
+      if mv_err=$(mv -T "$staging" "$target" 2>&1); then
+        :
+      elif [ -d "$target" ]; then
+        rm -rf "$staging"
+      else
+        echo "docker-entrypoint: failed to install plugin $name: $mv_err" >&2
+        exit 1
+      fi
+    fi
+
+    # Remove any other version of this plugin so it can't shadow the one
+    # above. Another replica may be racing the same cleanup, or have already
+    # finished it -- an entry that's already gone is success, not an error.
+    for existing in "$PLUGIN_DIR/$base"_*; do
+      [ -e "$existing" ] || continue
+      [ "$existing" = "$target" ] && continue
+      rm -rf "$existing" 2>/dev/null || true
+    done
+  done
+fi
+
+exec ./jellyfin \
+  --datadir /config \
+  --cachedir /cache \
+  --webdir /jellyfin/jellyfin-web \
+  "$@"