From f5cf162e67790488ac3394e3e409348b83ca79e6 Mon Sep 17 00:00:00 2001
From: cxfksword <718792+cxfksword@users.noreply.github.com>
Date: Sat, 30 Dec 2023 15:00:10 +0800
Subject: [PATCH] feat: add large poster configuration and optimize image
 handle. close #59 close #61

---
 .../Configuration/PluginConfiguration.cs      |  4 ++
 .../Configuration/configPage.html             | 10 ++++
 .../Model/DoubanSubject.cs                    | 19 ++++++++
 .../Providers/BaseProvider.cs                 | 47 +++++++------------
 .../Providers/MovieImageProvider.cs           |  4 +-
 .../Providers/SeasonImageProvider.cs          |  2 +-
 .../Providers/SeasonProvider.cs               |  1 +
 .../Providers/SeriesImageProvider.cs          |  4 +-
 README.md                                     |  4 +-
 9 files changed, 60 insertions(+), 35 deletions(-)

diff --git a/Jellyfin.Plugin.MetaShark/Configuration/PluginConfiguration.cs b/Jellyfin.Plugin.MetaShark/Configuration/PluginConfiguration.cs
index 5cbe2a5..1f966e3 100644
--- a/Jellyfin.Plugin.MetaShark/Configuration/PluginConfiguration.cs
+++ b/Jellyfin.Plugin.MetaShark/Configuration/PluginConfiguration.cs
@@ -24,6 +24,10 @@ public class PluginConfiguration : BasePluginConfiguration
     /// </summary>
     public bool EnableDoubanAvoidRiskControl { get; set; } = false;
     /// <summary>
+    /// 豆瓣海报使用大图
+    /// </summary>
+    public bool EnableDoubanLargePoster { get; set; } = false;
+    /// <summary>
     /// 豆瓣背景图使用原图
     /// </summary>
     public bool EnableDoubanBackdropRaw { get; set; } = false;
diff --git a/Jellyfin.Plugin.MetaShark/Configuration/configPage.html b/Jellyfin.Plugin.MetaShark/Configuration/configPage.html
index 12497bf..e1ab37c 100644
--- a/Jellyfin.Plugin.MetaShark/Configuration/configPage.html
+++ b/Jellyfin.Plugin.MetaShark/Configuration/configPage.html
@@ -54,6 +54,13 @@
                                 可为空，填写jellyfin访问域名（有端口时要加上端口），只有使用了Nginx代理、Docker部署或启用了HTTPS，并且豆瓣图片无法显示时，才需要填写
                             </div>
                         </div>
+                        <div class="checkboxContainer checkboxContainer-withDescription">
+                            <label class="emby-checkbox-label" for="EnableDoubanLargePoster">
+                                <input id="EnableDoubanLargePoster" name="EnableDoubanLargePoster" type="checkbox"
+                                    is="emby-checkbox" />
+                                <span>海报使用大图</span>
+                            </label>
+                        </div>
                         <div class="checkboxContainer checkboxContainer-withDescription">
                             <label class="emby-checkbox-label" for="EnableDoubanBackdropRaw">
                                 <input id="EnableDoubanBackdropRaw" name="EnableDoubanBackdropRaw" type="checkbox"
@@ -165,6 +172,7 @@
                         document.querySelector('#DoubanCookies').value = config.DoubanCookies;
                         document.querySelector('#DoubanImageProxyBaseUrl').value = config.DoubanImageProxyBaseUrl;
                         document.querySelector('#EnableDoubanAvoidRiskControl').checked = config.EnableDoubanAvoidRiskControl;
+                        document.querySelector('#EnableDoubanLargePoster').checked = config.EnableDoubanLargePoster;
                         document.querySelector('#EnableDoubanBackdropRaw').checked = config.EnableDoubanBackdropRaw;
 
                         document.querySelector('#EnableTmdb').checked = config.EnableTmdb;
@@ -193,6 +201,7 @@
                         config.DoubanCookies = document.querySelector('#DoubanCookies').value;
                         config.DoubanImageProxyBaseUrl = document.querySelector('#DoubanImageProxyBaseUrl').value;
                         config.EnableDoubanAvoidRiskControl = document.querySelector('#EnableDoubanAvoidRiskControl').checked;
+                        config.EnableDoubanLargePoster = document.querySelector('#EnableDoubanLargePoster').checked;
                         config.EnableDoubanBackdropRaw = document.querySelector('#EnableDoubanBackdropRaw').checked;
 
                         config.EnableTmdb = document.querySelector('#EnableTmdb').checked;
@@ -221,6 +230,7 @@
                     changeProxyDisplay();
                 });
 
+
             function changeProxyDisplay() {
                 let proxyType = document.querySelector('#TmdbProxyType').value;
                 if (proxyType) {
diff --git a/Jellyfin.Plugin.MetaShark/Model/DoubanSubject.cs b/Jellyfin.Plugin.MetaShark/Model/DoubanSubject.cs
index 3fa2318..d8a8f84 100644
--- a/Jellyfin.Plugin.MetaShark/Model/DoubanSubject.cs
+++ b/Jellyfin.Plugin.MetaShark/Model/DoubanSubject.cs
@@ -93,6 +93,15 @@ namespace Jellyfin.Plugin.MetaShark.Model
             }
         }
 
+        [JsonIgnore]
+        public string ImgLarge
+        {
+            get
+            {
+                return this.Img.Replace("s_ratio_poster", "l");
+            }
+        }
+
         [JsonIgnore]
         public string[] Genres
         {
@@ -155,6 +164,16 @@ namespace Jellyfin.Plugin.MetaShark.Model
                 return null;
             }
         }
+
+        [JsonIgnore]
+        public string ImgMiddle
+        {
+            get
+            {
+                return this.Img.Replace("/raw/", "/m/").Replace("/s_ratio_poster/", "/m/");
+            }
+        }
+
     }
 
     public class DoubanPhoto
diff --git a/Jellyfin.Plugin.MetaShark/Providers/BaseProvider.cs b/Jellyfin.Plugin.MetaShark/Providers/BaseProvider.cs
index 0ecbe51..b4aa47d 100644
--- a/Jellyfin.Plugin.MetaShark/Providers/BaseProvider.cs
+++ b/Jellyfin.Plugin.MetaShark/Providers/BaseProvider.cs
@@ -471,53 +471,32 @@ namespace Jellyfin.Plugin.MetaShark.Providers
 
         protected string GetLocalProxyImageUrl(string url)
         {
-            var baseUrl = Plugin.Instance?.GetLocalApiBaseUrl();
-            if (!string.IsNullOrEmpty(config.DoubanImageProxyBaseUrl))
+            var baseUrl = Plugin.Instance?.GetLocalApiBaseUrl() ?? string.Empty;
+            if (!string.IsNullOrWhiteSpace(config.DoubanImageProxyBaseUrl))
             {
                 baseUrl = config.DoubanImageProxyBaseUrl.TrimEnd('/');
             }
-            if (!string.IsNullOrEmpty(baseUrl))
-            {
-                var encodedUrl = HttpUtility.UrlEncode(url);
-                return $"{baseUrl}/plugin/metashark/proxy/image/?url={encodedUrl}";
-            }
 
-            return this.GetProxyImageUrl(url);
+            var encodedUrl = HttpUtility.UrlEncode(url);
+            return $"{baseUrl}/plugin/metashark/proxy/image/?url={encodedUrl}";
         }
 
         private string GetBaseUrl()
         {
             // 配置优先
-            if (!string.IsNullOrEmpty(this.config.DoubanImageProxyBaseUrl))
+            if (!string.IsNullOrWhiteSpace(config.DoubanImageProxyBaseUrl))
             {
                 return this.config.DoubanImageProxyBaseUrl.TrimEnd('/');
             }
 
-            // http请求时，获取请求的host (nginx代理/docker中部署时，没配置透传host时，本方式会有问题)
+            // TODO：http请求时，获取请求的host (nginx代理/docker中部署时，没配置透传host时，本方式会有问题)
+            // 除自动扫描之外都会执行这里，修改图片功能图片是直接下载，不走插件图片代理处理函数，host拿不到就下载不了
             if (Plugin.Instance != null && this._httpContextAccessor.HttpContext != null)
             {
-                // 特殊处理下搜索请求，直接使用相对链接，可以减少使用nginx代理但不透传host的问题
-                var userAgent = this._httpContextAccessor.HttpContext.Request.Headers.UserAgent.ToString();
-                var fromWeb = userAgent.Contains("Chrome") || userAgent.Contains("Safari");
-                var fromItemSearch = this._httpContextAccessor.HttpContext.Request.Path.ToString().Contains("/RemoteSearch");
-                if (fromWeb && fromItemSearch)
-                {
-                    // 处理通过nginx反向代理后，url加了subpath访问的情况
-                    var subpath = string.Empty;
-                    var baseUrl = Plugin.Instance.GetApiBaseUrl(this._httpContextAccessor.HttpContext.Request);
-                    var uri = new UriBuilder(baseUrl);
-                    if (!string.IsNullOrEmpty(uri.Path) && uri.Path != "/")
-                    {
-                        subpath = "/" + uri.Path.Trim('/');
-                    }
-
-                    return subpath;
-                }
-
                 return Plugin.Instance.GetApiBaseUrl(this._httpContextAccessor.HttpContext.Request);
             }
 
-            // 自动扫描刷新时，直接使用本地地址
+            // 自动扫描刷新时，直接使用本地地址(127.0.0.1)
             return Plugin.Instance?.GetLocalApiBaseUrl() ?? string.Empty;
         }
 
@@ -573,7 +552,17 @@ namespace Jellyfin.Plugin.MetaShark.Providers
 
             return string.Empty;
         }
+        
 
+        protected string GetDoubanPoster(DoubanSubject subject)
+        {
+            if (string.IsNullOrEmpty(subject.Img)) {
+                return string.Empty;
+            }
+
+            var url = config.EnableDoubanLargePoster ? subject.ImgLarge : subject.ImgMiddle;
+            return this.GetProxyImageUrl(url);
+        }
 
         protected string GetOriginalFileName(ItemLookupInfo info)
         {
diff --git a/Jellyfin.Plugin.MetaShark/Providers/MovieImageProvider.cs b/Jellyfin.Plugin.MetaShark/Providers/MovieImageProvider.cs
index da672b5..d2ec287 100644
--- a/Jellyfin.Plugin.MetaShark/Providers/MovieImageProvider.cs
+++ b/Jellyfin.Plugin.MetaShark/Providers/MovieImageProvider.cs
@@ -48,7 +48,7 @@ namespace Jellyfin.Plugin.MetaShark.Providers
             if (metaSource != MetaSource.Tmdb && !string.IsNullOrEmpty(sid))
             {
                 var primary = await this._doubanApi.GetMovieAsync(sid, cancellationToken);
-                if (primary == null || string.IsNullOrEmpty(primary.ImgMiddle))
+                if (primary == null || string.IsNullOrEmpty(primary.Img))
                 {
                     return Enumerable.Empty<RemoteImageInfo>();
                 }
@@ -58,7 +58,7 @@ namespace Jellyfin.Plugin.MetaShark.Providers
                     new RemoteImageInfo
                     {
                         ProviderName = this.Name,
-                        Url = this.GetProxyImageUrl(primary.ImgMiddle),
+                        Url = this.GetDoubanPoster(primary),
                         Type = ImageType.Primary,
                     },
                 };
diff --git a/Jellyfin.Plugin.MetaShark/Providers/SeasonImageProvider.cs b/Jellyfin.Plugin.MetaShark/Providers/SeasonImageProvider.cs
index 7131f95..b817d32 100644
--- a/Jellyfin.Plugin.MetaShark/Providers/SeasonImageProvider.cs
+++ b/Jellyfin.Plugin.MetaShark/Providers/SeasonImageProvider.cs
@@ -60,7 +60,7 @@ namespace Jellyfin.Plugin.MetaShark.Providers
                     new RemoteImageInfo
                     {
                         ProviderName = primary.Name,
-                        Url = this.GetProxyImageUrl(primary.ImgMiddle),
+                        Url = this.GetDoubanPoster(primary),
                         Type = ImageType.Primary,
                     },
                 };
