Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Firefox(version:129.0.2 AMD64) biliscope(version:0.6.1) 视频评论区显示IP地址有误 #224

Open
kimi996z opened this issue Sep 3, 2024 · 10 comments

Comments

@kimi996z
Copy link

kimi996z commented Sep 3, 2024

详情见图片
Uploading feedback_bili_plugin.png…

@kimi996z kimi996z changed the title Firefox(version129.0.2 AMD64) biliscope(version:0.6.1) 显示IP地址有误 Firefox(version:129.0.2 AMD64) biliscope(version:0.6.1) 显示IP地址有误 Sep 3, 2024
@kimi996z kimi996z changed the title Firefox(version:129.0.2 AMD64) biliscope(version:0.6.1) 显示IP地址有误 Firefox(version:129.0.2 AMD64) biliscope(version:0.6.1) 视频评论区显示IP地址有误 Sep 3, 2024
@kimi996z
Copy link
Author

kimi996z commented Sep 3, 2024

图片没上传成功,我再上传一次
feedback_bili_plugin

@kimi996z
Copy link
Author

kimi996z commented Sep 3, 2024

视频地址:https://www.bilibili.com/video/BV1FpHeekE7x/

@kimi996z
Copy link
Author

kimi996z commented Sep 3, 2024

现在IP又正常显示了。。。

@gaogaotiantian
Copy link
Owner

hmm,这个我还真不是非常确定具体可能是什么原因导致的……

@F-park
Copy link
Contributor

F-park commented Sep 4, 2024

这里的数据直接从评论的 __data 属性读取的,按理来说不会出错的吧。

@kimi996z
Copy link
Author

我知道怎么复现了,当评论排序改变后,IP似乎不会跟着改变。
案例链接[(https://www.bilibili.com/video/BV1xEtpeTE85/?#reply242672609953)]
Snipaste_2024-09-21_16-40-12
Snipaste_2024-09-21_16-41-11

@F-park
Copy link
Contributor

F-park commented Sep 22, 2024

复现了。就是 点击查看 之后原先的 ip 不会跟着刷新

@gaogaotiantian
Copy link
Owner

复现了。就是 点击查看 之后原先的 ip 不会跟着刷新

哦,我们是不是虽然hook了observer,但是现在的代码是只要有ip就直接不管了。

@F-park
Copy link
Contributor

F-park commented Oct 7, 2024

重新试了一下,现在又不会出问题了,估计是b站改了点击查看的逻辑。

@F-park
Copy link
Contributor

F-park commented Oct 7, 2024

@kimi996z 你是否还能复现这一问题,你上面发的案例视频被删掉了。

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

No branches or pull requests

3 participants