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

fix a little bug #7

Closed
wants to merge 5 commits into from
Closed

fix a little bug #7

wants to merge 5 commits into from

Conversation

chengchaos
Copy link

when the request url like "http://127.0.0.1:8080/webjars/jquery/" then the servlet throw "java.lang.ArrayIndexOutOfBoundsException: 5" exception.

@jamesward
Copy link
Member

Can you add a test for this that is failing before the patch?

@chengchaos
Copy link
Author

I add the a little test by JUnit4 , my gmail can't send any email .... cry ...

@chengchaos chengchaos closed this Dec 16, 2015
@jamesward
Copy link
Member

@chengchaos What happened?

@chengchaos
Copy link
Author

I don't confident which answer is the right one
when use the incorrect url then throw exception or return 404 not found.

@jamesward
Copy link
Member

Ok if I close this in favor of #12?

@chengchaos chengchaos closed this Feb 15, 2016
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

Successfully merging this pull request may close these issues.

None yet

2 participants