Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Fix/redis-insert]- 2글자 이하 입력 시 redis에서 error를 throw하는 에러 수정 #118

Merged
merged 5 commits into from
Dec 12, 2022

Conversation

JunYupK
Copy link
Collaborator

@JunYupK JunYupK commented Dec 9, 2022

개요

입력값을 2글자 미만인 경우를 validation pipe에서 막고 있었으나, redis검증 로직에서 2글자 이하인 경우를 에러를 내는 설계 오류가 있었습니다.

작업사항

  • 2글자 미만일시 return으로 변경하였습니다.

@github-actions github-actions bot requested review from leesungbin and Palwol December 9, 2022 13:04
@leesungbin
Copy link
Member

crossref 탐색하는 로직이 붙다보니, 기존 테스트코드가 통과하지 못했습니다~

Copy link
Collaborator

@Palwol Palwol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다.

@JunYupK JunYupK merged commit 0dfad90 into dev Dec 12, 2022
@JunYupK JunYupK deleted the fix/redis-insert branch December 12, 2022 05:44
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants