coolq-http-api icon indicating copy to clipboard operation
coolq-http-api copied to clipboard

反向 WS 连接的 Authorization 头应该使用 `Bearer` 而不是 `Token`(与正向 WS 和 HTTP 不一致)

Open ghost opened this issue 6 years ago • 6 comments

嗨~

在 v4.7.1 之后,http 和 ws 的请求都支持 TokenBearer 两种验证方案。然而当本项目作为 ws 客户端的时候发送的请求却是使用 Token 的。这个行为跟文档内注明的不符。

非常感谢!RCNB!

ghost avatar Jun 13 '19 03:06 ghost

诶,那应该是我疏忽了,回头看下,感谢提醒!

stdrc avatar Jun 13 '19 03:06 stdrc

我修了一下,有兴趣帮忙测试一下嘛,在 这里 下载 app.dllapp.json,放到 酷Q 的 dev/io.github.richardchien.coolqhttpapi 文件夹,并且在 酷Q 的应用管理右下角连续点击版本号,开启开发模式,就可以测试~

stdrc avatar Jun 13 '19 06:06 stdrc

这个会是个 breaking change 吧 ( 有些库好像默认 CQHTTP API 会使用 Token 诶 (

ghost avatar Jun 13 '19 15:06 ghost

对,如果发布的话应该会增加版本号到 4.11,不过也许也可以直接改文档说明一下这个问题

Zhangyuan Nie [email protected] 于 2019年6月13日周四 23:21写道:

这个会是个 breaking change 吧 ( 有些库好像默认 CQHTTP API 会使用 Token 诶 (

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/richardchien/coolq-http-api/issues/201?email_source=notifications&email_token=ABISDZ26ZPPTBZXSTID6RZLP2JQV3A5CNFSM4HXVRZK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXUBPPQ#issuecomment-501749694, or mute the thread https://github.com/notifications/unsubscribe-auth/ABISDZ5BROKWJ624IRH2G5TP2JQV3ANCNFSM4HXVRZKQ .

stdrc avatar Jun 13 '19 15:06 stdrc

Breaking change 还是到 v5.0.0 的时候改吧。4.x.x 还是保留当前行为,修改文档比较好~

ghost avatar Jun 13 '19 15:06 ghost

OK,我修改了文档,这个 issue 先不关吧,免得以后忘了

stdrc avatar Jun 14 '19 13:06 stdrc