Merge development into master

Author: github-actions[bot]

.github/workflows/ci.yml

@@ -16,15 +16,17 @@ on:
     branches: [development]
 
 env:
+  ROOT_DIRECTORY: .
   UI_DIRECTORY: ./frontend
+  SCRIPTS_DIRECTORY: .github/scripts
   UI_ARTIFACT_NAME: ui
 
 jobs:
   Frontend:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v4
+        uses: actions/checkout@v5
 
       - name: Cache node_modules
         uses: actions/cache@v4
@@ -34,7 +36,7 @@ jobs:
           restore-keys: ${{ runner.os }}-modules-
 
       - name: Setup NodeJS
-        uses: actions/setup-node@v4
+        uses: actions/setup-node@v5
         with:
           node-version-file: "${{ env.UI_DIRECTORY }}/.nvmrc"
 
@@ -71,17 +73,23 @@ jobs:
     runs-on: ubuntu-latest
     needs: Frontend
 
+    strategy:
+      fail-fast: false
+      matrix:
+        python-version: [ '3.8', '3.9', '3.10', '3.11', '3.12', '3.13' ]
+    name: Python ${{ matrix.python-version }} backend
+
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v4
+        uses: actions/checkout@v5
 
-      - name: Set up Python 3.8
-        uses: actions/setup-python@v5
+      - name: Set up Python ${{ matrix.python-version }}
+        uses: actions/setup-python@v6
         with:
-          python-version: "3.8"
+          python-version: ${{ matrix.python-version }}
 
       - name: Install UI
-        uses: actions/download-artifact@v4
+        uses: actions/download-artifact@v5
         with:
           name: ${{ env.UI_ARTIFACT_NAME }}
           path: "${{ env.UI_DIRECTORY }}/build"
@@ -91,15 +99,4 @@ jobs:
 
       - name: Unit Tests
         run: |
-          python3 bazarr.py --no-update &
-          PID=$!
-          sleep 15
-          if kill -s 0 $PID
-          then
-            echo "**** Bazarr launch successful ****"
-            kill $PID
-            exit 0
-          else
-          echo "**** Bazarr launch failed ****"
-            exit 1
-          fi
+          bash '${{ env.SCRIPTS_DIRECTORY }}/build_test.sh'

.github/workflows/release_beta_to_dev.yaml

@@ -18,7 +18,7 @@ jobs:
           exit 1
 
       - name: Checkout
-        uses: actions/checkout@v4
+        uses: actions/checkout@v5
         with:
           fetch-depth: ${{ env.FETCH_DEPTH }}
           ref: development
@@ -36,12 +36,12 @@ jobs:
           restore-keys: ${{ runner.os }}-modules-
 
       - name: Setup NodeJS
-        uses: actions/setup-node@v4
+        uses: actions/setup-node@v5
         with:
           node-version-file: "${{ env.UI_DIRECTORY }}/.nvmrc"
 
       - name: Install Global Tools
-        run: npm install -g release-it auto-changelog
+        run: npm install -g release-it@v16.3.0 auto-changelog
 
       - name: Install UI Dependencies
         run: npm install

.github/workflows/release_dev_to_master.yaml

@@ -22,7 +22,7 @@ jobs:
           exit 1
 
       - name: Checkout
-        uses: actions/checkout@v4
+        uses: actions/checkout@v5
         with:
           fetch-depth: 0
           ref: development
@@ -38,12 +38,12 @@ jobs:
           restore-keys: ${{ runner.os }}-modules-
 
       - name: Setup NodeJS
-        uses: actions/setup-node@v4
+        uses: actions/setup-node@v5
         with:
           node-version-file: "${{ env.UI_DIRECTORY }}/.nvmrc"
 
       - name: Install Global Tools
-        run: npm install -g release-it auto-changelog
+        run: npm install -g release-it@v16.3.0 auto-changelog
 
       - name: Install UI Dependencies
         run: npm install
@@ -64,7 +64,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
-        uses: actions/checkout@v4
+        uses: actions/checkout@v5
 
       - name: Merge development -> master
         uses: devmasx/[email protected]

.github/workflows/test_bazarr_execution.yml

@@ -6,8 +6,16 @@ jobs:
     runs-on: ubuntu-latest
     env:
       ROOT_DIRECTORY: .
+      UI_DIRECTORY: ./frontend
       SCRIPTS_DIRECTORY: .github/scripts
       FETCH_DEPTH: 15 # Should be enough
+
+    strategy:
+      fail-fast: false
+      matrix:
+        python-version: [ '3.8', '3.9', '3.10', '3.11', '3.12', '3.13' ]
+    name: Python ${{ matrix.python-version }} test
+
     steps:
       - name: Validate branch
         if: ${{ github.ref != 'refs/heads/development' }}
@@ -16,13 +24,13 @@ jobs:
           exit 1
 
       - name: Checkout
-        uses: actions/checkout@v4
+        uses: actions/checkout@v5
         with:
           fetch-depth: ${{ env.FETCH_DEPTH }}
           ref: development
 
       - name: Setup NodeJS
