[PR #746] [MERGED] Tor password authentication #962

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

📋 Pull Request Information

Original PR: https://github.com/benbusby/whoogle-search/pull/746
Author: @MadcowOG
Created: 5/4/2022
Status: Merged
Merged: 6/16/2022
Merged by: @benbusby

Base: mainHead: tor-auth


📝 Commits (10+)

  • c56fce5 Added password authentication for tor control port.
  • b100e9d For user configuration of access to tor control port. This file should be heavily restricted in file system.
  • 1f9e07c Added message that will show tor auth failed.
  • 9745f2d Fixing for tests.
  • 3aa410a Fixing for test 2. Electric Booglaoo
  • 5f9d21e Another one
  • 69a35ef Maybe this will fix the issue with docker? But could present issues in the future.
  • ad8c4b9 Codestyle fix.
  • 135e062 Codestyle fix.
  • d98e436 Passed?

📊 Changes

4 files changed (+27 additions, -5 deletions)

View changed files

📝 README.md (+5 -3)
📝 app/request.py (+21 -1)
📝 app/routes.py (+0 -1)
misc/tor/control.conf (+1 -0)

📄 Description

Added the ability for user to authenticate tor control ports with passwords. It could be a better alternative to cookies, as passwords doesn't require changing file permissions.


🔄 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/746 **Author:** [@MadcowOG](https://github.com/MadcowOG) **Created:** 5/4/2022 **Status:** ✅ Merged **Merged:** 6/16/2022 **Merged by:** [@benbusby](https://github.com/benbusby) **Base:** `main` ← **Head:** `tor-auth` --- ### 📝 Commits (10+) - [`c56fce5`](https://github.com/benbusby/whoogle-search/commit/c56fce51ec13dafadc76a39081124955bb0092d2) Added password authentication for tor control port. - [`b100e9d`](https://github.com/benbusby/whoogle-search/commit/b100e9d6c9d13aacf76c95de04c3b7f47a687420) For user configuration of access to tor control port. This file should be heavily restricted in file system. - [`1f9e07c`](https://github.com/benbusby/whoogle-search/commit/1f9e07c4fe38ea6e02bf9a6a3849270e95fd7d85) Added message that will show tor auth failed. - [`9745f2d`](https://github.com/benbusby/whoogle-search/commit/9745f2d2564f5557294c12a7036218bf2b6d5ded) Fixing for tests. - [`3aa410a`](https://github.com/benbusby/whoogle-search/commit/3aa410a2e6752c1de62acca4f7894e7888045216) Fixing for test 2. Electric Booglaoo - [`5f9d21e`](https://github.com/benbusby/whoogle-search/commit/5f9d21ebfd1bb41efcaf37cdfa7aa9a2048c7a6c) Another one - [`69a35ef`](https://github.com/benbusby/whoogle-search/commit/69a35efb16150967cafdc7516175ed7267451e47) Maybe this will fix the issue with docker? But could present issues in the future. - [`ad8c4b9`](https://github.com/benbusby/whoogle-search/commit/ad8c4b93de26b1ff02cde93d62f6b0dcc368ea25) Codestyle fix. - [`135e062`](https://github.com/benbusby/whoogle-search/commit/135e062b127eeefe710076df34bc8f82e5403801) Codestyle fix. - [`d98e436`](https://github.com/benbusby/whoogle-search/commit/d98e436941e8b9dfe5bb44967741f22ffd42826f) Passed? ### 📊 Changes **4 files changed** (+27 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+5 -3) 📝 `app/request.py` (+21 -1) 📝 `app/routes.py` (+0 -1) ➕ `misc/tor/control.conf` (+1 -0) </details> ### 📄 Description Added the ability for user to authenticate tor control ports with passwords. It could be a better alternative to cookies, as passwords doesn't require changing file permissions. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-25 20:37:14 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/whoogle-search#962
No description provided.