[버그 리포트] SonarQube - 인증 코드(비밀번호) 생성 시 Random 말고 SecureRandom을 쓰자!

소나 큐브를 처음 써보는데 검사 결과 목록 중 바로 아래 그림과 같은 결과가 있었다. 뭔가 봤더니 인증 코드 생성하는 부분에서 Random()으로 생성하는 게 좋지 않다고 적혀있었다. 자세히 설명을 읽어보자.When software generates predictable values in a context requiring unpredictability, it may be possible for an attacker to guess the next value that will be generated, and use this guess to impersonate another user or access sensitive information.As the java.util.Random class relies..

• format_list_bulleted 버그리포트
• · 2024. 5. 7.
• textsms

[버그 리포트] access token 재발급 무한 요청 문제

문제 상황익명 메시지 프로젝트를 진행하는 과정에 access token 재발급 요청이 수십 건 들어오는 문제가 발생   원인코드를 다시 확인해 보니 아래 1~3번 과정이 계속 반복되며 무한 요청 문제가 발생하고 있었다. 1. 서버의 Filter에서 access token을 재발급하라고 return 시킴.2. 클라이언트는 access token 재발급 요청을 서버에게 보냄.3. 서버는 토큰이 만료됐다고  다시 return 시킴.     해결재발급 요청 URI 인 경우 Filter에서  리턴 시키지 않고 통과시키는 방식으로 해결!!   Filter에 아래 코드를 추가하여 문제 해결 // 건너뛸 URIprivate final List excludedUris = Arrays.asList("/api/v1/auth..

• format_list_bulleted 버그리포트
• · 2024. 5. 7.
• textsms

[SonarQube 06] JUnit 5에서는 Test class에 접근제한자 붙이지 말기

JUnit 4 에서는 접근제한자 "public"을 반드시 붙여야 했다. test 코드에서 public을 붙여서 코드를 작성했더니 아래와 같은 말을 들었다. JUnit5 is more tolerant regarding the visibilities of Test classes than JUnit4, which required everything to be public. In this context, JUnit5 test classes can have any visibility but private, however, it is recommended to use the default package visibility, which improves readability of code. Noncompliant Co..

• format_list_bulleted 버그리포트
• · 2023. 4. 27.
• textsms

[SonarQube 05] 길이가 늘어나는 문자열은 String 대신 StringBuilder를 쓰자.

소나큐브에서 아래와 같은 minor한 제안을 해주었다. Strings are immutable objects, so concatenation doesn’t simply add the new String to the end of the existing string. Instead, in each loop iteration, the first String is converted to an intermediate object type, the second string is appended, and then the intermediate object is converted back to a String. Further, performance of these intermediate operations degrad..

• format_list_bulleted 버그리포트
• · 2023. 3. 26.
• textsms

[SonarQube 02] "Random" objects should be reued

소나 큐브에 bug로 딱 하나 있었는데 그게 이 내용이다. 메서드가 호출될 때마다 Random() 객채를 생성하는 것은 비효율적이고 상수처럼 한번만 new 해놓고 가져다 쓰는 게 좋을 것 같다. Creating a new Random object each time a random value is needed is inefficient and may produce numbers which are not random depending on the JDK. For better efficiency and randomness, create a single Random, then store, and reuse it. The Random() constructor tries to set the seed with a d..

• format_list_bulleted 버그리포트
• · 2023. 3. 25.
• textsms