Skip to content
This repository was archived by the owner on Feb 24, 2025. It is now read-only.

Conversation

@diegoreymendez
Copy link
Contributor

@diegoreymendez diegoreymendez commented Aug 30, 2023

Task/Issue URL: https://app.asana.com/0/0/1205381920971737/f
macOS PR: duckduckgo/macos-browser#1563
iOS PR: duckduckgo/iOS#1966

Description

Fixes some auth token regressions. See the parent task for background info.

The issue is that our system extension still needs per-build service names, while our App and agent don't (and are better off without them).

Steps to test this PR

Use the macOS PR to test this.


Internal references:

Software Engineering Expectations
Technical Design Template

Copy link
Contributor

@samsymons samsymons left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

diegoreymendez added a commit to duckduckgo/macos-browser that referenced this pull request Aug 30, 2023
Task/Issue URL: https://app.asana.com/0/0/1205381920971736/f
BSK PR: duckduckgo/BrowserServicesKit#484
iOS PR: duckduckgo/iOS#1966

## Description

Fixes some auth token regressions. See the parent task for background
info.

The issue is that our system extension still needs per-build service
names, while our App and agent don't (and are better off without them).
diegoreymendez added a commit to duckduckgo/iOS that referenced this pull request Aug 30, 2023
@diegoreymendez diegoreymendez merged commit cbf3c06 into main Aug 30, 2023
@diegoreymendez diegoreymendez deleted the diego/fix-auth-token-issues branch August 30, 2023 19:38
samsymons added a commit that referenced this pull request Aug 30, 2023
* main:
  Fix auth token issues (#484)
  Enable nightly CodeQL scans (#481)
  add assertions to AdClickAttributionLogic, improve debug checks (#476)
  Set unencrypted password for deduplicated Credentials object (#480)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants