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

ServletContextPath can start with servletPath, it should be taken into consideration #289

Merged
merged 4 commits into from Feb 3, 2016

Conversation

Projects
None yet
2 participants
@petromykhailysyn
Copy link
Contributor

petromykhailysyn commented Jan 22, 2016

… root) we should take it into consideration while determinating server base.

petromykhailysyn added some commits Jan 22, 2016

petromykhailysyn
If servletContextPath is not root (application is deployed not to the…
… root) we should take it into consideration while determinating server base.
petromykhailysyn
ServletContextPath can start with servletPath, now it's taking into c…
…onsideration while determinate contextIndex.
petromykhailysyn
Revert "If servletContextPath is not root (application is deployed no…
…t to the root) we should take it into consideration while determinating server base."

This reverts commit 0e3bb5a.

@petromykhailysyn petromykhailysyn changed the title If servletContextPath is not root (application is deployed not to the… ServletContextPath can start with servletPath, it should be taken into consideration Jan 26, 2016

@jamesagnew

This comment has been minimized.

Copy link
Owner

jamesagnew commented Feb 3, 2016

Thanks Petro! Looks great, I'll merge this now.

jamesagnew added a commit that referenced this pull request Feb 3, 2016

Merge pull request #289 from petromykhailysyn/master
ServletContextPath can start with servletPath, it should be taken into consideration

@jamesagnew jamesagnew merged commit def7fc9 into jamesagnew:master Feb 3, 2016

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls First build on master at 84.593%
Details

jamesagnew added a commit that referenced this pull request Feb 3, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.