Skip to content
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

mongo regex escape하도록 처리 #200

Merged
merged 2 commits into from
Jan 5, 2024
Merged

Conversation

Hank-Choi
Copy link
Contributor

@Hank-Choi Hank-Choi commented Jan 5, 2024

query 에 특수문자 올 때 regex로 들어가면 오류생겨서 뺐었는데 escape하도록 변경하고 query 특문 가능하도록 롤백

@Hank-Choi Hank-Choi requested review from PFCJeong and a team as code owners January 5, 2024 09:27
@Hank-Choi Hank-Choi requested review from davin111, Jhvictor4 and subeenpark-io and removed request for a team January 5, 2024 09:27
@davin111
Copy link
Member

davin111 commented Jan 5, 2024

@Hank-Choi #197 (comment) 여기서 리뷰해준 거랑 관련하는 거지? 요거 작업을 아직 안 해서 #197 release 를 안 하고 있었는데. 작업 땡큐.

Copy link
Member

@davin111 davin111 left a comment

Choose a reason for hiding this comment

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

그런데 . 이나 - 이나 space 외에 특수문자를 받아줄 필요가 있으려나? 저것만 허용해도 되지 않으려나.

lint 체크 필요할 듯.

@Hank-Choi Hank-Choi merged commit b82ec92 into develop Jan 5, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants