diff options
author | Vitaly Takmazov | 2022-12-08 07:26:58 +0300 |
---|---|---|
committer | Vitaly Takmazov | 2022-12-08 07:26:58 +0300 |
commit | 7b8a99a6074228a5680218690b7f1538ce7e8d1d (patch) | |
tree | 1a5803430b00737b3f36f79d3ecc4cdf40af6584 /src/main/java/com/juick/www/controllers | |
parent | 5e0500933d2f805fe879ced9171c65839c72579b (diff) |
Cleanup unused authorization checks handled by spring-security
Diffstat (limited to 'src/main/java/com/juick/www/controllers')
-rw-r--r-- | src/main/java/com/juick/www/controllers/Settings.java | 13 | ||||
-rw-r--r-- | src/main/java/com/juick/www/controllers/Site.java | 6 |
2 files changed, 4 insertions, 15 deletions
diff --git a/src/main/java/com/juick/www/controllers/Settings.java b/src/main/java/com/juick/www/controllers/Settings.java index 1e40b9d1..0b486766 100644 --- a/src/main/java/com/juick/www/controllers/Settings.java +++ b/src/main/java/com/juick/www/controllers/Settings.java @@ -91,18 +91,13 @@ public class Settings { protected String doGet( @Visitor User visitor, Locale locale, - HttpServletRequest request, - HttpServletResponse response, ModelMap model) throws IOException { - if (visitor.isAnonymous()) { - response.sendRedirect("/login"); - } + @RequestParam(required = false, defaultValue = "main") String page, + @RequestParam(required = false) String code, ModelMap model) throws IOException { visitor.setAvatar(webApp.getAvatarWebPath(visitor)); List<String> pages = Arrays.asList("main", "password", "about", "auth-email", "privacy"); - String page = request.getParameter("page"); - if (StringUtils.isEmpty(page) || !pages.contains(page)) { + if (!pages.contains(page)) { page = "main"; } - model.addAttribute("title", ResourceBundle.getBundle("messages", locale).getString("link.settings")); model.addAttribute("visitor", visitor); model.addAttribute("tags", tagService.getPopularTags()); @@ -120,7 +115,7 @@ public class Settings { model.addAttribute("notify_options", subscriptionService.getNotifyOptions(visitor)); model.addAttribute("userinfo", userService.getUserInfo(visitor)); if (page.equals("auth-email")) { - if (emailService.verifyAddressByCode(visitor.getUid(), request.getParameter("code"))) { + if (emailService.verifyAddressByCode(visitor.getUid(), code)) { model.addAttribute("result", "OK!"); } else { model.addAttribute("result", "Sorry, code unknown."); diff --git a/src/main/java/com/juick/www/controllers/Site.java b/src/main/java/com/juick/www/controllers/Site.java index aa6d574d..e28f8e15 100644 --- a/src/main/java/com/juick/www/controllers/Site.java +++ b/src/main/java/com/juick/www/controllers/Site.java @@ -451,9 +451,6 @@ public class Site { @GetMapping("/pm/inbox") protected String doGetInbox(@Visitor User visitor, ModelMap model) { - if (visitor.isAnonymous()) { - return "redirect:/login"; - } visitor.setAvatar(webApp.getAvatarWebPath(visitor)); String title = "PM: Inbox"; List<Message> msgs = pmQueriesService.getLastPMInbox(visitor.getUid()); @@ -468,9 +465,6 @@ public class Site { @GetMapping("/pm/sent") protected String doGetSent(@Visitor User visitor, @RequestParam(required = false) String uname, ModelMap model) { - if (visitor.isAnonymous()) { - return "redirect:/login"; - } visitor.setAvatar(webApp.getAvatarWebPath(visitor)); String title = "PM: Sent"; List<Message> msgs = pmQueriesService.getLastPMSent(visitor.getUid()); |