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.

Sign up for GitHub

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

Jump to bottom

没搞明白是怎么样给okhttp设置请求头的？ #42

Closed
ghost opened this issue Apr 25, 2018 · 3 comments
Closed

没搞明白是怎么样给okhttp设置请求头的？ #42

ghost opened this issue Apr 25, 2018 · 3 comments
Labels
bug
Milestone
1.0.2

Comments

@ghost
Copy link

ghost commented Apr 25, 2018

.setHeaderMapFields(map)好像没看到有把设置的请求头带过去
Jacksgong added a commit that referenced this issue Apr 26, 2018
@Jacksgong
tests: add unit test for setHeaderrMapFields on DownloadTask
0b4b3bb 
refs #42
@Jacksgong
Copy link
Collaborator

其实，每个分块的的连接上是有带上的，具体参看代码HeaderInterceptor#interceptConnect，不过在试探连接(BreakpointRemoteCheck)中确实是没有带上，这个确实不符合预期，我加下。

@Jacksgong Jacksgong added the bug label Apr 26, 2018
@Jacksgong Jacksgong added this to the 1.0.2 milestone Apr 26, 2018
Jacksgong added a commit that referenced this issue Apr 26, 2018
@Jacksgong
refactor: wrap addUserHeaderFields on the Util and inspect whether th…
4146c38 
…e userHeaderFields is valid before add to it

refs #42
@Jacksgong
Copy link
Collaborator

已经在1.0.2-SNAPSHOT上修复了该问题。

@ghost
Copy link
Author

ghost commented Apr 27, 2018 •
edited by ghost
Loading

哦哦，原来如此，我测试的是1.0.1版，我从抓包上看到没有带请求头过去

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
1.0.2
Development

No branches or pull requests
1 participant
@Jacksgong