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

Add /servlet/ to the DefaultBackEndLoginRequiredWebInterceptor #15548

Closed
wezell opened this issue Nov 14, 2018 · 3 comments

Comments

Projects
4 participants
@wezell
Copy link
Contributor

commented Nov 14, 2018

Followup to #15371 - we added /servlets/ and did not add /servlet/

wezell added a commit that referenced this issue Jan 31, 2019

@wezell

This comment has been minimized.

Copy link
Contributor Author

commented Jan 31, 2019

PR #15930

@wezell wezell added the WF : Needs QA label Feb 1, 2019

jgambarios added a commit that referenced this issue Feb 5, 2019

@bryanboza bryanboza added this to CODY in QA Feb 5, 2019

@bryanboza

This comment has been minimized.

Copy link
Contributor

commented Feb 5, 2019

Fixed, tested on master // Oracle // FF

@bryanboza bryanboza moved this from CODY to Done in QA Feb 5, 2019

@wezell wezell closed this Feb 6, 2019

@jgambarios

This comment has been minimized.

Copy link
Contributor

commented Feb 7, 2019

For documentation: As part of this change we needed to rename the test servlet path from /servlet/test to /dotTest and for that reason we need to update the URLs in https://dotcms.com/docs/latest/functional-tests

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.