使用 Apprise 推送通知#942
Open
LTSlw wants to merge 1 commit into
Open
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
|
挺好的,我要去抄一下这个库了( |
Contributor
|
我测了一下源仓库, telegram 的样式有点问题, bark的icon也有点要适配, 不过最大的问题还是这个用的 requests, 后面应该是要转httpx的 |
Author
|
还有一个问题和 requests 也有关系,Apprise 的每个通知模块各自独立调用 requests,导致没有办法遵守 proxy 设置,只能靠环境变量设置通知的 proxy。虽然现在的实现让通知和 rss 什么的都共用相同 header 也未必合理,但设置了 proxy 但通知不受控制显然更不合逻辑,下下策也应该额外说明一下 proxy 的工作范围。 我浏览了一下 pr,通知是一个比较热门的话题。相关:#513,#729,#747,#793,#815,#867 通知服务各种各样,我觉得把配置放在单个 url 里是正确的思路,哪怕最终没办法用 Apprise,也比现在改通知就要前后端一起改好的多。 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
使用 Apprise 推送通知,把推送通知的配置精简到一个 url,精简代码的同时一次性支持数十种通知服务。
注意,本 pr 存在一定破坏性,主要体现在配置文件的改动,如果需要本人可配合做出兼容性上的调整。
支持的通知服务:https://github.com/caronc/apprise/wiki