Skip to content

推送接口重构,抽离Pusher接口#303

Closed
qcgzxw wants to merge 1 commit intoFinb:masterfrom
qcgzxw:dev
Closed

推送接口重构,抽离Pusher接口#303
qcgzxw wants to merge 1 commit intoFinb:masterfrom
qcgzxw:dev

Conversation

@qcgzxw
Copy link
Copy Markdown

@qcgzxw qcgzxw commented Dec 25, 2024

  • 参数获取方式重构,更简洁
  • api v2暂未实现 device_keys的,兼容device_key 使用逗号分隔多个key
  • 抽离pusher接口,后续兼容多平台推送

@qcgzxw
Copy link
Copy Markdown
Author

qcgzxw commented Dec 25, 2024

想要实现其他平台的推送,抽离Pusher接口

@qcgzxw qcgzxw changed the title 推送接口重构,抽离Pusher接口 [wip]推送接口重构,抽离Pusher接口 Dec 25, 2024
@qcgzxw qcgzxw changed the title [wip]推送接口重构,抽离Pusher接口 推送接口重构,抽离Pusher接口 Dec 26, 2024
@Finb
Copy link
Copy Markdown
Owner

Finb commented Jan 7, 2025

哈喽,感谢PR,但有一些问题。

Bark 的参数因为用户的调用环境很多,所以支持在 url 路径、Query Parameters、请求体 body 中传递参数。其中 body 支持key-value 或 json 格式。 不管是GET POST 都会去这些位置取值。

我发现这个PR改动会导致很多之前能正常请求的参数传递方式,会变的不可用,但一般需要对这些参数传递方式进行兼容(因为无法通知到用户更新到新的传递方式,也不能突然让用户不能推送)

例如这个pr 忽略了body中的参数,除非使用了 /push 。
加密推送实际发送时只有一个 ciphertext 参数,没有title body 之类的,会被 Message.Check() 阻止,并且如果没传title body ,需要带上一个不为空的字符串(例如我用的是 Empty Message),否则会被APNs丢弃

另外考虑到稳定性,bark-server也不想对代码进行大改,所以暂时关闭这个pr,感谢!

@Finb Finb closed this Jan 7, 2025
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