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

Feat: Chat-133-테스트 페이지 추가 #10

Merged
merged 3 commits into from
Jan 7, 2024

Conversation

Mouon
Copy link
Member

@Mouon Mouon commented Jan 6, 2024

요약 (Summary)

  • [x]브렌치가 잘못 되어 다시 올립니다.
  • [x]chat.html추가
  • [x]UserController 리다이렉션 수정
  • [x]gradle파일에 의존성 추가

변경 사항 (Changes)

  • chat.html추가
  • UserController 리다이렉션 수정
  • gradle파일에 의존성 추가

리뷰 요구사항

  • 충돌 없이 잘 머지되는지
  • 테스트 페이지 정상 작동 여부

확인 방법 (선택)

chat.html추가
UserController 리다이렉션 수정
gradle파일에 의존성 추가
@Mouon Mouon changed the title Feat: CHAT-133-테스트 페이지 추가 Feat: Chat-133-테스트 페이지 추가 Jan 6, 2024
Copy link
Contributor

@nonaninona nonaninona left a comment

Choose a reason for hiding this comment

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

url 변경 후에도 테스트 페이지 잘 작동하는 것 확인했습니다!

git ignore 재적용하면서 application.yml이 제외되었는데,
원래 application.yml은 따로 공유하는 게 맞는 걸로 알고 있긴 합니다.
아니라면 알려주세요

@Kjiw0n
Copy link
Collaborator

Kjiw0n commented Jan 7, 2024

수고하셨습니다!
몇가지 부분에 관해 질문드립니다!

  • application.yml파일은 깃허브에 올린 후 민감정보는 환경변수 처리하는 것으로 알고있는데, 왜 제외하신건지 알 수 있을까요??
  • http://localhost:8080/testChat.html 테스트 페이지에서 메세지 전송 시, POST http://localhost:8080/testAsk 500 (Internal Server Error) 의 에러가 뜨는 것으로 확인됩니다. 추가로 필요한 설정이 있으면 말씀부탁드립니다.
  • 혹시 왜 콘솔창이 아닌 페이지 생성을 통해 서버 응답을 확인하신 특별한 이유가 있을까요??

수정 요청
ChatController에서 OPEN_AI_KEY가 환경변수 처리 부분이 주석처리되어있고 키 값이 그대로 깃허브에 올라와있습니다. 이 부분 확인 부탁드립니다!

@nonaninona
Copy link
Contributor

수고하셨습니다! 몇가지 부분에 관해 질문드립니다!

  • application.yml파일은 깃허브에 올린 후 민감정보는 환경변수 처리하는 것으로 알고있는데, 왜 제외하신건지 알 수 있을까요??
  • http://localhost:8080/testChat.html 테스트 페이지에서 메세지 전송 시, POST http://localhost:8080/testAsk 500 (Internal Server Error) 의 에러가 뜨는 것으로 확인됩니다. 추가로 필요한 설정이 있으면 말씀부탁드립니다.
  • 혹시 왜 콘솔창이 아닌 페이지 생성을 통해 서버 응답을 확인하신 특별한 이유가 있을까요??

수정 요청 ChatController에서 OPEN_AI_KEY가 환경변수 처리 부분이 주석처리되어있고 키 값이 그대로 깃허브에 올라와있습니다. 이 부분 확인 부탁드립니다!

  1. application.yml은 gitignore에서 기본적으로 build 폴더를 제외하기 때문에 같이 날아간거라
    gitignore 설정을 다시해주면 되긴 합니다

  2. 아마 제가 키값 공개하는 실수해서 그런거 같습니다

  3. 이건 왜 포스트맨 안쓰고 testChat 페이지를 만들었냐는 질문 같은데
    사실 저도 모릅니다... 현준이형이 실습해놓은 페이지 그냥 버리기가 아까워서도 있고
    제가 포스트맨을 잘 활용을 안해서일 수도...
    있든 없든 상관없어서 넣는게 좋다고 생각하고 있었습니다

저는 사실 백엔드팀의 결정사항에 따르는 거라 불필요하다고 생각되면 PR close 해도 상관없긴합니다

@Mouon
Copy link
Member Author

Mouon commented Jan 7, 2024

수고하셨습니다! 몇가지 부분에 관해 질문드립니다!

  • application.yml파일은 깃허브에 올린 후 민감정보는 환경변수 처리하는 것으로 알고있는데, 왜 제외하신건지 알 수 있을까요??
  • http://localhost:8080/testChat.html 테스트 페이지에서 메세지 전송 시, POST http://localhost:8080/testAsk 500 (Internal Server Error) 의 에러가 뜨는 것으로 확인됩니다. 추가로 필요한 설정이 있으면 말씀부탁드립니다.
  • 혹시 왜 콘솔창이 아닌 페이지 생성을 통해 서버 응답을 확인하신 특별한 이유가 있을까요??

수정 요청 ChatController에서 OPEN_AI_KEY가 환경변수 처리 부분이 주석처리되어있고 키 값이 그대로 깃허브에 올라와있습니다. 이 부분 확인 부탁드립니다!

  1. application.yml은 gitignore에서 기본적으로 build 폴더를 제외하기 때문에 같이 날아간거라
    gitignore 설정을 다시해주면 되긴 합니다
  2. 아마 제가 키값 공개하는 실수해서 그런거 같습니다
  3. 이건 왜 포스트맨 안쓰고 testChat 페이지를 만들었냐는 질문 같은데
    사실 저도 모릅니다... 현준이형이 실습해놓은 페이지 그냥 버리기가 아까워서도 있고
    제가 포스트맨을 잘 활용을 안해서일 수도...
    있든 없든 상관없어서 넣는게 좋다고 생각하고 있었습니다

저는 사실 백엔드팀의 결정사항에 따르는 거라 불필요하다고 생각되면 PR close 해도 상관없긴합니다

  1. 접근성이 쉬운 테스트 페이지가 있으면 좋겠다 싶어서 제작했지만, 저도 PM님 말씀처럼 백엔드팀장님의 결정사항에 따르는 거라 불필요하다고 생각되면 PR close 해도 상관없습니다
  2. application.yml은 gitignore에서 제외된거같은데 gitignore파일을 제가 따로 수정하지는 않아서 추가하는게 맞다면 추가해야할것 같습니다.

@nonaninona
Copy link
Contributor

일단 gitignore 업데이트해서 yml 들은 버전관리하도록 하겠습니다

이게 build에 있는 자질구레한 파일들이 git에 잡혀있어서 자꾸 커밋되는게 거슬려서 제외한거라...
yml을 관리하도록 할게유

키 값은... Secret에 관리하고 있는 걸로 아는데 일단 변경해야겠네요... 누구껄로하지..

@Mouon
Copy link
Member Author

Mouon commented Jan 7, 2024

일단 gitignore 업데이트해서 yml 들은 버전관리하도록 하겠습니다

이게 build에 있는 자질구레한 파일들이 git에 잡혀있어서 자꾸 커밋되는게 거슬려서 제외한거라... yml을 관리하도록 할게유

키 값은... Secret에 관리하고 있는 걸로 아는데 일단 변경해야겠네요... 누구껄로하지..

키 값은 제가 다시 발급해서 보내드렸습니다!

@nonaninona
Copy link
Contributor

아... 제가 바보네요... application.yml 무사합니다.

src에 있는 애들은 무사하고

build에 있는 애들을 없앤거예용...

@nonaninona nonaninona force-pushed the CHAT-133-API_테스트_페이지 branch from 5010e0d to d850757 Compare January 7, 2024 08:14
@Kjiw0n
Copy link
Collaborator

Kjiw0n commented Jan 7, 2024

키는 다시 발급받음 될것같고, yml파일들 정리 후 머지하면 될 것 같습니다!

Copy link
Collaborator

@Kjiw0n Kjiw0n left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

Copy link
Collaborator

@Kjiw0n Kjiw0n left a comment

Choose a reason for hiding this comment

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

Approve로 달아야 승인이 되네요 .. ㅎㅎ
수고하셨습니다!

@Kjiw0n Kjiw0n merged commit 30cbbeb into develop Jan 7, 2024
2 checks passed
@nonaninona nonaninona deleted the CHAT-133-API_테스트_페이지 branch February 4, 2024 20:17
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.

3 participants