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

Update VPN auth token check logic #3585

Merged
merged 2 commits into from
Nov 18, 2024

Conversation

samsymons
Copy link
Contributor

Task/Issue URL: https://app.asana.com/0/1207603085593419/1208194658564979/f
Tech Design URL:
CC:

Description:

This PR updates the auth token check logic.

Steps to test this PR:

  1. Run this branch on a device and then enable the VPN
  2. Shut down your device then relaunch it, and wait a minute or so before unlocking it
  3. Unlock it, and check that the VPN is reconnected within ~20 seconds after doing so

Definition of Done (Internal Only):

Copy Testing:

  • Use of correct apostrophes in new copy, ie rather than '

Orientation Testing:

  • Portrait
  • Landscape

Device Testing:

  • iPhone SE (1st Gen)
  • iPhone 8
  • iPhone X
  • iPhone 14 Pro
  • iPad

OS Testing:

  • iOS 15
  • iOS 16
  • iOS 17

Theme Testing:

  • Light theme
  • Dark theme

Internal references:

Software Engineering Expectations
Technical Design Template

diegoreymendez pushed a commit to duckduckgo/BrowserServicesKit that referenced this pull request Nov 18, 2024
<!--
Note: This checklist is a reminder of our shared engineering
expectations.
-->

Please review the release process for BrowserServicesKit
[here](https://app.asana.com/0/1200194497630846/1200837094583426).

**Required**:

Task/Issue URL:
https://app.asana.com/0/1207603085593419/1208194658564979/f
iOS PR: duckduckgo/iOS#3585
macOS PR: duckduckgo/macos-browser#3562
What kind of version bump will this require?: Patch

**Description**:

This PR updates the logic used when the auth token startup check fails.
@diegoreymendez diegoreymendez merged commit 52b2b79 into main Nov 18, 2024
13 checks passed
@diegoreymendez diegoreymendez deleted the sam/avoid-disabling-vpn-after-reboot branch November 18, 2024 11:06
mgurgel pushed a commit to duckduckgo/BrowserServicesKit that referenced this pull request Nov 18, 2024
<!--
Note: This checklist is a reminder of our shared engineering
expectations.
-->

Please review the release process for BrowserServicesKit
[here](https://app.asana.com/0/1200194497630846/1200837094583426).

**Required**:

Task/Issue URL:
https://app.asana.com/0/1207603085593419/1208194658564979/f
iOS PR: duckduckgo/iOS#3585
macOS PR: duckduckgo/macos-browser#3562
What kind of version bump will this require?: Patch

**Description**:

This PR updates the logic used when the auth token startup check fails.
samsymons added a commit that referenced this pull request Nov 21, 2024
…g-error

# By Mariusz Śpiewak (4) and others
# Via GitHub (1) and Mariusz Śpiewak (1)
* main:
  Re-prompt for crash reporting (#3595)
  Fix running BoolFileMarkerTest on device (#3601)
  Run Sync e2e tests only on latest version (#3602)
  Release 7.146.0-1 (#3600)
  Fix autofill search authentication looping (#3598)
  Fix invalid OmniBar state transition on loading  (#3599)
  Update autoconsent to v11.5.0 (#3581)
  Release 7.146.0-0 (#3594)
  deps: Bump CSS@6.36.0 and BSK@211 (#3588)
  Update subtitle (#3592)
  Limits TipKit availability to iOS 18+ (#3589)
  Fix alpha build compilation (#3584)
  Update VPN auth token check logic (#3585)
  Fix button shapes on New Tab Page (#3586)
  Temporarily disable TipKit (#3564)

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
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