bugfix

- update ftl path
- delete duplicate security settings

README.md

@@ -1,14 +1,9 @@
-![Java_8](https://img.shields.io/badge/java-v1.8-red?logo=java)
-![Java_11](https://img.shields.io/badge/java-v11-red?logo=java)
-![Spring_Boot](https://img.shields.io/badge/Spring_Boot-v2.1.4-green.svg?logo=spring)
-![Spring_Security](https://img.shields.io/badge/Spring_Security-v5.1.5-green.svg?logo=spring)
-![GitHub stars](https://img.shields.io/github/stars/codej99/SpringRestApi?style=social)
 # Spring Rest Api 만들기 프로젝트
 
 ### 0. 개요
 - SpringBoot2 framework 기반에서 RESTful api 서비스를 Step by Step으로 만들어 나가는 프로젝트
 - daddyprogrammer.org에서 연재 및 소스 Github 등록
-    - https://daddyprogrammer.org/post/series/springboot2-make-rest-api/
+    - https://daddyprogrammer.org/post/series/springboot2%EB%A1%9C-rest-api-%EB%A7%8C%EB%93%A4%EA%B8%B0/
 
 ### 1. 개발환경
 - Java 8~11
@@ -125,8 +120,3 @@ alter table user_roles
         - https://daddyprogrammer.org/post/3870/spring-rest-api-redis-caching/
     - Git
         - https://github.com/codej99/SpringRestApi/tree/cache-data-redis
-  - SpringBoot2로 Rest api 만들기(16) – AOP와 Custom Annotation을 이용한 금칙어(Forbidden Word) 처리
-    - Document
-        - https://daddyprogrammer.org/post/11356/springboot2-forbidden-word-by-aop-annotation/
-    - Git
-        - https://github.com/codej99/SpringRestApi/tree/feature/block_fobidden_word

src/main/java/com/rest/api/advice/ExceptionAdvice.java

@@ -77,12 +77,6 @@ public class ExceptionAdvice {
         return responseService.getFailResult(Integer.valueOf(getMessage("resourceNotExist.code")), getMessage("resourceNotExist.msg"));
     }
 
-    @ExceptionHandler(CForbiddenWordException.class)
-    @ResponseStatus(HttpStatus.BAD_REQUEST)
-    public CommonResult forbiddenWordException(HttpServletRequest request, CForbiddenWordException e) {
-        return responseService.getFailResult(Integer.valueOf(getMessage("forbiddenWord.code")), getMessage("forbiddenWord.msg", new Object[]{e.getMessage()}));
-    }
-
     // code정보에 해당하는 메시지를 조회합니다.
     private String getMessage(String code) {
         return getMessage(code, null);

src/main/java/com/rest/api/advice/exception/CForbiddenWordException.java

@@ -1,16 +0,0 @@
-package com.rest.api.advice.exception;
-
-public class CForbiddenWordException extends RuntimeException {
-
-    public CForbiddenWordException(String msg, Throwable t) {
-        super(msg, t);
-    }
-
-    public CForbiddenWordException(String msg) {
-        super(msg);
-    }
-
-    public CForbiddenWordException() {
-        super();
-    }
-}

src/main/java/com/rest/api/annotation/ForbiddenWordCheck.java

@@ -1,15 +0,0 @@
-package com.rest.api.annotation;
-
-import com.rest.api.model.board.ParamsPost;
-
-import java.lang.annotation.ElementType;
-import java.lang.annotation.Retention;
-import java.lang.annotation.RetentionPolicy;
-import java.lang.annotation.Target;
-
-@Target({ElementType.METHOD})
-@Retention(RetentionPolicy.RUNTIME)
-public @interface ForbiddenWordCheck {
-    String param() default "paramsPost.content";
-    Class<?> checkClazz() default ParamsPost.class;
-}

src/main/java/com/rest/api/annotation/aspect/ForbiddenWordCheckAspect.java

@@ -1,74 +0,0 @@
-package com.rest.api.annotation.aspect;
-
-import com.rest.api.advice.exception.CForbiddenWordException;
-import com.rest.api.annotation.ForbiddenWordCheck;
-import io.micrometer.core.instrument.util.StringUtils;
-import org.aspectj.lang.JoinPoint;
-import org.aspectj.lang.annotation.Aspect;
-import org.aspectj.lang.annotation.Before;
-import org.aspectj.lang.reflect.MethodSignature;
-import org.springframework.stereotype.Component;
-
-import java.lang.reflect.Field;
-import java.util.Arrays;
-import java.util.List;
-import java.util.Optional;
-
-@Aspect
-@Component
-public class ForbiddenWordCheckAspect {
-
-    // 어노테이션이 설정된 메서드의 메인 프로세스가 시작되기전(Before)에 금칙어 체크 로직이 적용된다.
-    @Before(value = "@annotation(forbiddenWordCheck)")
-    public void forbiddenWordCheck(JoinPoint pjp, ForbiddenWordCheck forbiddenWordCheck) throws Throwable {
-        // 금칙어를 체크할 메서드의 파라미터가 객체인지(객체.필드명) 일반 String인지에 따라 구분하여 처리한다.
-        String[] param = forbiddenWordCheck.param().split("\\.");
-        String paramName;
-        String fieldName = "";
-        if (param.length == 2) {
-            paramName = param[0];
-            fieldName = param[1];
-        } else {
-            paramName = forbiddenWordCheck.param();
-        }
-        // 메서드의 파라미터 이름으로 메서드의 몇번째 파라미터인지 구한다.
-        Integer parameterIdx = getParameterIdx(pjp, paramName);
-        if (parameterIdx == -1)
-            throw new IllegalArgumentException();
-
-        String checkWord;
-        // 객체내의 필드값에서 금칙어 체크 문장을 얻어내야 할 경우
-        if (StringUtils.isNotEmpty(fieldName)) {
-            Class<?> clazz = forbiddenWordCheck.checkClazz();
-            Field field = clazz.getDeclaredField(fieldName);
-            field.setAccessible(true);
-            checkWord = (String) field.get(pjp.getArgs()[parameterIdx]);
-        // 금칙어 체크 문장이 String형의 파라미터로 넘어오는 경우
-        } else {
-            checkWord = (String) pjp.getArgs()[parameterIdx];
-        }
-        // 체크할 문장에 금칙어가 포함되어 있는지 확인
-        checkForbiddenWord(checkWord);
-    }
-
-    // 메서드의 파라미터 이름으로 몇번째에 파라미터가 위치하는지 구함
-    private Integer getParameterIdx(JoinPoint joinPoint, String paramName) {
-        MethodSignature methodSignature = (MethodSignature) joinPoint.getSignature();
-        String[] parameterNames = methodSignature.getParameterNames();
-        for (int i = 0; i < parameterNames.length; i++) {
-            String parameterName = parameterNames[i];
-            if (paramName.equals(parameterName)) {
-                return i;
-            }
-        }
-        return -1;
-    }
-
-    // 입력된 문장에 금칙어가 포함되어 있으면 Exception을 발생시킨다.
-    private void checkForbiddenWord(String word) {
-        List<String> forbiddenWords = Arrays.asList("개새끼", "쌍년", "씨발");
-        Optional<String> forbiddenWord = forbiddenWords.stream().filter(word::contains).findFirst();
-        if (forbiddenWord.isPresent())
-            throw new CForbiddenWordException(forbiddenWord.get());
-    }
-}

src/main/java/com/rest/api/config/RedisConfig.java

@@ -9,6 +9,8 @@ import org.springframework.data.redis.cache.CacheKeyPrefix;
 import org.springframework.data.redis.cache.RedisCacheConfiguration;
 import org.springframework.data.redis.cache.RedisCacheManager;
 import org.springframework.data.redis.connection.RedisConnectionFactory;
+import org.springframework.data.redis.core.RedisTemplate;
+import org.springframework.data.redis.serializer.Jackson2JsonRedisSerializer;
 import org.springframework.data.redis.serializer.RedisSerializationContext;
 import org.springframework.data.redis.serializer.StringRedisSerializer;
 

src/main/java/com/rest/api/config/security/CustomAccessDeniedHandler.java

@@ -1,13 +1,10 @@
 package com.rest.api.config.security;
 
 import lombok.extern.slf4j.Slf4j;
-import org.springframework.context.i18n.LocaleContextHolder;
 import org.springframework.security.access.AccessDeniedException;
 import org.springframework.security.web.access.AccessDeniedHandler;
 import org.springframework.stereotype.Component;
 
-import javax.servlet.RequestDispatcher;
-import javax.servlet.ServletException;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 import java.io.IOException;
@@ -17,9 +14,7 @@ import java.io.IOException;
 public class CustomAccessDeniedHandler implements AccessDeniedHandler {
 
     @Override
-    public void handle(HttpServletRequest request, HttpServletResponse response, AccessDeniedException exception) throws IOException, ServletException {
-        response.setLocale(LocaleContextHolder.getLocale());
-        RequestDispatcher rd = request.getRequestDispatcher("/exception/accessdenied");
-        rd.forward(request, response);
+    public void handle(HttpServletRequest request, HttpServletResponse response, AccessDeniedException exception) throws IOException {
+        response.sendRedirect("/exception/accessdenied");
     }
 }

src/main/java/com/rest/api/config/security/CustomAuthenticationEntryPoint.java

@@ -1,13 +1,10 @@
 package com.rest.api.config.security;
 
 import lombok.extern.slf4j.Slf4j;
-import org.springframework.context.i18n.LocaleContextHolder;
 import org.springframework.security.core.AuthenticationException;
 import org.springframework.security.web.AuthenticationEntryPoint;
 import org.springframework.stereotype.Component;
 
-import javax.servlet.RequestDispatcher;
-import javax.servlet.ServletException;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 import java.io.IOException;
@@ -16,9 +13,7 @@ import java.io.IOException;
 @Component
 public class CustomAuthenticationEntryPoint implements AuthenticationEntryPoint {
     @Override
-    public void commence(HttpServletRequest request, HttpServletResponse response, AuthenticationException ex) throws IOException, ServletException {
-        response.setLocale(LocaleContextHolder.getLocale());
-        RequestDispatcher rd = request.getRequestDispatcher("/exception/entrypoint");
-        rd.forward(request, response);
+    public void commence(HttpServletRequest request, HttpServletResponse response, AuthenticationException ex) throws IOException {
+        response.sendRedirect("/exception/entrypoint");
     }
 }

src/main/java/com/rest/api/config/security/JwtTokenProvider.java

@@ -25,7 +25,7 @@ public class JwtTokenProvider { // JWT 토큰을 생성 및 검증 모듈
     @Value("${spring.jwt.secret}")
     private String secretKey;
 
-    private long tokenValidMilisecond = 1000L * 60 * 60 * 24; // 24시간만 토큰 유효
+    private long tokenValidMilisecond = 1000L * 60 * 60; // 1시간만 토큰 유효
 
     private final UserDetailsService userDetailsService;
 

src/main/java/com/rest/api/config/security/SecurityConfiguration.java

@@ -3,7 +3,6 @@ package com.rest.api.config.security;
 import lombok.RequiredArgsConstructor;
 import org.springframework.context.annotation.Bean;
 import org.springframework.context.annotation.Configuration;
-import org.springframework.context.i18n.LocaleContextHolder;
 import org.springframework.http.HttpMethod;
 import org.springframework.security.authentication.AuthenticationManager;
 import org.springframework.security.config.annotation.web.builders.HttpSecurity;
@@ -11,12 +10,6 @@ import org.springframework.security.config.annotation.web.builders.WebSecurity;
 import org.springframework.security.config.annotation.web.configuration.WebSecurityConfigurerAdapter;
 import org.springframework.security.config.http.SessionCreationPolicy;
 import org.springframework.security.web.authentication.UsernamePasswordAuthenticationFilter;
-import org.springframework.web.servlet.i18n.SessionLocaleResolver;
-
-import javax.servlet.*;
-import javax.servlet.http.HttpServletRequest;
-import java.io.IOException;
-import java.util.Locale;
 
 @RequiredArgsConstructor
 @Configuration
@@ -39,14 +32,13 @@ public class SecurityConfiguration extends WebSecurityConfigurerAdapter {
             .and()
                 .authorizeRequests() // 다음 리퀘스트에 대한 사용권한 체크
                     .antMatchers("/*/signin", "/*/signin/**", "/*/signup", "/*/signup/**", "/social/**").permitAll() // 가입 및 인증 주소는 누구나 접근가능
-                    .antMatchers(HttpMethod.GET, "/exception/**", "/helloworld/**","/actuator/health", "/v1/board/**", "/favicon.ico").permitAll() // 등록된 GET요청 리소스는 누구나 접근가능
+                    .antMatchers(HttpMethod.GET, "/exception/**", "/helloworld/**","/actuator/health", "/v1/board/**", "/favicon.ico").permitAll() // 등록한 GET요청 리소스는 누구나 접근가능
                     .anyRequest().hasRole("USER") // 그외 나머지 요청은 모두 인증된 회원만 접근 가능
             .and()
                 .exceptionHandling().accessDeniedHandler(new CustomAccessDeniedHandler())
             .and()
                 .exceptionHandling().authenticationEntryPoint(new CustomAuthenticationEntryPoint())
             .and()
-                .addFilterBefore(new AthenticationEntryLocaleFilter(), UsernamePasswordAuthenticationFilter.class)
                 .addFilterBefore(new JwtAuthenticationFilter(jwtTokenProvider), UsernamePasswordAuthenticationFilter.class); // jwt token 필터를 id/password 인증 필터 전에 넣어라.
 
     }
@@ -57,31 +49,4 @@ public class SecurityConfiguration extends WebSecurityConfigurerAdapter {
                 "/swagger-ui.html", "/webjars/**", "/swagger/**");
 
     }
-
-    private static class AthenticationEntryLocaleFilter implements Filter {
-        private SessionLocaleResolver localeResolver;
-
-        private AthenticationEntryLocaleFilter() {
-            localeResolver = new SessionLocaleResolver();
-            localeResolver.setDefaultLocale(Locale.KOREAN);
-        }
-
-        @Override
-        public void init(FilterConfig filterConfig) {
-        }
-
-        @Override
-        public void doFilter(ServletRequest request, ServletResponse response, FilterChain chain) throws IOException, ServletException {
-            if(request.getParameter("lang") != null)
-                localeResolver.setDefaultLocale(Locale.forLanguageTag(request.getParameter("lang")));
-            Locale locale = localeResolver.resolveLocale((HttpServletRequest) request);
-            LocaleContextHolder.setLocale(locale);
-            chain.doFilter(request, response);
-        }
-
-        @Override
-        public void destroy() {
-        }
-    }
 }
-

src/main/java/com/rest/api/controller/exception/ExceptionController.java

@@ -18,7 +18,7 @@ public class ExceptionController {
         throw new CAuthenticationEntryPointException();
     }
 
-    @RequestMapping(value = "/accessdenied")
+    @GetMapping(value = "/accessdenied")
     public CommonResult accessdeniedException() {
         throw new AccessDeniedException("");
     }

src/main/java/com/rest/api/controller/v1/board/BoardController.java

@@ -28,15 +28,6 @@ public class BoardController {
     private final BoardService boardService;
     private final ResponseService responseService;
 
-    @ApiImplicitParams({
-            @ApiImplicitParam(name = "X-AUTH-TOKEN", value = "로그인 성공 후 access_token", required = true, dataType = "String", paramType = "header")
-    })
-    @ApiOperation(value = "게시판 생성", notes = "신규 게시판을 생성한다.")
-    @PostMapping(value = "/{boardName}")
-    public SingleResult<Board> createBoard(@PathVariable String boardName) {
-        return responseService.getSingleResult(boardService.insertBoard(boardName));
-    }
-
     @ApiOperation(value = "게시판 정보 조회", notes = "게시판 정보를 조회한다.")
     @GetMapping(value = "/{boardName}")
     public SingleResult<Board> boardInfo(@PathVariable String boardName) {
@@ -53,7 +44,7 @@ public class BoardController {
             @ApiImplicitParam(name = "X-AUTH-TOKEN", value = "로그인 성공 후 access_token", required = true, dataType = "String", paramType = "header")
     })
     @ApiOperation(value = "게시글 작성", notes = "게시글을 작성한다.")
-    @PostMapping(value = "/{boardName}/post")
+    @PostMapping(value = "/{boardName}")
     public SingleResult<Post> post(@PathVariable String boardName, @Valid @ModelAttribute ParamsPost post) {
         Authentication authentication = SecurityContextHolder.getContext().getAuthentication();
         String uid = authentication.getName();

src/main/java/com/rest/api/entity/board/Board.java

@@ -1,19 +1,15 @@
 package com.rest.api.entity.board;
 
 import com.rest.api.entity.common.CommonDateEntity;
-import lombok.AllArgsConstructor;
-import lombok.Builder;
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 
 import javax.persistence.*;
 import java.io.Serializable;
 
-@Builder
 @Entity
 @Getter
 @NoArgsConstructor
-@AllArgsConstructor
 public class Board extends CommonDateEntity implements Serializable {
     @Id
     @GeneratedValue(strategy = GenerationType.IDENTITY)

src/main/java/com/rest/api/entity/common/CommonDateEntity.java

@@ -7,13 +7,12 @@ import org.springframework.data.jpa.domain.support.AuditingEntityListener;
 
 import javax.persistence.EntityListeners;
 import javax.persistence.MappedSuperclass;
-import java.io.Serializable;
 import java.time.LocalDateTime;
 
 @Getter
 @MappedSuperclass
 @EntityListeners(AuditingEntityListener.class)
-public abstract class CommonDateEntity implements Serializable {
+public abstract class CommonDateEntity {
     @CreatedDate
     private LocalDateTime createdAt;
     @LastModifiedDate

src/main/java/com/rest/api/repo/board/PostJpaRepo.java

@@ -7,5 +7,5 @@ import org.springframework.data.jpa.repository.JpaRepository;
 import java.util.List;
 
 public interface PostJpaRepo extends JpaRepository<Post, Long> {
-    List<Post> findByBoardOrderByPostIdDesc(Board board);
+    List<Post> findByBoard(Board board);
 }

src/main/java/com/rest/api/service/board/BoardService.java

@@ -1,10 +1,8 @@
 package com.rest.api.service.board;
 
-import com.rest.api.advice.exception.CForbiddenWordException;
 import com.rest.api.advice.exception.CNotOwnerException;
 import com.rest.api.advice.exception.CResourceNotExistException;
 import com.rest.api.advice.exception.CUserNotFoundException;
-import com.rest.api.annotation.ForbiddenWordCheck;
 import com.rest.api.common.CacheKey;
 import com.rest.api.entity.User;
 import com.rest.api.entity.board.Board;
@@ -22,7 +20,6 @@ import org.springframework.cache.annotation.Cacheable;
 import org.springframework.stereotype.Service;
 
 import javax.transaction.Transactional;
-import java.util.Arrays;
 import java.util.List;
 import java.util.Optional;
 
@@ -37,10 +34,6 @@ public class BoardService {
     private final UserJpaRepo userJpaRepo;
     private final CacheSevice cacheSevice;
 
-    public Board insertBoard(String boardName) {
-        return boardJpaRepo.save(Board.builder().name(boardName).build());
-    }
-
     // 게시판 이름으로 게시판을 조회. 없을경우 CResourceNotExistException 처리
     @Cacheable(value = CacheKey.BOARD, key = "#boardName", unless = "#result == null")
     public Board findBoard(String boardName) {
@@ -50,7 +43,7 @@ public class BoardService {
     // 게시판 이름으로 게시글 리스트 조회.
     @Cacheable(value = CacheKey.POSTS, key = "#boardName", unless = "#result == null")
     public List<Post> findPosts(String boardName) {
-        return postJpaRepo.findByBoardOrderByPostIdDesc(findBoard(boardName));
+        return postJpaRepo.findByBoard(findBoard(boardName));
     }
 
     // 게시글ID로 게시글 단건 조회. 없을경우 CResourceNotExistException 처리
@@ -61,7 +54,6 @@ public class BoardService {
 
     // 게시글을 등록합니다. 게시글의 회원UID가 조회되지 않으면 CUserNotFoundException 처리합니다.
     @CacheEvict(value = CacheKey.POSTS, key = "#boardName")
-    @ForbiddenWordCheck
     public Post writePost(String uid, String boardName, ParamsPost paramsPost) {
         Board board = findBoard(boardName);
         Post post = new Post(userJpaRepo.findByUid(uid).orElseThrow(CUserNotFoundException::new), board, paramsPost.getAuthor(), paramsPost.getTitle(), paramsPost.getContent());
@@ -70,7 +62,6 @@ public class BoardService {
 
     // 게시글을 수정합니다. 게시글 등록자와 로그인 회원정보가 틀리면 CNotOwnerException 처리합니다.
     //@CachePut(value = CacheKey.POST, key = "#postId") 갱신된 정보만 캐시할경우에만 사용!
-    @ForbiddenWordCheck
     public Post updatePost(long postId, String uid, ParamsPost paramsPost) {
         Post post = getPost(postId);
         User user = post.getUser();

src/main/resources/i18n/exception_en.yml

@@ -24,7 +24,4 @@ notOwner:
   msg: "You are not the owner of this resource."
 resourceNotExist:
   code: "-1007"
-  msg: "This resource does not exist."
-forbiddenWord:
-  code: "-1008"
-  msg: "forbidden words ({0}) are included in the input."
+  msg: "This resource does not exist."

src/main/resources/i18n/exception_ko.yml

@@ -24,7 +24,4 @@ notOwner:
   msg: "해당 자원의 소유자가 아닙니다."
 resourceNotExist:
   code: "-1007"
-  msg: "요청한 자원이 존재 하지 않습니다."
-forbiddenWord:
-  code: "-1008"
-  msg: "입력한 내용에 금칙어({0})가 포함되어 있습니다."
+  msg: "요청한 자원이 존재 하지 않습니다."