로컬 디비 생성 로직 및 utils, 암호화 부분 생성 - #32
Merged
Merged
Conversation
apaals2
reviewed
Aug 1, 2025
Contributor
There was a problem hiding this comment.
_key 값은 추후에 환경변수로 분리하실 예정이시죠?
Contributor
Author
There was a problem hiding this comment.
- 해당 부분은 환경 변수로 분리하면 좋으나 .env 파일로 해당 앱에 저장해도 노출되는 문제가 존재하여 고민이 필요할 거 같습니다.
Contributor
Author
There was a problem hiding this comment.
uuid는 auto_increment로 진행되는 부분을 uuid로 진행할지 고민중이라 일단 만들어놨습니다.
아래 장단점 적어놓겠습니다.
- auto_increment
- 사용하기 매우 쉽고 직관적
- 정수형이라 인덱싱과 조인속도가 UUID보다 빠름
- 저장 공간을 적게 차지
- ID가 순차적으로 증가하여 외부로 노출될 경우 다음 ID를 쉽게 예측
- 여러 데이터베이스의 내용을 하나로 합칠 때 ID가 충돌
- uuid
- 다른 시스템과 동기화하거나 병합할 때 매우 안전
- ID를 추측할 수 없어 보안에 더 유리
- 애플리케이션 코드에서 ID를 미리 만들어 데이터베이스에 삽입
- 문자열이라 상대적으로 저장 공간을 더 많이 차지
- 정수보다 인덱싱과 조인 속도가 느릴 수 있음
- 향후 기능 확장이나 데이터 동기화 가능성시 유리
Contributor
There was a problem hiding this comment.
그러면 로컬 db에 값 추가(생성)할때 generate_prefixed_uuid 사용하면 되는건가요?
Contributor
There was a problem hiding this comment.
ConnectionProfile 테이블 생성안의 name이 profile_name인거죠?
나머지 DB 테이블들은 추후 추가 ?
Contributor
Author
There was a problem hiding this comment.
- 네, profile_name 맞습니다. 디비 자체가 프로필 정보를 담는 부분이기 때문에 프로필을 제거한 상태입니다.
- 나머지 테이블도 다 만들고 올리려고 합니다. 시간이 없어 해당 부분 진행하다가 멈춘 상태입니다.
ChoiSeungWoo98
commented
Aug 2, 2025
Contributor
Author
There was a problem hiding this comment.
- 해당 부분은 환경 변수로 분리하면 좋으나 .env 파일로 해당 앱에 저장해도 노출되는 문제가 존재하여 고민이 필요할 거 같습니다.
Contributor
Author
There was a problem hiding this comment.
uuid는 auto_increment로 진행되는 부분을 uuid로 진행할지 고민중이라 일단 만들어놨습니다.
아래 장단점 적어놓겠습니다.
- auto_increment
- 사용하기 매우 쉽고 직관적
- 정수형이라 인덱싱과 조인속도가 UUID보다 빠름
- 저장 공간을 적게 차지
- ID가 순차적으로 증가하여 외부로 노출될 경우 다음 ID를 쉽게 예측
- 여러 데이터베이스의 내용을 하나로 합칠 때 ID가 충돌
- uuid
- 다른 시스템과 동기화하거나 병합할 때 매우 안전
- ID를 추측할 수 없어 보안에 더 유리
- 애플리케이션 코드에서 ID를 미리 만들어 데이터베이스에 삽입
- 문자열이라 상대적으로 저장 공간을 더 많이 차지
- 정수보다 인덱싱과 조인 속도가 느릴 수 있음
- 향후 기능 확장이나 데이터 동기화 가능성시 유리
Contributor
Author
There was a problem hiding this comment.
- 네, profile_name 맞습니다. 디비 자체가 프로필 정보를 담는 부분이기 때문에 프로필을 제거한 상태입니다.
- 나머지 테이블도 다 만들고 올리려고 합니다. 시간이 없어 해당 부분 진행하다가 멈춘 상태입니다.
ChoiSeungWoo98
force-pushed
the
feature/local-db-setup
branch
from
August 2, 2025 06:46
1c3a220 to
6bb4811
Compare
ChoiSeungWoo98
force-pushed
the
feature/local-db-setup
branch
from
August 3, 2025 15:28
91f1966 to
416235e
Compare
Contributor
|
sqlite 관련 내용(사용법 등) readme에 추가 하실 예정인가요? |
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
2 participants
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.
로컬 디비 생성
utils 및 암호화 부분 생성