Skip to content

Feature(HK-165): 서비스 사용 가이드 모달 구현#72

Merged
juiuj merged 6 commits intomainfrom
feature/HK-165-service-modal
Sep 16, 2025
Merged

Feature(HK-165): 서비스 사용 가이드 모달 구현#72
juiuj merged 6 commits intomainfrom
feature/HK-165-service-modal

Conversation

@juiuj
Copy link
Copy Markdown
Member

@juiuj juiuj commented Sep 6, 2025

Motivation

  • HK-165 참고
  • 사용자가 키체인, SSH 연결 등 사용법을 확인하고자 함

Problem Solving

  • /dashboard 페이지에서 사용 가이드 모달 구현
스크린샷 2025-09-06 오후 11 08 21

@juiuj juiuj requested review from Jsplix and soooheeee September 6, 2025 14:11
@juiuj juiuj self-assigned this Sep 6, 2025
@juiuj juiuj added the enhancement New feature or request label Sep 6, 2025
@aws-amplify-ap-northeast-2
Copy link
Copy Markdown

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-72.d15ua1f8x2sllg.amplifyapp.com

Copy link
Copy Markdown
Contributor

@Jsplix Jsplix 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
Copy Markdown
Contributor

@soooheeee soooheeee left a comment

Choose a reason for hiding this comment

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

확인했습니다😎

@juiuj juiuj requested review from Jsplix and soooheeee September 15, 2025 15:39
Copy link
Copy Markdown
Contributor

@Jsplix Jsplix 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
Copy Markdown
Contributor

@soooheeee soooheeee left a comment

Choose a reason for hiding this comment

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

cookie 처리를 위해 js-cookie 라이브러리를 쓰신 것 같네요.
다만 js-cookie는 클라이언트 단에서 동작하기 때문에 HttpOnly 속성을 지정할 수 없기에
민감한 쿠키를 저장하면 XSS에 취약해질 수 있습니다...!

따라서 보안이 중요한 데이터에는 js-cookie 라이브러리를 지양하는 게 좋은 걸로 알고 있습니다!!
나머지 수정한 부분 확인했고 수고하셨습니다👍
(혹시 틀린 내용이면 지적해 주세요,,,🙀)

@juiuj
Copy link
Copy Markdown
Member Author

juiuj commented Sep 16, 2025

cookie 처리를 위해 js-cookie 라이브러리를 쓰신 것 같네요.

다만 js-cookie는 클라이언트 단에서 동작하기 때문에 HttpOnly 속성을 지정할 수 없기에

민감한 쿠키를 저장하면 XSS에 취약해질 수 있습니다...!

따라서 보안이 중요한 데이터에는 js-cookie 라이브러리를 지양하는 게 좋은 걸로 알고 있습니다!!

나머지 수정한 부분 확인했고 수고하셨습니다👍

(혹시 틀린 내용이면 지적해 주세요,,,🙀)

헉 이 부분은 몰랐는데 그렇군요..! 민감한 정보를 다룰 때 꼭 확인해보고 구현하겠습니다 유용한 피드백 감사합니다🥹

@juiuj juiuj merged commit ab50f71 into main Sep 16, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants