Skip to content

chore: Consolidate SearchEngines policy into single page#142

Merged
bsmth merged 5 commits into
mozilla:mainfrom
bsmth:remove-dupes
May 12, 2026
Merged

chore: Consolidate SearchEngines policy into single page#142
bsmth merged 5 commits into
mozilla:mainfrom
bsmth:remove-dupes

Conversation

@bsmth
Copy link
Copy Markdown
Collaborator

@bsmth bsmth commented May 8, 2026

Description:

Testing with the following example:

{
  "policies": {
    "SearchEngines": {
      "Add": [
        {
          "Name": "Example1",
          "URLTemplate": "https://www.example.org/q={searchTerms}",
          "Method": "GET",
          "IconURL": "https://www.example.org/favicon.ico",
          "Alias": "example",
          "Description": "Description",
          "PostData": "name=value&q={searchTerms}",
          "SuggestURLTemplate": "https://www.example.org/suggestions/q={searchTerms}"
        }
      ],
      "Default": "Perplexity",
      "PreventInstalls": true,
      "Remove": ["Ecosia"]
    }
  }
}

Although noting PreventInstalls doesn't seem to be applying either on its own or as part of a larger example like this. Should confirm expectations about this one or if we need a bug to track it.

Motivation:

We currently have four pages for the SearchEngines policy, although it appears the options are not mutually exclusive.

Related issues and pull requests:

Exists from initial import based on https://mozilla.github.io/policy-templates/

Note: we could also do the same for Certificates and SanitizeOnShutdown.

Other questions

  • DefaultPrivate behavior
  • PreventInstalls behavior

@bsmth bsmth requested a review from mkaply May 8, 2026 12:52
@bsmth
Copy link
Copy Markdown
Collaborator Author

bsmth commented May 12, 2026

Merging now, thanks!

@bsmth bsmth merged commit fdfa7aa into mozilla:main May 12, 2026
1 check passed
@bsmth bsmth deleted the remove-dupes branch May 12, 2026 11:55
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