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

Add NavigationPreferences CustomHeaderFields #2147

Merged
merged 10 commits into from
Feb 7, 2024
Merged

Conversation

mallexxx
Copy link
Collaborator

@mallexxx mallexxx commented Feb 2, 2024

Task/Issue URL: https://app.asana.com/0/0/1206426619289877/f
BSK PR: duckduckgo/BrowserServicesKit#651

Description:

  • Adds custom header fields for SERP and GPC without request redirect

Steps to test this PR:

  1. Validate X-DuckDuckGo-Client header is passed to SERP (e.g. by adding || navigationAction.url.host == "myhttpheader.com at SerpHeadersNavigationResponder.swift:30 and navigating to https://myhttpheader.com/

Internal references:

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

@mallexxx mallexxx requested a review from tomasstrba February 2, 2024 11:26
@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 Feb 2, 2024
@github-actions github-actions bot removed the bot: not in app board Added by automation for pull requests with tasks not added to macOS App Board Asana project label Feb 2, 2024
Copy link
Contributor

@tomasstrba tomasstrba left a comment

Choose a reason for hiding this comment

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

LGTM! ✅ X-DuckDuckGo-Client is present, GPC signal is detected too 👍

mallexxx added a commit to duckduckgo/BrowserServicesKit that referenced this pull request Feb 7, 2024
Task/Issue URL: https://app.asana.com/0/0/1206426619289877/f
iOS PR: not affected
macOS PR: duckduckgo/macos-browser#2147

Includes  #644 changes except Content-Scope-Scripts rev change: 
Task/Issue URL: https://app.asana.com/0/72649045549333/1203487090719123/f
iOS PR: - not affected
@mallexxx mallexxx merged commit 23e491d into main Feb 7, 2024
17 checks passed
@mallexxx mallexxx deleted the alex/custom-header-fields branch February 7, 2024 06:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants