Skip to content
Snippets Groups Projects
Commit 9dedac65 authored by uhensler's avatar uhensler
Browse files

OO-4801: No waring in log if url is invalid

parent 9cfa7208
No related branches found
No related tags found
No related merge requests found
...@@ -42,6 +42,7 @@ import org.olat.core.gui.control.WindowControl; ...@@ -42,6 +42,7 @@ import org.olat.core.gui.control.WindowControl;
import org.olat.core.gui.control.creator.ControllerCreator; import org.olat.core.gui.control.creator.ControllerCreator;
import org.olat.core.helpers.Settings; import org.olat.core.helpers.Settings;
import org.olat.core.logging.Tracing; import org.olat.core.logging.Tracing;
import org.olat.core.util.StringHelper;
import org.olat.core.util.UserSession; import org.olat.core.util.UserSession;
import org.olat.core.util.i18n.I18nManager; import org.olat.core.util.i18n.I18nManager;
import org.olat.dispatcher.LocaleNegotiator; import org.olat.dispatcher.LocaleNegotiator;
...@@ -89,13 +90,19 @@ public class DocEditorDispatcher implements Dispatcher { ...@@ -89,13 +90,19 @@ public class DocEditorDispatcher implements Dispatcher {
@Override @Override
public void execute(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException { public void execute(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
log.debug("Dispatch dispatcher request: {}", request.getRequestURI()); log.debug("Dispatch dispatcher request: {}", request.getRequestURI());
String accessToken = getAccessToken(request); String accessToken = getAccessToken(request);
if (!StringHelper.containsNonWhitespace(accessToken) || accessToken.contains(":")) {
log.debug("Invalid access token {}", accessToken);
DispatcherModule.sendNotFound(request.getPathInfo(), response);
return;
}
log.debug("Document dispatcher: Get document. Token {}", accessToken); log.debug("Document dispatcher: Get document. Token {}", accessToken);
Access access = docEditorService.getAccess(accessToken); Access access = docEditorService.getAccess(accessToken);
if (access == null) { if (access == null) {
log.warn("Doc editor edit callback. No access for token {}", accessToken); log.warn("No access for token {}", accessToken);
DispatcherModule.sendNotFound(request.getPathInfo(), response); DispatcherModule.sendNotFound(request.getPathInfo(), response);
return; return;
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment