-
Notifications
You must be signed in to change notification settings - Fork 7
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
ci: 2024-07-14 배포 #827
Merged
ci: 2024-07-14 배포 #827
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
searchUsersById 서비스 함수의 종속성이 생각보다 많음.controller 에서 items 의 length 를 확인하도록
Co-authored-by: kylee <kylee@fitfuns.com>
* refactor: 400번대 에러 반환 제거 * refactor: overDueDay 반환 값에서 제거
* feat(utils): rate limit 모듈 추가 - R, CUD에 해당하는 rate limit 모듈 추가 * refactor(cursus): rate limit 모듈 import해서 사용하도록 변경 * feat: getRateLimiter 적용 * feat(books): 유효한 ISBN인지 검사하는 로직 추가 * feat(tags): tags router에 rate limit 추가 * feat(routes): router에서 authValidate를 미들웨어로 쓰는 곳에 rate limiter 추가 * feat(auth): /get/me에 rate limiter 추가 * feat(users): 유저 생성 후 created 문장 출력 시, db에 저장된 email 값을 사용 * build: csrf 방지를 위한 lusca 패키지 추가 * feat(app): csrf 방지 로직 추가 * feat(app): csrf 방지 옵션 수정 * build: express-session 패키지 추가 * feat(app): lusca 상태 유지를 위한 세션 추가 * feat(app): cookie에도 secret 추가 * feat(app): session에서 cookie 설정 및 lusca에서 부가적인 설정 제거
* refactor: 400번대 에러 반환 제거 * refactor: overDueDay 반환 값에서 제거
This reverts commit 9adc701.
* feat(lending): 사서의 대출 제한 권수 4권으로 조정 * style(lending): prettier 재적용
scarf005
approved these changes
Jul 13, 2024
Jake1152
approved these changes
Jul 25, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
기록을 위해 슬랙에서 남겼던 질의 응답 깃허브에서도 동일하게 남기는 것으로 리뷰 대체합니다
jim
2권만 빌릴 수 있는 이유가 db 테이블의 대출한 정보를 2권까지 기록할 수 있다고 들었었는데 맞을까요
맞다면 테이블 항목이 추가된 것일까요?
yena
DB 자체에서 행 개수를 제한하지는 않고, 서비스단에서만 2권 제약을 걸어둔 거라 서비스의 제약 조건을 변경했어요
nyj001012
added a commit
that referenced
this pull request
Oct 3, 2024
* ci: 2024-07-14 배포 (#827) * feat: 유저 search 할 때 id 가 undefined 인 경우 핸들링 * fix: searchUsersById 타입을 이전과 같이 리턴하도록 변경 searchUsersById 서비스 함수의 종속성이 생각보다 많음.controller 에서 items 의 length 를 확인하도록 * [fix] backend dockerfile error (#764) Co-authored-by: kylee <kylee@fitfuns.com> * User API 경로 정리 (#777) * refactor: 400번대 에러 반환 제거 * refactor: overDueDay 반환 값에서 제거 * fix: `dev/v2` 경로 복구 (#808) * security: 보안 취약점 해결 (#818) * feat(utils): rate limit 모듈 추가 - R, CUD에 해당하는 rate limit 모듈 추가 * refactor(cursus): rate limit 모듈 import해서 사용하도록 변경 * feat: getRateLimiter 적용 * feat(books): 유효한 ISBN인지 검사하는 로직 추가 * feat(tags): tags router에 rate limit 추가 * feat(routes): router에서 authValidate를 미들웨어로 쓰는 곳에 rate limiter 추가 * feat(auth): /get/me에 rate limiter 추가 * feat(users): 유저 생성 후 created 문장 출력 시, db에 저장된 email 값을 사용 * build: csrf 방지를 위한 lusca 패키지 추가 * feat(app): csrf 방지 로직 추가 * feat(app): csrf 방지 옵션 수정 * build: express-session 패키지 추가 * feat(app): lusca 상태 유지를 위한 세션 추가 * feat(app): cookie에도 secret 추가 * feat(app): session에서 cookie 설정 및 lusca에서 부가적인 설정 제거 * User API 경로 정리 (#777) * refactor: 400번대 에러 반환 제거 * refactor: overDueDay 반환 값에서 제거 * Revert "User API 경로 정리 (#777)" This reverts commit 9adc701. * feat(lending): 사서의 대출 제한 권수 4권으로 조정 (#826) * feat(lending): 사서의 대출 제한 권수 4권으로 조정 * style(lending): prettier 재적용 --------- Co-authored-by: jimin <tesla52@naver.com> Co-authored-by: gilee <gilee@student.42seoul.kr> Co-authored-by: kylee <kylee@fitfuns.com> Co-authored-by: scarf <greenscarf005@gmail.com> * cd: 버킷 이름 테스트 (#829) * ci: Update stop.sh * cd: Update appspec.yml --------- Co-authored-by: jimin <tesla52@naver.com> Co-authored-by: gilee <gilee@student.42seoul.kr> Co-authored-by: kylee <kylee@fitfuns.com> Co-authored-by: scarf <greenscarf005@gmail.com>
nyj001012
added a commit
that referenced
this pull request
Oct 3, 2024
* ci: 2024-07-14 배포 (#827) * feat: 유저 search 할 때 id 가 undefined 인 경우 핸들링 * fix: searchUsersById 타입을 이전과 같이 리턴하도록 변경 searchUsersById 서비스 함수의 종속성이 생각보다 많음.controller 에서 items 의 length 를 확인하도록 * [fix] backend dockerfile error (#764) * User API 경로 정리 (#777) * refactor: 400번대 에러 반환 제거 * refactor: overDueDay 반환 값에서 제거 * fix: `dev/v2` 경로 복구 (#808) * security: 보안 취약점 해결 (#818) * feat(utils): rate limit 모듈 추가 - R, CUD에 해당하는 rate limit 모듈 추가 * refactor(cursus): rate limit 모듈 import해서 사용하도록 변경 * feat: getRateLimiter 적용 * feat(books): 유효한 ISBN인지 검사하는 로직 추가 * feat(tags): tags router에 rate limit 추가 * feat(routes): router에서 authValidate를 미들웨어로 쓰는 곳에 rate limiter 추가 * feat(auth): /get/me에 rate limiter 추가 * feat(users): 유저 생성 후 created 문장 출력 시, db에 저장된 email 값을 사용 * build: csrf 방지를 위한 lusca 패키지 추가 * feat(app): csrf 방지 로직 추가 * feat(app): csrf 방지 옵션 수정 * build: express-session 패키지 추가 * feat(app): lusca 상태 유지를 위한 세션 추가 * feat(app): cookie에도 secret 추가 * feat(app): session에서 cookie 설정 및 lusca에서 부가적인 설정 제거 * User API 경로 정리 (#777) * refactor: 400번대 에러 반환 제거 * refactor: overDueDay 반환 값에서 제거 * Revert "User API 경로 정리 (#777)" This reverts commit 9adc701. * feat(lending): 사서의 대출 제한 권수 4권으로 조정 (#826) * feat(lending): 사서의 대출 제한 권수 4권으로 조정 * style(lending): prettier 재적용 --------- * cd: 버킷 이름 테스트 (#829) * ci: Update stop.sh * cd: Update appspec.yml --------- Co-authored-by: jimin <tesla52@naver.com> Co-authored-by: gilee <gilee@student.42seoul.kr> Co-authored-by: kylee <kylee@fitfuns.com> Co-authored-by: scarf <greenscarf005@gmail.com>
nyj001012
added a commit
that referenced
this pull request
Oct 4, 2024
* cd: appspec.yml 변경 (#830) * ci: 2024-07-14 배포 (#827) * feat: 유저 search 할 때 id 가 undefined 인 경우 핸들링 * fix: searchUsersById 타입을 이전과 같이 리턴하도록 변경 searchUsersById 서비스 함수의 종속성이 생각보다 많음.controller 에서 items 의 length 를 확인하도록 * [fix] backend dockerfile error (#764) Co-authored-by: kylee <kylee@fitfuns.com> * User API 경로 정리 (#777) * refactor: 400번대 에러 반환 제거 * refactor: overDueDay 반환 값에서 제거 * fix: `dev/v2` 경로 복구 (#808) * security: 보안 취약점 해결 (#818) * feat(utils): rate limit 모듈 추가 - R, CUD에 해당하는 rate limit 모듈 추가 * refactor(cursus): rate limit 모듈 import해서 사용하도록 변경 * feat: getRateLimiter 적용 * feat(books): 유효한 ISBN인지 검사하는 로직 추가 * feat(tags): tags router에 rate limit 추가 * feat(routes): router에서 authValidate를 미들웨어로 쓰는 곳에 rate limiter 추가 * feat(auth): /get/me에 rate limiter 추가 * feat(users): 유저 생성 후 created 문장 출력 시, db에 저장된 email 값을 사용 * build: csrf 방지를 위한 lusca 패키지 추가 * feat(app): csrf 방지 로직 추가 * feat(app): csrf 방지 옵션 수정 * build: express-session 패키지 추가 * feat(app): lusca 상태 유지를 위한 세션 추가 * feat(app): cookie에도 secret 추가 * feat(app): session에서 cookie 설정 및 lusca에서 부가적인 설정 제거 * User API 경로 정리 (#777) * refactor: 400번대 에러 반환 제거 * refactor: overDueDay 반환 값에서 제거 * Revert "User API 경로 정리 (#777)" This reverts commit 9adc701. * feat(lending): 사서의 대출 제한 권수 4권으로 조정 (#826) * feat(lending): 사서의 대출 제한 권수 4권으로 조정 * style(lending): prettier 재적용 --------- Co-authored-by: jimin <tesla52@naver.com> Co-authored-by: gilee <gilee@student.42seoul.kr> Co-authored-by: kylee <kylee@fitfuns.com> Co-authored-by: scarf <greenscarf005@gmail.com> * cd: 버킷 이름 테스트 (#829) * ci: Update stop.sh * cd: Update appspec.yml --------- Co-authored-by: jimin <tesla52@naver.com> Co-authored-by: gilee <gilee@student.42seoul.kr> Co-authored-by: kylee <kylee@fitfuns.com> Co-authored-by: scarf <greenscarf005@gmail.com> * ci: Update main.yml --------- Co-authored-by: jimin <tesla52@naver.com> Co-authored-by: gilee <gilee@student.42seoul.kr> Co-authored-by: kylee <kylee@fitfuns.com> Co-authored-by: scarf <greenscarf005@gmail.com>
nyj001012
added a commit
that referenced
this pull request
Oct 4, 2024
* ci: 2024-07-14 배포 (#827) * feat: 유저 search 할 때 id 가 undefined 인 경우 핸들링 * fix: searchUsersById 타입을 이전과 같이 리턴하도록 변경 searchUsersById 서비스 함수의 종속성이 생각보다 많음.controller 에서 items 의 length 를 확인하도록 * [fix] backend dockerfile error (#764) Co-authored-by: kylee <kylee@fitfuns.com> * User API 경로 정리 (#777) * refactor: 400번대 에러 반환 제거 * refactor: overDueDay 반환 값에서 제거 * fix: `dev/v2` 경로 복구 (#808) * security: 보안 취약점 해결 (#818) * feat(utils): rate limit 모듈 추가 - R, CUD에 해당하는 rate limit 모듈 추가 * refactor(cursus): rate limit 모듈 import해서 사용하도록 변경 * feat: getRateLimiter 적용 * feat(books): 유효한 ISBN인지 검사하는 로직 추가 * feat(tags): tags router에 rate limit 추가 * feat(routes): router에서 authValidate를 미들웨어로 쓰는 곳에 rate limiter 추가 * feat(auth): /get/me에 rate limiter 추가 * feat(users): 유저 생성 후 created 문장 출력 시, db에 저장된 email 값을 사용 * build: csrf 방지를 위한 lusca 패키지 추가 * feat(app): csrf 방지 로직 추가 * feat(app): csrf 방지 옵션 수정 * build: express-session 패키지 추가 * feat(app): lusca 상태 유지를 위한 세션 추가 * feat(app): cookie에도 secret 추가 * feat(app): session에서 cookie 설정 및 lusca에서 부가적인 설정 제거 * User API 경로 정리 (#777) * refactor: 400번대 에러 반환 제거 * refactor: overDueDay 반환 값에서 제거 * Revert "User API 경로 정리 (#777)" This reverts commit 9adc701. * feat(lending): 사서의 대출 제한 권수 4권으로 조정 (#826) * feat(lending): 사서의 대출 제한 권수 4권으로 조정 * style(lending): prettier 재적용 --------- Co-authored-by: jimin <tesla52@naver.com> Co-authored-by: gilee <gilee@student.42seoul.kr> Co-authored-by: kylee <kylee@fitfuns.com> Co-authored-by: scarf <greenscarf005@gmail.com> * cd: 버킷 이름 테스트 (#829) * ci: Update stop.sh * cd: Update appspec.yml * cd: appspec.yml 변경 (#830) (#831) * ci: 2024-07-14 배포 (#827) * feat: 유저 search 할 때 id 가 undefined 인 경우 핸들링 * fix: searchUsersById 타입을 이전과 같이 리턴하도록 변경 searchUsersById 서비스 함수의 종속성이 생각보다 많음.controller 에서 items 의 length 를 확인하도록 * [fix] backend dockerfile error (#764) * User API 경로 정리 (#777) * refactor: 400번대 에러 반환 제거 * refactor: overDueDay 반환 값에서 제거 * fix: `dev/v2` 경로 복구 (#808) * security: 보안 취약점 해결 (#818) * feat(utils): rate limit 모듈 추가 - R, CUD에 해당하는 rate limit 모듈 추가 * refactor(cursus): rate limit 모듈 import해서 사용하도록 변경 * feat: getRateLimiter 적용 * feat(books): 유효한 ISBN인지 검사하는 로직 추가 * feat(tags): tags router에 rate limit 추가 * feat(routes): router에서 authValidate를 미들웨어로 쓰는 곳에 rate limiter 추가 * feat(auth): /get/me에 rate limiter 추가 * feat(users): 유저 생성 후 created 문장 출력 시, db에 저장된 email 값을 사용 * build: csrf 방지를 위한 lusca 패키지 추가 * feat(app): csrf 방지 로직 추가 * feat(app): csrf 방지 옵션 수정 * build: express-session 패키지 추가 * feat(app): lusca 상태 유지를 위한 세션 추가 * feat(app): cookie에도 secret 추가 * feat(app): session에서 cookie 설정 및 lusca에서 부가적인 설정 제거 * User API 경로 정리 (#777) * refactor: 400번대 에러 반환 제거 * refactor: overDueDay 반환 값에서 제거 * Revert "User API 경로 정리 (#777)" This reverts commit 9adc701. * feat(lending): 사서의 대출 제한 권수 4권으로 조정 (#826) * feat(lending): 사서의 대출 제한 권수 4권으로 조정 * style(lending): prettier 재적용 --------- * cd: 버킷 이름 테스트 (#829) * ci: Update stop.sh * cd: Update appspec.yml --------- Co-authored-by: jimin <tesla52@naver.com> Co-authored-by: gilee <gilee@student.42seoul.kr> Co-authored-by: kylee <kylee@fitfuns.com> Co-authored-by: scarf <greenscarf005@gmail.com> * ci: 배포 깃허브 액션 이름 EC2에서 LightSail로 변경 (#832) * cd: appspec.yml 변경 (#830) * ci: 2024-07-14 배포 (#827) * feat: 유저 search 할 때 id 가 undefined 인 경우 핸들링 * fix: searchUsersById 타입을 이전과 같이 리턴하도록 변경 searchUsersById 서비스 함수의 종속성이 생각보다 많음.controller 에서 items 의 length 를 확인하도록 * [fix] backend dockerfile error (#764) Co-authored-by: kylee <kylee@fitfuns.com> * User API 경로 정리 (#777) * refactor: 400번대 에러 반환 제거 * refactor: overDueDay 반환 값에서 제거 * fix: `dev/v2` 경로 복구 (#808) * security: 보안 취약점 해결 (#818) * feat(utils): rate limit 모듈 추가 - R, CUD에 해당하는 rate limit 모듈 추가 * refactor(cursus): rate limit 모듈 import해서 사용하도록 변경 * feat: getRateLimiter 적용 * feat(books): 유효한 ISBN인지 검사하는 로직 추가 * feat(tags): tags router에 rate limit 추가 * feat(routes): router에서 authValidate를 미들웨어로 쓰는 곳에 rate limiter 추가 * feat(auth): /get/me에 rate limiter 추가 * feat(users): 유저 생성 후 created 문장 출력 시, db에 저장된 email 값을 사용 * build: csrf 방지를 위한 lusca 패키지 추가 * feat(app): csrf 방지 로직 추가 * feat(app): csrf 방지 옵션 수정 * build: express-session 패키지 추가 * feat(app): lusca 상태 유지를 위한 세션 추가 * feat(app): cookie에도 secret 추가 * feat(app): session에서 cookie 설정 및 lusca에서 부가적인 설정 제거 * User API 경로 정리 (#777) * refactor: 400번대 에러 반환 제거 * refactor: overDueDay 반환 값에서 제거 * Revert "User API 경로 정리 (#777)" This reverts commit 9adc701. * feat(lending): 사서의 대출 제한 권수 4권으로 조정 (#826) * feat(lending): 사서의 대출 제한 권수 4권으로 조정 * style(lending): prettier 재적용 --------- Co-authored-by: jimin <tesla52@naver.com> Co-authored-by: gilee <gilee@student.42seoul.kr> Co-authored-by: kylee <kylee@fitfuns.com> Co-authored-by: scarf <greenscarf005@gmail.com> * cd: 버킷 이름 테스트 (#829) * ci: Update stop.sh * cd: Update appspec.yml --------- Co-authored-by: jimin <tesla52@naver.com> Co-authored-by: gilee <gilee@student.42seoul.kr> Co-authored-by: kylee <kylee@fitfuns.com> Co-authored-by: scarf <greenscarf005@gmail.com> * ci: Update main.yml --------- Co-authored-by: jimin <tesla52@naver.com> Co-authored-by: gilee <gilee@student.42seoul.kr> Co-authored-by: kylee <kylee@fitfuns.com> Co-authored-by: scarf <greenscarf005@gmail.com> --------- Co-authored-by: jimin <tesla52@naver.com> Co-authored-by: gilee <gilee@student.42seoul.kr> Co-authored-by: kylee <kylee@fitfuns.com> Co-authored-by: scarf <greenscarf005@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
개요
스크린샷 (optional)