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

qiniu api v7.6.0; add debug info for http.DefaultClient #288

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

Conversation

sxci
Copy link
Collaborator

@sxci sxci commented Oct 28, 2020

变更背景描述

升级到 qiniu api v7.6.0 ;
qiniu/api.v7 kodo 相关 http 使用了 自定义的 Client 支持打印 http 请求头、请求体;
qiniu/api.v7 cdn 等 api ,使用了 http.DefaultClient ,也添加打印 http 请求头、请求体 能力

qshell -d 打印请求头、响应头
qshell -D 打印请求头、请求体、响应头、响应体

jira issue链接

https://jira.qiniu.io/browse/TS-1030

主要变更点

  • []fix bug
  • [v]new feature
  • []不兼容变更

Checklist

  • [v] 已自测
  • [] 已更新Readme

@qiniu-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bachue, sxci

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@bachue bachue force-pushed the master branch 4 times, most recently from 180cb0f to 7d38ae8 Compare June 27, 2023 12:18
@codecov
Copy link

codecov bot commented Jun 27, 2023

Codecov Report

Merging #288 (ad12113) into master (2ab8bc4) will decrease coverage by 0.41%.
The diff coverage is 0.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #288      +/-   ##
=========================================
- Coverage    6.76%   6.36%   -0.41%     
=========================================
  Files          40      40              
  Lines        4876    4492     -384     
=========================================
- Hits          330     286      -44     
+ Misses       4540    4201     -339     
+ Partials        6       5       -1     
Impacted Files Coverage Δ
cmd/root.go 8.69% <0.00%> (-7.70%) ⬇️

... and 39 files with indirect coverage changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants