[PR #842] [MERGED] Add preferences parameter to url #991

New issue

Closed

opened 2026-02-25 20:37:20 +03:00 by kerem · 0 comments

kerem commented

2026-02-25 20:37:20 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/benbusby/whoogle-search/pull/842
Author: @jacr13
Created: 9/7/2022
Status: ✅ Merged
Merged: 9/22/2022
Merged by: @benbusby

Base: main ← Head: add-pref-url

📝 Commits (10+)

27e16e5 add support for config through url
5fa4272 make encrypted string shorter
90d1135 fix url description and add english translation
0405b97 add config variable
08c31de add brotli library to requirements
2b79356 add missing library for brotli
5e75fd7 add encryption only when option is set
0d31b1a Merge branch 'main' into add-pref-url
8f39087 fix encryption tag in preferences token
b26904b add new environment variables

📊 Changes

15 files changed (+249 additions, -39 deletions)

View changed files

📝 Dockerfile (+1 -1)
📝 README.md (+19 -17)
📝 app.json (+10 -0)
📝 app/models/config.py (+76 -3)
📝 app/routes.py (+10 -1)
📝 app/static/settings/translations.json (+54 -0)
📝 app/templates/header.html (+8 -2)
📝 app/templates/index.html (+11 -0)
📝 app/templates/opensearch.xml (+3 -0)
📝 app/utils/results.py (+4 -0)
📝 charts/whoogle/values.yaml (+17 -15)
📝 docker-compose-traefik.yaml (+2 -0)
📝 requirements.txt (+1 -0)
📝 test/test_misc.py (+27 -0)
📝 whoogle.template.env (+6 -0)

📄 Description

The preferences' parameter contain an encrypted version of the desired config of the user.

The encryption step is a choice and not required, as it is here a key is mandatory to show the new url in configuration.
If we decide to go without encryption, the preferences' parameter could work on different instances of whoogle. Maybe it is useful for people using farside to have it not encrypted?

The next possible step, would be to add a config variable to deactivate completely the cookies ?!

Fixes: #817

Translation needs to be done.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/benbusby/whoogle-search/pull/842 **Author:** [@jacr13](https://github.com/jacr13) **Created:** 9/7/2022 **Status:** ✅ Merged **Merged:** 9/22/2022 **Merged by:** [@benbusby](https://github.com/benbusby) **Base:** `main` ← **Head:** `add-pref-url` --- ### 📝 Commits (10+) - [`27e16e5`](https://github.com/benbusby/whoogle-search/commit/27e16e5e2c6330e3129dba51dd95d0d86a27f83b) add support for config through url - [`5fa4272`](https://github.com/benbusby/whoogle-search/commit/5fa4272484f0540c24da0db35b1ef68093282545) make encrypted string shorter - [`90d1135`](https://github.com/benbusby/whoogle-search/commit/90d1135904b3119ca19d92bf8bb20f99b22ccaab) fix url description and add english translation - [`0405b97`](https://github.com/benbusby/whoogle-search/commit/0405b97678139ca351d4081d0b2f831cfe7694a0) add config variable - [`08c31de`](https://github.com/benbusby/whoogle-search/commit/08c31de9aaddb6d694933bf940152fe615722a8b) add brotli library to requirements - [`2b79356`](https://github.com/benbusby/whoogle-search/commit/2b7935618c0bfa62b2cfbf634784fd1b3ed1b03f) add missing library for brotli - [`5e75fd7`](https://github.com/benbusby/whoogle-search/commit/5e75fd7ea8a18c3187a929d1c6d41dcf700b4641) add encryption only when option is set - [`0d31b1a`](https://github.com/benbusby/whoogle-search/commit/0d31b1aeeb4b3f825156402b102eae6b882d155c) Merge branch 'main' into add-pref-url - [`8f39087`](https://github.com/benbusby/whoogle-search/commit/8f39087ffb17424b332b82a47baefeb10910e1f3) fix encryption tag in preferences token - [`b26904b`](https://github.com/benbusby/whoogle-search/commit/b26904bd33fd24f8b74723789ce802ebe2d164f1) add new environment variables ### 📊 Changes **15 files changed** (+249 additions, -39 deletions) <details> <summary>View changed files</summary> 📝 `Dockerfile` (+1 -1) 📝 `README.md` (+19 -17) 📝 `app.json` (+10 -0) 📝 `app/models/config.py` (+76 -3) 📝 `app/routes.py` (+10 -1) 📝 `app/static/settings/translations.json` (+54 -0) 📝 `app/templates/header.html` (+8 -2) 📝 `app/templates/index.html` (+11 -0) 📝 `app/templates/opensearch.xml` (+3 -0) 📝 `app/utils/results.py` (+4 -0) 📝 `charts/whoogle/values.yaml` (+17 -15) 📝 `docker-compose-traefik.yaml` (+2 -0) 📝 `requirements.txt` (+1 -0) 📝 `test/test_misc.py` (+27 -0) 📝 `whoogle.template.env` (+6 -0) </details> ### 📄 Description The preferences' parameter contain an encrypted version of the desired config of the user. The encryption step is a choice and not required, as it is here a key is mandatory to show the new url in configuration. If we decide to go without encryption, the preferences' parameter could work on different instances of whoogle. Maybe it is useful for people using farside to have it not encrypted? The next possible step, would be to add a config variable to deactivate completely the cookies ?! Fixes: #817 Translation needs to be done. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>