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

Adapter: WXMP #3218

Open
YangRucheng opened this issue Dec 29, 2024 · 11 comments · May be fixed by #3219
Open

Adapter: WXMP #3218

YangRucheng opened this issue Dec 29, 2024 · 11 comments · May be fixed by #3219
Labels
Adapter Publish Adapter Publish Publish things to store

Comments

@YangRucheng
Copy link

YangRucheng commented Dec 29, 2024

适配器名称

WXMP

适配器描述

微信公众平台 客服适配器

PyPI 项目名

nonebot-adapter-wxmp

适配器 import 包名

nonebot.adapters.wxmp

适配器项目仓库/主页链接

https://github.com/YangRucheng/nonebot-adapter-wxmp

标签

[{"label":"微信公众平台","color":"#843dbc"}]

@YangRucheng YangRucheng added Adapter Publish Adapter Publish Publish things to store labels Dec 29, 2024
@noneflow
Copy link
Contributor

noneflow bot commented Dec 29, 2024

📃 商店发布检查结果

Adapter: WXMP

✅ 所有测试通过,一切准备就绪!

详情
  • ✅ 项目 主页 返回状态码 200。
  • ✅ 项目 nonebot-adapter-wxmp 已发布至 PyPI。
  • ✅ 标签: 微信公众平台-#843dbc。
  • ✅ 版本号: 0.1.0。
  • ✅ 发布时间:2025-01-13 20:10:11 CST。

  • 💡 如需修改信息,请直接修改 issue,机器人会自动更新检查结果。
    💡 当插件加载测试失败时,请发布新版本后勾选插件测试勾选框重新运行插件测试。

    ♻️ 评论已更新至最新检查结果

    💪 Powered by NoneFlow

    @noneflow noneflow bot changed the title Adapter: 微信公众平台客服 Adapter: WXMP Dec 29, 2024
    noneflow bot pushed a commit that referenced this issue Dec 29, 2024
    @noneflow noneflow bot linked a pull request Dec 29, 2024 that will close this issue
    @RF-Tar-Railt
    Copy link
    Member

    @YangRucheng
    Copy link
    Author

    YangRucheng commented Dec 29, 2024

    大哥 https://github.com/YangRucheng/nonebot-adapter-wxmp/blob/b744379a50af30efbdf2a9fb5b49f6934221db60/nonebot/adapters/wxmp/bot.py#L84-L105 你从哪学来的重写bot.call_api

    抱歉,现在我把它移到 adapter._call_api 了。

    @yanyongyu
    Copy link
    Member

    适配器需要在 shutdown 的时候 cancel 运行中 task,你可以参考别的适配器。

    noneflow bot pushed a commit that referenced this issue Dec 31, 2024
    @YangRucheng
    Copy link
    Author

    适配器需要在 shutdown 的时候 cancel 运行中 task,你可以参考别的适配器。

    好的,已经加上了

    @yanyongyu
    Copy link
    Member

    yanyongyu commented Jan 1, 2025

    setup 不要写在 on_startup 里。另外所有的适配器接口调用都要走 call api,使用 nb driver 提供的 http client,而不是使用 httpx 直接请求,这样会绕过 nb 的 hook,你也可以参考 QQ 适配器的写法。

    noneflow bot pushed a commit that referenced this issue Jan 1, 2025
    @YangRucheng
    Copy link
    Author

    setup 不要写在 on_startup 里。另外所有的适配器接口调用都要走 call api,使用 nb driver 提供的 http client,而不是使用 httpx 直接请求,这样会绕过 nb 的 hook,你也可以参考 QQ 适配器的写法。

    已修改,具体:

    • on_startup 中的内容移到 setup。(之前参考 飞书适配器 写在 on_startup,之前完全没想到是因为飞书驱动器要等 startup 后获取 bot 信息)。
    • 除获取 access_token 的以外的接口都使用 call_api 了(因为 call_api 需要 access_token)。
    • 移除了 httpx 依赖,使用 nb 的 http client。

    每次都写出新问题,给大佬添麻烦了🤝

    noneflow bot pushed a commit that referenced this issue Jan 1, 2025
    @yanyongyu
    Copy link
    Member

    你的 exception 没有继承,raise 的内容也没有信息。verify 的安全性有问题,你应该对不正常的请求返回400+,signature 比较应该使用 secrets.compare_digest

    noneflow bot pushed a commit that referenced this issue Jan 4, 2025
    @YangRucheng
    Copy link
    Author

    由于我之前没有看到 RFC: 适配器规范,只阅读了 文档,导致存在很多问题。

    我会在近俩天修改,确保符合 RFC 的规范要求。

    noneflow bot pushed a commit that referenced this issue Jan 14, 2025
    @yanyongyu
    Copy link
    Member

    @YangRucheng 是否修改完成?

    @YangRucheng
    Copy link
    Author

    修改完成了,劳烦ddl帮忙看一下

    noneflow bot pushed a commit that referenced this issue Jan 14, 2025
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    Adapter Publish Adapter Publish Publish things to store
    Development

    Successfully merging a pull request may close this issue.

    3 participants