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

Don't support transferring Ruuvi Station data on device change #1852

Merged
merged 1 commit into from
Jan 9, 2024

Conversation

rinat-enikeev
Copy link
Contributor

  • introduces the migrateIfNeeded function, designed to modify the backup behavior of a specific SQLite database file used within the app. The primary objective is to ensure that this database file is not included in the iCloud backups.

  • introduces a change in the configuration of a Keychain instance used in the app, specifically targeting the way the app handles the accessibility of the stored keychain items.

The significant change in this pull request is the modification of the keychain's accessibility attribute from .afterFirstUnlock to .afterFirstUnlockThisDeviceOnly. This change impacts when the keychain items become accessible:

Previously: .afterFirstUnlock - The keychain items were accessible after the device was first unlocked following a restart. This level allows the items to be accessible even when the device is locked again, as long as it hasn't been restarted.

Now: .afterFirstUnlockThisDeviceOnly - This setting is similar to .afterFirstUnlock in terms of accessibility after the device's first unlock. However, it adds an additional restriction that the items are only accessible on the current device. This means the keychain items are not included in device backups and cannot be restored to a new device.

@rinat-enikeev rinat-enikeev requested a review from priyonto January 9, 2024 20:35
Copy link

sonarqubecloud bot commented Jan 9, 2024

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

1 New issue
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Collaborator

@priyonto priyonto left a comment

Choose a reason for hiding this comment

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

Nice 🙏

@rinat-enikeev rinat-enikeev merged commit df5e3ae into dev Jan 9, 2024
2 checks passed
@rinat-enikeev rinat-enikeev deleted the rinat/1848 branch January 9, 2024 20:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants