bug report - webx3 在构造rundata parameters的时候 如果发现是文件上传就会忽略 请求url上的参数

[loyrevel]

---ParameterParserImpl 98 行开始

if (requestContext.isAutoUpload() && upload != null) {
// 如果是multipart/*请求，则调用upload service来解析。
isMultipart = upload.isMultipartContent(wrappedRequest);

        if (isMultipart) {
            try {
                parseUpload();
            } catch (UploadSizeLimitExceededException e) {
                add(ParserRequestContext.UPLOAD_FAILED, Boolean.TRUE);
                add(ParserRequestContext.UPLOAD_SIZE_LIMIT_EXCEEDED, Boolean.TRUE);
                log.warn("File upload exceeds the size limit", e);
            } catch (UploadException e) {
                add(ParserRequestContext.UPLOAD_FAILED, Boolean.TRUE);
                log.warn("Upload failed", e);
            }
        }
    }

    // 从request中取参数
    if (!isMultipart) {
        String method = wrappedRequest.getMethod();

[superbaobao]

这个行为是符合规范的吧？为什么不把参数放在form body中？

发自我的 iPhone

在 2015年7月5日，22:30，萧山 [email protected] 写道：

---ParameterParserImpl 98 行开始

if (requestContext.isAutoUpload() && upload != null) {
// 如果是multipart/*请求，则调用upload service来解析。
isMultipart = upload.isMultipartContent(wrappedRequest);

    if (isMultipart) {
        try {
            parseUpload();
        } catch (UploadSizeLimitExceededException e) {
            add(ParserRequestContext.UPLOAD_FAILED, Boolean.TRUE);
            add(ParserRequestContext.UPLOAD_SIZE_LIMIT_EXCEEDED, Boolean.TRUE);
            log.warn("File upload exceeds the size limit", e);
        } catch (UploadException e) {
            add(ParserRequestContext.UPLOAD_FAILED, Boolean.TRUE);
            log.warn("Upload failed", e);
        }
    }
}

// 从request中取参数
if (!isMultipart) {
    String method = wrappedRequest.getMethod();

—
Reply to this email directly or view it on GitHub.

[loyrevel]

逻辑上是可以把参数放子啊form body中；
但是webx 里面的rundata 和原生的servlet里面 request 的行为是不一致的;
会带来编码时的理解成本和前后端的沟通成本；
前端认为这个是符合请求规范的

[superbaobao]

是一致的，你可以去查servlet spec，post请求是不应该包含url参数的，否则在合并参数时会造成不确定性。

发自我的 iPhone

在 2015年7月6日，05:39，萧山 [email protected] 写道：

逻辑上是可以把参数放子啊form body中；
但是webx 里面的rundata 和原生的servlet里面 request 的行为是不一致的;
会带来编码时的理解成本和前后端的沟通成本；
前端认为这个是符合请求规范的

—
Reply to this email directly or view it on GitHub.