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

DBP: Enable DBP Operations to Run When Device is Locked #2797

Merged
merged 13 commits into from
Jun 3, 2024

Conversation

aataraxiaa
Copy link
Contributor

@aataraxiaa aataraxiaa commented May 20, 2024

Task/Issue URL: https://app.asana.com/0/1206488453854252/1206891169830209/f
Tech Design URL: https://app.asana.com/0/1206488453854252/1207313481959835/f

Description:
This PR changes the accessibility attribute of DBP Keychain items to enable them to be accessed when the device is locked. It also migrates Keychain values for existing DBP users.

** Testing Pre-requisites **

  1. Before launching the PR branch, launch from main
  2. Setup PIR if needed (e.g get a subscription)
  3. Run a scan, wait for results

Steps to test this PR:

Ensure Keychain Migration Works & Existing Data Is Retained

  1. Launch the Post-Implementation Build (Release)
  2. Open PIR
  3. Ensure that you see the results you previously received when running PIR with the main build

Ensure Scans Work

  1. Still in PIR, edit your profile to initiate a scan
  2. Ensure you see results and that the scan completes successfully

Ensure Scans Work When Device is Locked/Sleeping

Note: Device should be plugged in, and left for an hour or more (during lunch, after work etc.)

  1. Still in PIR, edit your profile to initiate a scan
  2. Lock your device or choose ‘Sleep’ from the Apple menu
  3. After more than 1 hour has passed, unlock your device
  4. Ensure you see results and that the scan completes successfully

Internal references:

Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation

@github-actions github-actions bot added the bot: not in app board Added by automation for pull requests with tasks not added to macOS App Board Asana project label May 20, 2024
@aataraxiaa aataraxiaa removed the bot: not in app board Added by automation for pull requests with tasks not added to macOS App Board Asana project label May 20, 2024
@aataraxiaa aataraxiaa requested a review from THISISDINOSAUR May 20, 2024 16:57
@aataraxiaa aataraxiaa marked this pull request as ready for review May 20, 2024 16:57
@THISISDINOSAUR
Copy link
Contributor

Screenshot 2024-05-21 at 8 34 17 PM I have bad news, this is what greeted me when I opened my laptop back up. I'm not sure why yet. It wasn't plugged in, so maybe that's why, I'll try again. It did continue at least

Copy link
Contributor

@THISISDINOSAUR THISISDINOSAUR left a comment

Choose a reason for hiding this comment

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

Okay yep that was it, looks good!

@@ -18,7 +18,11 @@

import Foundation

extension Bundle {
protocol GroupNameProviding {
Copy link
Contributor

Choose a reason for hiding this comment

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

This was very inevitable 😅

Copy link
Contributor

This PR has been inactive for more than 7 days and will be automatically closed 7 days from now.

@github-actions github-actions bot added the stale label May 31, 2024
# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
#	LocalPackages/DataBrokerProtection/Package.swift
#	LocalPackages/NetworkProtectionMac/Package.swift
#	LocalPackages/SubscriptionUI/Package.swift
@aataraxiaa aataraxiaa merged commit 9984e50 into main Jun 3, 2024
19 checks passed
@aataraxiaa aataraxiaa deleted the pete/dbp-locked-keychain-access-and-migration branch June 3, 2024 10:12
samsymons added a commit that referenced this pull request Jun 3, 2024
* main:
  Add optional "multiple" property to PageElement (#2827)
  Update BSK for iOS RMF changes (#2831)
  Bump version to 1.91.0 (197)
  Set marketing version to 1.91.0
  Update embedded files
  DBP: Enable DBP Operations to Run When Device is Locked (#2797)
  Update autoconsent to v10.9.0 (#2822)
samsymons added a commit that referenced this pull request Jun 4, 2024
* main:
  Add optional "multiple" property to PageElement (#2827)
  Update BSK for iOS RMF changes (#2831)
  Bump version to 1.91.0 (197)
  Set marketing version to 1.91.0
  Update embedded files
  DBP: Enable DBP Operations to Run When Device is Locked (#2797)
  Update autoconsent to v10.9.0 (#2822)
  Bump version to 1.90.0 (196)
  Revert "Add PIPAgent entitlement (#2821)" (#2823)
  Bump version to 1.90.0 (195)
  Bump version to 1.90.0 (194)
  Add PIPAgent entitlement (#2821)
  Fixes some VPN uninstallation issues (#2820)
  Bump version to 1.90.0 (193)
  Remove autofill survey (#2819)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants