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

LogClick 구현 #7

Merged
merged 3 commits into from
Jan 31, 2024
Merged

LogClick 구현 #7

merged 3 commits into from
Jan 31, 2024

Conversation

Hanna922
Copy link
Member

@Hanna922 Hanna922 commented Jan 29, 2024

1️⃣ 어떤 작업을 했나요? (Summary)

  • LogPayloadParams를 수정했습니다.

  • onClick event를 감지하는 LogClick을 구현했습니다.

  • 간단하게 테스트를 하려고 demo를 만들어뒀는데 나중에 삭제할게요!

  • 브랜치명이 #4인데... #6입니다....ㅎ

  • resolved LogClick 컴포넌트 개발 #6

3️⃣ 추후 작업

randomId 생성 작업을 해야할 것 같은데 구현 방식이 조금 고민이네요..!

로그인 한 사용자일 경우: userId를 받아 복호화 불가능한 hash 값을 구함
비로그인 사용자일 경우: random string 값을 바탕으로 hash 값을 구함
생성된 식별 불가 값(hash)은 localStorage에 저장

비로그인 사용자인데 local에 random 값이 존재한다면(이탈 전) 해당 값을 같이 태워서 백엔드로 보냄

이런 flow로 진행하면 좋을 것 같은데 어떻게 생각하시나용?

4️⃣ 체크리스트 (Checklist)

  • main 브랜치의 최신 코드를 pull 받았나요?

- 테스트 간단하게 하려고 demo 만들었는데 나중에 삭제할게요!
@Hanna922 Hanna922 self-assigned this Jan 29, 2024
@Hanna922 Hanna922 requested a review from JjungminLee January 29, 2024 10:24
name: string;
path?: string;
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기 path는 저희 웹 routing path고 LoggerType의 path는 어떻게 되는걸까요?

Copy link
Member Author

Choose a reason for hiding this comment

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

LoggerType의 path도 동일하게 routing path입니다!

Copy link
Collaborator

Choose a reason for hiding this comment

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

흠.. 그러면 LoggerType의 path없애는거 어떨까용?

Copy link
Member Author

Choose a reason for hiding this comment

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

엇 로그 내부 event에 path 들어가야 하는 거 아닌가요??

@JjungminLee
Copy link
Collaborator

로그인 한 사용자일 경우: userId를 받아 복호화 불가능한 hash 값을 구함
-> 좋아영
비로그인 사용자일 경우: random string 값을 바탕으로 hash 값을 구함
-> 얘는 복호화 가능인가요?
생성된 식별 불가 값(hash)은 localStorage에 저장
-> userId저장인가요 아니면 로그 저장인가요?

@Hanna922
Copy link
Member Author

Hanna922 commented Jan 31, 2024

@JjungminLee userId는 YLS에 들어오는 순간 복호화 불가능한 hash 값으로 변환되고, 스토리지에 저장하는 건 해당 hash 값이요! YLS에서 취급하는 모든 값들은 식별이 불가능해야 해서 전부 복호화 불가능이용

@JjungminLee
Copy link
Collaborator

생성된 식별 불가 값(hash)은 localStorage에 저장
-> userId저장인가요 아니면 로그 저장인가요?

@JjungminLee userId는 YLS에 들어오는 순간 복호화 불가능한 hash 값으로 변환되고, 스토리지에 저장하는 건 해당 hash 값이 포함된 log요! YLS에서 취급하는 모든 값들은 식별이 불가능해야 해서 전부 복호화 불가능이용

그러면 userId 복호화 불가능하게 만들고 이걸 로그 안에 넣어서 로컬스토리지에 넣는다는 말씀이죠? 그러면 로컬스토리지 key값도 hash값 쓰실건가요?

@Hanna922
Copy link
Member Author

@JjungminLee 넵 로그 내부 userId 값이 되는 겁니다! 로컬스토리지 키 값은 논의해서 정하면 좋을 것 같네용

@Hanna922 Hanna922 merged commit 4ac5e08 into develop Jan 31, 2024
1 check passed
@Hanna922 Hanna922 deleted the feat/#4 branch January 31, 2024 10:03
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.

LogClick 컴포넌트 개발
2 participants