Skip to content
This repository was archived by the owner on Jun 20, 2023. It is now read-only.

Conversation

@d4rken
Copy link
Member

@d4rken d4rken commented Aug 18, 2020

I came upon this while researching various avenues that could cause a 9002 error.

Resetting the preferences deletes our database encryption credentials, but we don't delete the database itself.
On app restart, there is a mismatch between the newly generated credentials and the old database.

I also annotated resetInstance as it should really only be used for testing (and currently only is).

…, but we don't delete the database itself.

On app restart, there is a mismatch between the newly generated credentials and the old database.
@d4rken d4rken added the maintainers Tag pull requests created by maintainers label Aug 18, 2020
@d4rken d4rken requested review from a team and jakobmoellerdev August 18, 2020 11:22
The previous commit accidentally used the transitive dependency from google-utils.
@d4rken d4rken changed the title Fixes 9002 error caused by test fragment risk reset Fixes 9002 error caused by test fragment risk reset (related to EXPOSUREAPP-1851) Aug 18, 2020
@d4rken d4rken requested a review from jakobmoellerdev August 18, 2020 12:18
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@jakobmoellerdev jakobmoellerdev merged commit 8f9f27c into dev Aug 19, 2020
@jakobmoellerdev jakobmoellerdev deleted the fix/risk-calculation-test-reset-causing-9002 branch August 19, 2020 07:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

maintainers Tag pull requests created by maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants