search

caryyu / jellyfin-plugin-opendouban

Douban metadata provider for Jellyfin
188 stars 15 forks source link

get images with referer header #33

Open xcz1997 opened 2 years ago

xcz1997 commented 2 years ago

用于下载open-server高清image url

caryyu commented 2 years ago

@xcz1997 我先关闭该 PR 了，有需要再开。

xcz1997 commented 2 years ago

@xcz1997 我先关闭该 PR 了，有需要再开。

@caryyu 客户端的这个PR似乎是没有任何影响的，而且在版本依赖的角度上来讲也应该先上，还是说针对高清图片的下载需求没有计划？

caryyu commented 2 years ago

@xcz1997 该变更主要是添加了 Referer ，难道高清图与非高清图是按照 Referer 进行区分的嘛？你能发我一个图片外链示例我这边测测嘛？

xcz1997 commented 2 years ago

@xcz1997 该变更主要是添加了 Referer ，难道高清图与非高清图是按照 Referer 进行区分的嘛？你能发我一个图片外链示例我这边测测嘛？

@caryyu https://movie.douban.com/subject/2124724/ 中，高清图外链为https://img2.doubanio.com/view/photo/raw/public/p2568025412.jpg 没有referer的时候，get会403 高清图的外链是根据普通封面图的url主动拼接的，原html中没有直接渲染

caryyu commented 2 years ago

@xcz1997 所以我的理解是这个 PR 的目的是修复图片无法正常下载的问题而不是高清图片？因为目前没有任何拼接的成分在代码中。

xcz1997 commented 2 years ago

@xcz1997 所以我的理解是这个 PR 的目的是修复图片无法正常下载的问题而不是高清图片？因为目前没有任何拼接的成分在代码中。

@caryyu 拼接的部分在第一次的PR中，是在server端解析的时候做的拼接，PR已经被我关了，所以你的理解其实也没有错。

caryyu commented 2 years ago

@xcz1997 那就奇怪了，就算没有该 Referer 的设置，之前也是可以下载到图片的，你那边现在使用该插件下载不了图片了吗？

xcz1997 commented 2 years ago

@xcz1997 那就奇怪了，就算没有该 Referer 的设置，之前也是可以下载到图片的，你那边下载不了吗？

@caryyu 能下载啊，我只是提前将客户端的代码先提交了，client端加不加Referer都对当前server端没有影响，但是如果server端后续对接了高清的功能的话，那对client端是强依赖的，考虑到不是所有人都会及时更新自己部署的client或者server，所以我先提交了client的代码让他优先server端运行几个版本

caryyu commented 2 years ago

@xcz1997 好的，我的理解是如果要加入高清图片功能则 Server 至少要先支持才行，否则该 PR 作用不大；我们先处理 Server 端。