fix: chown /appdata volume so non-root user can create subdirectories

Removed unnecessary ownership changes for websocket and tasks files.

apps/nextjs/src/app/[locale]/boards/(content)/_creator.tsx

@@ -1,4 +1,3 @@
-import { cache } from "react";
 import type { Metadata } from "next";
 import { TRPCError } from "@trpc/server";
 
@@ -35,20 +34,17 @@ interface Props<TParams extends Params> {
 export const createBoardContentPage = <TParams extends Record<string, unknown>>({
   getInitialBoardAsync: getInitialBoard,
 }: Props<TParams>) => {
-  const getCachedBoard = cache((_paramsKey: string, params: TParams) => getInitialBoard(params));
-  const getBoardForParams = (params: TParams) => getCachedBoard(JSON.stringify(params), params);
-
   return {
     layout: createBoardLayout({
       headerActions: <BoardContentHeaderActions />,
-      getInitialBoardAsync: getBoardForParams,
+      getInitialBoardAsync: getInitialBoard,
     }),
     // eslint-disable-next-line no-restricted-syntax
     page: async ({ params }: { params: Promise<TParams> }) => {
       const resolvedParams = await params;
       const queryClient = getQueryClient();
 
-      const [board, session] = await Promise.all([getBoardForParams(resolvedParams), auth()]);
+      const [board, session] = await Promise.all([getInitialBoard(resolvedParams), auth()]);
 
       const itemsMap = board.items.reduce((acc, item) => {
         const existing = acc.get(item.kind);
@@ -85,7 +81,7 @@ export const createBoardContentPage = <TParams extends Record<string, unknown>>(
     },
     generateMetadataAsync: async ({ params }: { params: Promise<TParams> }): Promise<Metadata> => {
       try {
-        const board = await getBoardForParams(await params);
+        const board = await getInitialBoard(await params);
         const t = await getI18n();
 
         return {
@@ -101,6 +97,7 @@ export const createBoardContentPage = <TParams extends Record<string, unknown>>(
           },
         };
       } catch (error) {
+        // Ignore not found errors and return empty metadata
         if (error instanceof TRPCError && error.code === "NOT_FOUND") {
           return {};
         }

packages/old-import/src/widgets/options.ts

@@ -44,7 +44,6 @@ const optionMapping: OptionMapping = {
     hideIcon: (oldOptions) => oldOptions.items.some((item) => item.hideIcon),
     hideHostname: (oldOptions) => oldOptions.items.some((item) => item.hideHostname),
     openNewTab: (oldOptions) => oldOptions.items.some((item) => item.openNewTab),
-    withBorder: () => undefined,
   },
   calendar: {
     releaseType: (oldOptions) => [oldOptions.radarrReleaseType],

scripts/entrypoint.sh

@@ -9,12 +9,21 @@ echo "Starting with UID='$PUID', GID='$PGID'"
 if [ "${PUID}" != "0" ] || [ "${PGID}" != "0" ]; then
     # The below command will change the owner of all files in the /app directory (except node_modules) to the new UID and GID
     echo "Changing owner to $PUID:$PGID, this will take about 10 seconds..."
-    find . -name 'node_modules' -prune -o -mindepth 1 -maxdepth 1 -exec chown -R $PUID:$PGID {} +
+    chown $PUID:$PGID ./run.sh
+    chown $PUID:$PGID ./apps/cli
+    chown $PUID:$PGID ./apps/nextjs/server.js
+    chown $PUID:$PGID ./apps/nextjs/.next
+    mkdir -p ./apps/nextjs/.next/cache
+    chown $PUID:$PGID ./apps/nextjs/.next/cache
+    chown $PUID:$PGID ./db/migrations/sqlite/migrate.cjs
+    chown $PUID:$PGID ./db/migrations/mysql/migrate.cjs
+    chown $PUID:$PGID ./build/better_sqlite3.node
     chown -R $PUID:$PGID /var/cache/nginx
     chown -R $PUID:$PGID /var/log/nginx
     chown -R $PUID:$PGID /var/lib/nginx
     chown -R $PUID:$PGID /run/nginx/nginx.pid
     chown -R $PUID:$PGID /etc/nginx
+    chown $PUID:$PGID /appdata
     echo "Changing owner to $PUID:$PGID, done."
 fi