(BUG,0)com.jeecms.core.web.AccessControlFilter.getUrl - JEECMS官方论坛|开源java cms,jsp cms,jsp bbs

主题 : (BUG,0)com.jeecms.core.web.AccessControlFilter.getUrl

pauli

级别: 童生

UID: 1360
积分:47 加为好友
威望: 0 精华: 0
主题:4 回复:30
注册时间:2009-12-15
在线时长:0

1# 发表于:2010-04-15 16:01:51 IP:203.5.*.*

显示全部只看该作者

版本: jeecms-2.4.2-final
Java类: com.jeecms.core.web.AccessControlFilter
方法: getUrl
类别: 逻辑错误

private String getUrl(HttpServletRequest req) { String url = req.getRequestURI(); String context = req.getContextPath(); if (url.indexOf(".") != -1) { return url.substring(context.length(), url.indexOf(".")); } else if (url.indexOf("?") != -1) { return url.substring(context.length(), url.indexOf("?")); } else { return url.substring(context.length()); } }

这里是不是有个BUG?
如, 请求为 http://www.xyz.com/path1/path2/filename3.html
本意是得到 filename3
如果请求改为 http://www.xyz.com/path1/path2/file.name3.html
那么只能得到 file
这是不对的, 应该得到 file.name3
所以说, url.indexOf(".")和url.indexOf("?")
都应该改为: url.lastIndexOf(".")和url.lastIndexOf("?")

修改后的代码

考虑通用性, 也可以提取为静态方法(utility)

片面的考虑, 仅供参考, 请指正

使用道具
顶端

1 共1页