Feat/recommendation feedback - #28
Merged
Merged
Conversation
1. 추천 피드백 별점(1-5)과 서술형 의견(300자)을 RecommendationFeedback 으로 구조화 저장. 전송은 기존 /v1/recommendation-events 를 그대로 쓰고 서버가 옮겨 담는다. recommendationContextSchema 에 source 가 없어서 앱이 보낸 값이 400도 없이 버려지던 것도 함께 고쳤다. 2. env.ts https 게이트 완화 production 에서 평문 ML 주소를 통째로 undefined 로 만들어, echo 배포 (host.docker.internal:8000)에서 리캡 배경이 조용히 안 뜬다. docker0 브리지는 호스트를 안 벗어나므로 공개망 평문만 막고 내부 주소는 허용한다. 3. openapi YAML 파서 오류 수정 placeName description 의 '(예: 광안리해수욕장)' 이 따옴표 없이 들어가 콜론+공백을 매핑으로 해석해 문서 전체가 파싱되지 않았다. 4. docker-compose.override.yml 추가 (echo 전용) docker-compose.yml 은 environment: 에 나열된 키만 넘기므로 MODERATION_* 등 신규 변수가 컨테이너에 도달하지 않는다. env_file 로 .env 를 주입한다. 5. 저장소에 커밋돼 있던 soundlogserver-ml-recap.patch 제거
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
무엇
함께 고친 것
env.tshttps 게이트가 production에서 평문 ML 주소를 통째로 버려서,echo 배포(host.docker.internal:8000)에서 ML 기능이 에러도 로그도 없이
폴백으로 내려갔다. 공개망 평문만 막고 내부망은 허용하도록 완화.
recommendationContextSchema에source가 없어 앱이 보낸 값이 400도 없이버려지던 문제. zod가 모르는 키를 조용히 버리고 validate 미들웨어가
req.body를 파싱 결과로 교체하기 때문.
openapi/soundlog-api.yaml이 파싱되지 않던 문제. placeName description의(예: 광안리해수욕장)이 따옴표 없이 들어가 콜론+공백을 매핑으로 해석.검증
check:openapi-sync69 operations 통과피드백 저장 확인 (기존 데이터 보존)