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

一些问题 #16

Open
YuHangZM opened this issue Sep 7, 2017 · 2 comments
Open

一些问题 #16

YuHangZM opened this issue Sep 7, 2017 · 2 comments

Comments

@YuHangZM
Copy link

YuHangZM commented Sep 7, 2017

  1. pom.xml 中没有标明 都是什么包,或者说不友好
    2.为什么要和tomcat绑在一起,jdk也设定好了,无语
    3.shiro中自己定义了过滤器,但是你没有正确的引用kickout,看看bean的名称就知道了。
@baichengzhou
Copy link
Owner

1.pom.xml 我个人觉得不用每个包独立说明是什么包,因为它本身有groupId和artifactId,没什么隐藏的东西,另外我每个包尽量分组放在一起,然后说明了。特殊的包,我也说明了。
2.为什么要和Tomcat绑在一起?并没有,或者换句话说,我并没有特意和Tomcat绑在一起。只是我在Tomcat环境下开发的,为什么登录页强调了用Tomcat去跑,因为好多同学用Jetty去跑的时候,会运行不起来(其实是因为要稍作修改一些地方),其次JDK,你需要看readme文件里的文档链接:http://www.sojson.com/shiro 里面描述的很清楚,需要用JDK7或以上。我不知道你为什么无语,我并没有设定好必须用哪个JDK。
3.Shiro中自己如果自定义了Kickout类似功能的Filter,请你分支修改,请求合并,我来合并即可。

4.请您态度端正一下。不要有语气助词去说问题。
你可以下载01版本,可能会更糟糕,因为开始并没有考虑开源,或者说对外开放,只是对我本网站内部交流群使用,但是我发现外部也有好多人用,因为托管到了(github),为此我优化了一版本,并且把原来的02版本也删掉重建,优化+注释。我并没有觉得这个项目有多好,但我觉得我这个在这个大环境中,还算是良心作品,不管从文档还是从更新还是从注释,还是从群里交流问题解决问题。我都是有空就回复和探讨,我也一直在交流群里强调这只是一个Demo,供初学者入门的Demo,所以里面好多方式在编写的时候,并不是和实际场景相符,而是考虑到能展现出来的更多常用的功能,所以Filter都是自定义的,等等。
你有更优雅的方式,更完美的姿态,请您分支,写完后,请求合并,我如果觉得OK,我会合并进去的。但请不要改变我的初衷代码!

@YuHangZM
Copy link
Author

YuHangZM commented Sep 7, 2017

淡定,淡定。

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

No branches or pull requests

2 participants