Skip to content
This repository has been archived by the owner on Aug 2, 2020. It is now read-only.

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

Open
ghost opened this issue Jun 13, 2019 · 6 comments
Labels

Comments

@ghost
Copy link

ghost commented Jun 13, 2019

嗨~

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

非常感谢!RCNB!

@stdrc
Copy link
Member

stdrc commented Jun 13, 2019 via email

@stdrc
Copy link
Member

stdrc commented Jun 13, 2019

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

@stdrc stdrc added the Bug label Jun 13, 2019
@ghost
Copy link
Author

ghost commented Jun 13, 2019

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

@stdrc
Copy link
Member

stdrc commented Jun 13, 2019 via email

@ghost
Copy link
Author

ghost commented Jun 13, 2019

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

@stdrc
Copy link
Member

stdrc commented Jun 14, 2019

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

@stdrc stdrc changed the title Documentation does not match ws behaviour 反向 WS 连接的 Authorization 头应该使用 Bearer 而不是 Token(与正向 WS 和 HTTP 不一致) Jun 14, 2019
@stdrc stdrc added this to the next milestone Dec 24, 2019
@stdrc stdrc removed this from the next milestone Jan 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

1 participant