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

[SKRB-152] feat: 암호화를 통한 develop 환경 서브모듈로 부터 제거 #12

Merged
merged 5 commits into from
Oct 25, 2023

Conversation

juno-junho
Copy link
Member

@juno-junho juno-junho commented Oct 24, 2023

💠 Jira 티켓 링크

🖥️ 작업 내용


✅ PR시 확인 사항

  • 테스트 코드 작성
  • Linear History 여부

📢 리뷰어 전달 사항

  • global.utils 패키지 안에 패스워드를 통해서 암호화할수 있습니다.
  • develop 브랜치 배포 시 한번 다시 확인해봐야 할것 같습니다

@juno-junho juno-junho self-assigned this Oct 24, 2023
@juno-junho juno-junho added the FEAT feature label Oct 25, 2023
Copy link
Collaborator

@hyeon-z hyeon-z 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
Member

@choi5798 choi5798 left a comment

Choose a reason for hiding this comment

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

수고하셨어요~ 코멘트 몇 개 남겼습니다

Comment on lines 11 to 13
@Configuration
@Profile("develop")
public class PropertiesConfig {
Copy link
Member

Choose a reason for hiding this comment

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

저희 어노테이션 길이 정렬 컨벤션 맞춰주세요

String password = "";
String urlResult = encryptor.encrypt(url);
String usernameResult = encryptor.encrypt(username);
String passwordReesult = encryptor.encrypt(password);
Copy link
Member

Choose a reason for hiding this comment

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

변수명에 Reesult는 오타인가요? 아니면 다른 뜻이 있는건가요?

Copy link
Member Author

Choose a reason for hiding this comment

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

오타에요 ㅋㅋ

Comment on lines +3 to +5
url: ENC(otDA9awhWaUEIpsnbNeFYbtGftRKQJGT0sJCi+wsST/vLIAY7qgtfWUJ5DboDjtm75FCMg9NpMIO51aQRFjS9cjjelWOUltFnMEd2c295NvXg8xVH7Xm3eoDuXuSjWRH)
username: ENC(zmfwreAfhObIQDyz369dXg==)
password: ENC(u5djZgMAhFCL4roMXqGn8j3oj6AEcxh0)
Copy link
Member

Choose a reason for hiding this comment

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

ENC로 감싸진 해시값은 실제 값과는 다른 값인가요?

Copy link
Member Author

Choose a reason for hiding this comment

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

ENC로 감싸진 해시값은 실제 값을 위 entryptionTest를 통해서 암호화를 시킨 값이에요
비밀번호는 dm으로 보내 드릴게요

@juno-junho juno-junho merged commit a0b645e into develop Oct 25, 2023
1 check passed
@juno-junho juno-junho deleted the feat/SKRB-152 branch October 25, 2023 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FEAT feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants