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

Fix release branch VPN auth issues #2903

Merged
merged 1 commit into from
May 30, 2024

Conversation

samsymons
Copy link
Contributor

…oblem.

Task/Issue URL: https://app.asana.com/0/414235014887631/1207438971509836/f
Tech Design URL:
CC:

Description:

This PR fixes an issue with VPN authentication, where the app was keeping a value of false for isSubscriptionAvailable if you launched the app when signed out. This should be fixed by removing all isSubscriptionAvailable checks soon, but for now let's fix the bug by hardcoding this to true.

Steps to test this PR:

  1. Launch the app while signed out of Privacy Pro
  2. Sign into Privacy Pro
  3. Verify that the VPN connects and does not give you any auth token errors

Device Testing:

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

OS Testing:

  • iOS 14
  • iOS 15
  • iOS 16

Internal references:

Software Engineering Expectations
Technical Design Template

@samsymons samsymons requested a review from diegoreymendez May 30, 2024 03:53
isSubscriptionEnabled: accountManager.isUserAuthenticated,
isSubscriptionEnabled: true,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was the core issue, this was being used for the lifecycle of the app, so if you launched the app when you were not signed in, we didn't go through the subscription flow. Since the subscription is always enabled (this means "has launched", not "is signed in"), let's just hardcode this to true until we can remove this code completely.

Copy link
Contributor

@diegoreymendez diegoreymendez left a comment

Choose a reason for hiding this comment

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

Tested and this works perfectly for me.

@diegoreymendez diegoreymendez merged commit d5722e7 into release/7.122.0 May 30, 2024
18 checks passed
@diegoreymendez diegoreymendez deleted the sam/release-pr-fix-vpn-auth-failure branch May 30, 2024 09:16
samsymons added a commit that referenced this pull request May 31, 2024
* main:
  Update VPN support copy (#2894)
  Settings experiment disabled and new Settings rolled out to all users (#2904)
  Release 7.122.0-2 (#2909)
  PrivacyPro terms of service link (#2907)
  Release 7.122.0-1 (#2906)
  Fix release branch VPN auth issues (#2903)
  Disable the autofill survey (#2899)
  fire RMF message updated notification if we make any change to the store (#2891)
  Fix maestro config so it runs (#2896)
  Disable the autofill survey (#2899)
  Release 7.122.0-0 (#2898)
  Do not refresh collection view when we are changing the display mode (#2895)
  Add option to crash the app with exception from debug menu (#2897)
  Remove default.profraw (#2893)
  Guard swipe tabs scroll state (#2888)
samsymons added a commit that referenced this pull request Jun 1, 2024
# By bwaresiak (6) and others
# Via Bartek Waresiak (1) and others
* main: (30 commits)
  Update VPN support copy (#2894)
  Settings experiment disabled and new Settings rolled out to all users (#2904)
  Release 7.122.0-2 (#2909)
  PrivacyPro terms of service link (#2907)
  Release 7.122.0-1 (#2906)
  Fix release branch VPN auth issues (#2903)
  Disable the autofill survey (#2899)
  fire RMF message updated notification if we make any change to the store (#2891)
  Fix maestro config so it runs (#2896)
  Disable the autofill survey (#2899)
  Release 7.122.0-0 (#2898)
  Do not refresh collection view when we are changing the display mode (#2895)
  Add option to crash the app with exception from debug menu (#2897)
  Remove default.profraw (#2893)
  Guard swipe tabs scroll state (#2888)
  Add debug pixels for data cleanup (#2890)
  Bump rexml from 3.2.6 to 3.2.8 (#2866)
  Remove VPN waitlist code (#2795)
  Privacy Pro Follow ups (#2882)
  Clean up subscriptionFeature on init (#2880)
  ...

# 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