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

feat: support jwch user login #22

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

feat: support jwch user login #22

wants to merge 3 commits into from

Conversation

ozline
Copy link
Member

@ozline ozline commented Dec 31, 2024

这个 pr 需要继续由 @klxiaoniu 协助完成安卓端 native-request 的 302 禁用设置

同时由 @renbaoshuo 合入代码前将其修改成合适的工程代码

ozline and others added 2 commits December 31, 2024 13:24
Different from Alamofire, okhttp lowercases headers automatically with HTTP 2, so disable it as a workaround.
@klxiaoniu
Copy link
Contributor

HTTP规范对headers大小写不敏感,而HTTP2要求必须为小写,因此okhttp在HTTP2情况下会对响应的headers的key做转小写处理。square/okhttp#6670

与之不同的是,Alamofire不会有这样的处理。这里采用为okhttp禁用HTTP2,只使用1.1的方式来达到表现上的一致性。

@ozline
Copy link
Member Author

ozline commented Jan 1, 2025

符合预期,后端的库也是禁用 HTTP2.0 和 302 就可以正常实现

现在可以由 @renbaoshuo 来工程化了,正好我来学习一下

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.

3 participants