-        uses: actions/setup-node@v4
+        uses: actions/setup-node@v5
         with:
           node-version-file: "${{ env.UI_DIRECTORY }}/.nvmrc"
 
@@ -34,10 +42,10 @@ jobs:
         run: npm run build
         working-directory: ${{ env.UI_DIRECTORY }}
 
-      - name: Set up Python 3.8
-        uses: actions/setup-python@v5
+      - name: Set up Python ${{ matrix.python-version }}
+        uses: actions/setup-python@v6
         with:
-          python-version: "3.8"
+          python-version: ${{ matrix.python-version }}
 
       - name: Install Python dependencies
         run: |

README.md

@@ -79,6 +79,7 @@ If you need something that is not already part of Bazarr, feel free to create a
 - Subs4Series
 - Subscene
 - Subscenter
+- SubsRo
 - Subsunacs.net
 - SubSynchro
 - Subtitrari-noi.ro

bazarr.py

@@ -10,6 +10,8 @@
 from bazarr.app.get_args import args
 from bazarr.literals import EXIT_PYTHON_UPGRADE_NEEDED, EXIT_NORMAL, FILE_RESTART, FILE_STOP, ENV_RESTARTFILE, ENV_STOPFILE, EXIT_INTERRUPT
 
+# always flush print statements
+sys.stdout.reconfigure(line_buffering=True)
 
 def exit_program(status_code):
     print(f'Bazarr exited with status code {status_code}.')
@@ -63,8 +65,11 @@ def start_bazarr():
 
 
 def terminate_child():
+    global child_process
     print(f"Terminating child process with PID {child_process.pid}")
-    child_process.terminate()
+    if child_process.poll() is None:   # Process is still running
+        child_process.terminate()      # Send termination signal
+    child_process.wait()               # Ensure it exits
 
 
 def get_stop_status_code(input_file):
@@ -132,7 +137,7 @@ def interrupt_handler(signum, frame):
         interrupted = True
         print('Handling keyboard interrupt...')
     else:
-        if not is_process_running(child_process):
+        if not is_process_running(child_process.pid):
             # this will be caught by the main loop below
             raise SystemExit(EXIT_INTERRUPT)
 

bazarr/api/__init__.py

@@ -13,6 +13,7 @@
 from .subtitles import api_ns_list_subtitles
 from .system import api_ns_list_system
 from .webhooks import api_ns_list_webhooks
+from .plex import api_ns_list_plex
 from .swaggerui import swaggerui_api_params
 
 api_ns_list = [
@@ -26,6 +27,7 @@
     api_ns_list_subtitles,
     api_ns_list_system,
     api_ns_list_webhooks,
+    api_ns_list_plex,
 ]
 
 authorizations = {

bazarr/api/plex/__init__.py

@@ -0,0 +1,7 @@
+# coding=utf-8
+
+from flask_restx import Namespace
+api_ns_plex = Namespace('Plex Authentication', description='Plex OAuth and server management')
+
+from .oauth import *  # noqa
+api_ns_list_plex = [api_ns_plex]

bazarr/api/plex/exceptions.py

@@ -0,0 +1,36 @@
+# coding=utf-8
+
+class PlexAuthError(Exception):
+    def __init__(self, message, status_code=500, error_code=None):
+        super().__init__(message)
+        self.message = message
+        self.status_code = status_code
+        self.error_code = error_code
+
+class InvalidTokenError(PlexAuthError):
+    def __init__(self, message="Invalid or malformed Plex authentication token. Please re-authenticate with Plex."):
+        super().__init__(message, status_code=401, error_code="INVALID_TOKEN")
+
+class TokenExpiredError(PlexAuthError):
+    def __init__(self, message="Plex authentication token has expired. Please re-authenticate with Plex to continue."):
+        super().__init__(message, status_code=401, error_code="TOKEN_EXPIRED")
+
+class PlexConnectionError(PlexAuthError):
+    def __init__(self, message="Unable to establish connection to Plex server. Please check server status and network connectivity."):
+        super().__init__(message, status_code=503, error_code="CONNECTION_ERROR")
+
+class PlexServerNotFoundError(PlexAuthError):
+    def __init__(self, message="Plex server not found or not accessible. Please verify server URL and authentication credentials."):
+        super().__init__(message, status_code=404, error_code="SERVER_NOT_FOUND")
+
+class PlexPinExpiredError(PlexAuthError):
+    def __init__(self, message="Plex authentication PIN has expired. Please request a new PIN and try again."):
+        super().__init__(message, status_code=410, error_code="PIN_EXPIRED")
+
+class PlexAuthTimeoutError(PlexAuthError):
+    def __init__(self, message="Plex authentication process timed out. Please try again or check your internet connection."):
+        super().__init__(message, status_code=408, error_code="AUTH_TIMEOUT")
+
+class UnauthorizedError(PlexAuthError):
+    def __init__(self, message="Access denied. Please check your Plex authentication credentials and permissions."):
+        super().__init__(message, status_code=401, error_code="UNAUTHORIZED")