若依Plus 的XSSFilter 的bug

现象

当我A字段输入 '【】、=-·!@#~¥%......&*()------+{}|":》?《,./;'\][=-`~!@#$%^&*()_+{}|":>????????????????<

B字段输入'【】、=-·!@#~¥%......&*()------+{}|":》?《,./;'\][=-`~!@#$%^&*()_+{}|":>????????????????/>

发现A字段被截断,B字段为空

原因

原因在于XSSFilter中对PUT和POST请求XSS过滤,重新包装了请求头,防止XML攻击,

弊端在于,他传入的content是整段json,也就是A字段和B字段合并起来产生了HTML匹配,所以修改方案是,单个字段校验

java 复制代码
   public static String cleanHtmlTag(String content) {
        return content.replaceAll("(<[^<]*?>)|(<[\\s]*?/[^<]*?>)|(<[^<]*?/[\\s]*?>)", "");
    }

原XssHttpServletRequestWrapper过滤器的getInputStream

java 复制代码
    @Override
    public ServletInputStream getInputStream() throws IOException {
        // 非json类型,直接返回
        if (!isJsonRequest()) {
            return super.getInputStream();
        }

        // 为空,直接返回
        String json = StrUtil.str(IoUtil.readBytes(super.getInputStream(), false), StandardCharsets.UTF_8);
        if (StringUtils.isEmpty(json)) {
            return super.getInputStream();
        }

        // xss过滤
        json = HtmlUtil.cleanHtmlTag(json).trim();
        byte[] jsonBytes = json.getBytes(StandardCharsets.UTF_8);
        final ByteArrayInputStream bis = IoUtil.toStream(jsonBytes);
        return new ServletInputStream() {
            @Override
            public boolean isFinished() {
                return true;
            }

            @Override
            public boolean isReady() {
                return true;
            }

            @Override
            public int available() throws IOException {
                return jsonBytes.length;
            }

            @Override
            public void setReadListener(ReadListener readListener) {
            }

            @Override
            public int read() throws IOException {
                return bis.read();
            }
        };
    }

修改后的

java 复制代码
 @Override
    public ServletInputStream getInputStream() throws IOException {
        // 非 JSON 类型,直接返回
        if (!isJsonRequest()) {
            return super.getInputStream();
        }

        // 读取原始 JSON
        String json = StrUtil.str(IoUtil.readBytes(super.getInputStream(), false), StandardCharsets.UTF_8);
        if (StringUtils.isEmpty(json)) {
            return super.getInputStream();
        }

        // ===== 新逻辑:针对 JSON 的每个 value 做 XSS 清理 =====
        ObjectMapper mapper = new ObjectMapper();
        Map<String, Object> map;
        try {
            map = mapper.readValue(json, new TypeReference<Map<String, Object>>() {});
        } catch (Exception e) {
            // JSON 解析失败,保守返回原始流
            return super.getInputStream();
        }

        // 遍历 Map,把每个 String 类型的 value 做清理
        for (Map.Entry<String, Object> entry : map.entrySet()) {
            if (entry.getValue() instanceof String) {
                entry.setValue(HtmlUtil.cleanHtmlTag((String) entry.getValue()).trim());
            }
        }

        // 序列化回 JSON
        byte[] jsonBytes = mapper.writeValueAsBytes(map);
        final ByteArrayInputStream bis = new ByteArrayInputStream(jsonBytes);

        // 返回新的 ServletInputStream
        return new ServletInputStream() {
            @Override
            public boolean isFinished() {
                return true;
            }

            @Override
            public boolean isReady() {
                return true;
            }

            @Override
            public void setReadListener(ReadListener readListener) {}

            @Override
            public int available() throws IOException {
                return jsonBytes.length;
            }

            @Override
            public int read() throws IOException {
                return bis.read();
            }
        };
    }
相关推荐
小此方2 天前
「插曲:Git」企业规范篇:DevOps开发模型、Git Flow五类分支设计与测试/预发布/生产环境Bug修复及Hotfix紧急发布流程
git·bug·devops
中文英文-我选中文4 天前
FreeRTOS 玄学Bug:调换任务创建顺序,程序时而正常、时而死机(栈溢出内存踩踏深度复盘)
bug
Keven_114 天前
BUG:UVA116
算法·bug·图论
鬼手点金4 天前
opencode-隐私优先配置
服务器·前端·javascript·bug·openclaw
itmigrate5 天前
异步请求竞态,藏在页面渲染里的隐形bug
bug
今夕资源网6 天前
OmniVoice 今夕整合包 本地语音TTS断网可用 修复了一些BUG 优化生成速度 极速秒生成
bug·语音克隆·omnivoice·断网可用·本地语音tts·语音tts
烧烧的酒0.o11 天前
Codex桌面版更新后不能使用最新模型问题-排查修复总结
chatgpt·bug
sg_knight12 天前
ZCode Bug 定位实战:把报错丢给 ZCode,它是怎么修的
llm·bug·agent·ai编程·glm·智谱·zcode
轩轶子13 天前
Python CLI Todo:一次数据结构错误引发的 Bug
数据结构·python·bug
鬼手点金13 天前
Claude Code示范案例-修复 Bug 工作流
java·服务器·前端·javascript·bug·openclaw