Skip to content

Conversation

@beanbaganalytica
Copy link
Contributor

@beanbaganalytica beanbaganalytica commented Jan 22, 2026

Summary

This PR fixes two issues with Jellyfin integration:

  1. Path-specific scanning: JellyfinServer.scan_path() was ignoring the path parameter and calling /Library/Refresh, which triggers a full scan of ALL libraries. Now uses /Library/Media/Updated to only refresh the library containing the specified path.

  2. Modern authentication: Updates from legacy X-MediaBrowser-Token header to modern Authorization: MediaBrowser Token="..." format, which is required when EnableLegacyAuthorization is disabled (the default in modern Jellyfin).

Changes

  • Uses /Library/Media/Updated endpoint with proper MediaUpdateInfoDto payload
  • Uses modern Authorization: MediaBrowser Token="..." header format
  • Adds debug logging for scanned paths
  • Fixed JellyfinServer.scan_path(), MediaServerScanner._scan_jellyfin(), and test connection endpoint

Test plan

  • Tested with Radarr webhook imports
  • Verified Jellyfin logs show targeted library refresh instead of "Validating media library"
  • Confirmed movies appear in Jellyfin without full library scan
  • Tested with Jellyfin 10.10.3 with EnableLegacyAuthorization=false (default)

Supersedes #6 (auth fix is included in this PR)

…fresh

The JellyfinServer.scan_path() method was ignoring the path parameter and
calling /Library/Refresh, which triggers a full scan of ALL libraries.

This change:
- Uses /Library/Media/Updated endpoint which accepts specific paths
- Only refreshes the library containing the specified path
- Uses modern Authorization header format (MediaBrowser Token="...")
- Adds proper JSON payload with Updates array

This matches the behavior of the Plex implementation which correctly uses
path-specific scanning.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@beanbaganalytica beanbaganalytica force-pushed the fix/jellyfin-path-specific-scan branch from e42c233 to 268458e Compare January 23, 2026 00:51
@Pukabyte Pukabyte merged commit c1ded37 into Pukabyte:main Jan 23, 2026
1 of 2 checks passed
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