diff --git a/Jellyfin.Plugin.MetaShark/Providers/SeasonProvider.cs b/Jellyfin.Plugin.MetaShark/Providers/SeasonProvider.cs
index 4154c82..74e3528 100644
--- a/Jellyfin.Plugin.MetaShark/Providers/SeasonProvider.cs
+++ b/Jellyfin.Plugin.MetaShark/Providers/SeasonProvider.cs
@@ -93,6 +93,7 @@ namespace Jellyfin.Plugin.MetaShark.Providers
                             ProviderIds = new Dictionary<string, string> { { DoubanProviderId, c.Id } },
                         }));
 
+                        this.Log($"GetSeasonMetaData of douban [sid]: {seasonSid}");
                         return result;
                     }
                 }
diff --git a/Jellyfin.Plugin.MetaShark/Providers/SeriesImageProvider.cs b/Jellyfin.Plugin.MetaShark/Providers/SeriesImageProvider.cs
index cca62d2..eb2d7b3 100644
--- a/Jellyfin.Plugin.MetaShark/Providers/SeriesImageProvider.cs
+++ b/Jellyfin.Plugin.MetaShark/Providers/SeriesImageProvider.cs
@@ -48,7 +48,7 @@ namespace Jellyfin.Plugin.MetaShark.Providers
             if (metaSource != MetaSource.Tmdb && !string.IsNullOrEmpty(sid))
             {
                 var primary = await this._doubanApi.GetMovieAsync(sid, cancellationToken);
-                if (primary == null || string.IsNullOrEmpty(primary.ImgMiddle))
+                if (primary == null || string.IsNullOrEmpty(primary.Img))
                 {
                     return Enumerable.Empty<RemoteImageInfo>();
                 }
@@ -58,7 +58,7 @@ namespace Jellyfin.Plugin.MetaShark.Providers
                     new RemoteImageInfo
                     {
                         ProviderName = this.Name,
-                        Url = this.GetProxyImageUrl(primary.ImgMiddle),
+                        Url = this.GetDoubanPoster(primary),
                         Type = ImageType.Primary,
                     },
                 };
diff --git a/README.md b/README.md
index f667960..a3f0e8b 100644
--- a/README.md
+++ b/README.md
@@ -22,7 +22,7 @@ jellyfin电影元数据插件，影片信息只要从豆瓣获取，并由TheMov
 
 国外访问：https://github.com/cxfksword/jellyfin-plugin-metashark/releases/download/manifest/manifest.json
 
-> 如果无法访问，可以直接从 [Release](https://github.com/cxfksword/jellyfin-plugin-metashark/releases) 页面下载，并解压到 jellyfin 插件目录中使用
+> 如果都无法访问，可以直接从 [Release](https://github.com/cxfksword/jellyfin-plugin-metashark/releases) 页面下载，并解压到 jellyfin 插件目录中使用
 
 ## 如何使用
 
@@ -36,6 +36,8 @@ jellyfin电影元数据插件，影片信息只要从豆瓣获取，并由TheMov
 
 > 🚨假如需要刮削大量电影，请到插件配置中打开防封禁功能，避免频繁请求豆瓣导致被封IP（封IP需要等6小时左右才能恢复访问）
 
+> :fire:遇到图片显示不出来时，请到插件配置中配置jellyfin访问域名
+
 ## How to build
 
 1. Clone or download this repository