[Hotfix] KeyStoreException #469
Conversation
Summary of ChangesHello @HI-JIN2, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 KeyStoreException으로 인해 앱이 비정상적으로 종료되는 문제를 해결하기 위한 중요한 업데이트입니다. 손상된 암호화된 데이터를 감지하고 자동으로 복구하는 메커니즘을 도입하여 사용자 경험을 개선하고 앱의 안정성을 높였습니다. 이 변경으로 인해 오류 발생 시 암호화된 사용자 데이터는 초기화되지만, 앱은 계속해서 정상적으로 작동할 수 있게 됩니다. Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Changelog
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
이 PR은 KeyStoreException 발생 시 앱이 강제 종료되는 문제를 해결하기 위해 SecurePrefs 초기화 로직을 개선하는 중요한 수정입니다. 예외 발생 시 데이터를 삭제하고 재시도하는 접근 방식은 올바르다고 생각합니다. 코드의 안정성과 유지보수성을 더욱 높이기 위해 몇 가지 제안 사항을 남깁니다. 주요 내용은 재시도 로직의 실패 케이스 처리, 더 구체적인 예외 처리, 그리고 상수를 사용한 코드 개선에 관한 것입니다.
| } catch (e: Exception) { | ||
| Timber.e(e, "Failed to create SecurePrefs. Resetting...") | ||
| FirebaseCrashlytics.getInstance().recordException(e) | ||
|
|
||
| // Clear data | ||
| context.deleteSharedPreferences("secure_prefs") | ||
|
|
||
| // Retry | ||
| createSecurePrefs(context) | ||
| } |
There was a problem hiding this comment.
이 catch 블록에는 두 가지 잠재적인 문제가 있습니다.
-
재시도 실패 시 앱 충돌 (심각도: Critical): 31번째 줄의
createSecurePrefs(context)재시도가 실패할 경우, 해당 예외가 처리되지 않아 앱이 충돌하게 됩니다. 이는 핫픽스의 주 목적인 '앱 충돌 방지'에 위배됩니다. 재시도 로직을try-catch로 한번 더 감싸서, 두 번째 실패 시에는 명확한 예외를 던지도록 처리해야 합니다. -
일반적인
Exception처리 (심각도: High):KeyStoreException과 같은 특정 예외가 아닌 일반Exception을 잡게 되면, 예상치 못한 다른 종류의 예외(예:NullPointerException)까지 이 복구 로직을 타게 되어 의도치 않은 데이터 삭제를 유발할 수 있습니다.when절을 사용해GeneralSecurityException이나IOException같은 예상된 예외에 대해서만 복구 로직을 수행하고, 그 외에는 예외를 다시 던져서 문제를 파악할 수 있도록 하는 것이 더 안전합니다.
아래는 이 두 가지 문제를 모두 수정한 제안 코드입니다. 참고로, 제안된 코드를 적용하려면 java.io.IOException과 java.security.GeneralSecurityException을 import해야 할 수 있습니다.
} catch (e: Exception) {
when (e) {
is java.security.GeneralSecurityException, is java.io.IOException -> {
Timber.e(e, "Failed to create SecurePrefs. Resetting...")
FirebaseCrashlytics.getInstance().recordException(e)
// Clear data
context.deleteSharedPreferences("secure_prefs")
// Retry
try {
createSecurePrefs(context)
} catch (retryException: Exception) {
Timber.e(retryException, "Failed to create SecurePrefs even after reset.")
FirebaseCrashlytics.getInstance().recordException(retryException)
throw IllegalStateException("Could not initialize SecurePrefs after reset.", retryException)
}
}
else -> throw e
}
}| FirebaseCrashlytics.getInstance().recordException(e) | ||
|
|
||
| // Clear data | ||
| context.deleteSharedPreferences("secure_prefs") |
There was a problem hiding this comment.
하드코딩된 문자열 "secure_prefs"가 28라인과 41라인 두 곳에서 사용되고 있습니다. 이 문자열을 object 내에 private const val로 선언하여 재사용하면 오타 발생 가능성을 줄이고 유지보수성을 높일 수 있습니다.
예시:
object SecurePrefsModule {
private const val SECURE_PREFS_FILENAME = "secure_prefs"
// ...
}1 participant
Summary
KeyStoreException (특히 "Signature/MAC verification failed") 발생 시 앱이 강제 종료되지 않고, 손상된 키나 데이터를 삭제한 후 재설정(Graceful Degradation)하여 앱을 정상적으로 사용할 수 있도록 합니다.
키 저장소 오류가 발생하면 저장된 모든 암호화된 데이터(로그인 토큰 등)가 초기화됩니다.
Describe your changes
KeyStoreException 또는 GeneralSecurityException 발생 시
Issue
To reviewers