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.

Sign up for GitHub

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

Jump to bottom

Fix: 회원만 조회 수 증가 가능하도록 수정 #177

Merged

Seongjun-Kwon merged 2 commits into develop from fix/#176

Sep 1, 2023

Owner

Seongjun-Kwon commented Sep 1, 2023

관련 이슈

close Fix: 회원만 조회 수 증가 가능하도록 수정 #176

내용

기존 방식은 userKey 쿠키에 UUID 를 담아서 사용자를 구분하고, UUID + 조회한 후기 id 형태의 데이터를 redis 에 저장하여 조회 수 중복 증가를 방지하였다. 하지만, 이 방식은 사용자가 쿠키를 비우면 새로운 사용자로 취급되므로 조회 수 어뷰징의 가능성이 있다. 그러므로, 회원만 조회 수 증가가 이루어지도록 방식을 변경하였다.

Seongjun-Kwon added 2 commits

September 1, 2023 19:06


          Fix: 회원만 조회 수 증가 가능하도록 수정

ea04527


          Style: 메서드명 가독성 좋게 수정

914799f

- hasNotViewedReview -> doNotViewReview
  - 회원이 후기를 봤는지를 판별하는 메서드임

Seongjun-Kwon self-assigned this

Seongjun-Kwon merged commit f77cfe7 into develop

1 check passed

Seongjun-Kwon deleted the fix/#176 branch

September 1, 2023 10:14

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet