[bug] DELETE 请求被意外添加了 Transfer-Encoding 头

[1574242600]

环境

whistle: v2.6.16

描述

所有 DELETE HTTP 1.1 请求 (代理，Composer)，无论此前是否有 Transfer-Encoding 头，都被添加了 Transfer-Encoding 头（没有显示在 Inspectors）以及分块传输编码。

示例

原始的请求:

whistle发出的请求:

复现

使用任意方式通过 whistle 发送 DELETE HTTP 1.1 请求

可能相关的代码

whistle/lib/inspectors/res.js

Lines 517 to 543 in 0ad8e9e

	var isDelete = options.method === 'DELETE' && optHeaders['content-length'] == null;
	if (isDelete) {
	delete optHeaders['transfer-encoding'];
	}
	if (proxyUrl) {
	util.setClientId(optHeaders, req.enable, req.disable, req.clientIp, isInternalProxy);
	} else {
	var clientId = optHeaders[config.CLIENT_ID_HEADER];
	if (clientId) {
	if (!util.isEnable(req, 'keepClientId')) {
	util.removeClientId(optHeaders);
	}
	req.setClientId && req.setClientId(clientId);
	}
	}
	if (req.isH2 && (headers[config.HTTPS_FIELD] || options.isPlugin)) {
	headers[config.ALPN_PROTOCOL_HEADER] = 'h2';
	}
	options.headers = optHeaders = formatHeaders(optHeaders, req.rawHeaderNames);
	delete headers[config.WEBUI_HEAD];
	if (headers[config.HTTPS_FIELD]) {
	delete headers[config.HTTPS_FIELD];
	delete headers[config.ALPN_PROTOCOL_HEADER];
	}
	if (isDelete) {
	optHeaders['Transfer-Encoding'] = 'chunked';
	}

[avwo]

你的问题会导致什么后果？DELETE 方法比较特殊，标准文档里面没有说明能不能携带 body，有些服务支持有些不支持，如 Node 就不支持 DELETE 方法带请求内容，上面的实现是为了规避此问题。

[1574242600]

nginx 的 webdav 模块会检查 DELETE 请求的 Transfer-Encoding
https://github.com/nginx/nginx/blob/ecbe06b9fee57207d84be9ac39d1aa10b2d0fbd8/src/http/modules/ngx_http_dav_module.c#L315-L319
导致直接返回415

主要是修改了原始请求而 Inspectors 又没显示，这非常坑人
还有没有其它会修改原始请求的地方？