Skip to content

Conversation

JasonLovesDoggo
Copy link
Member

Closes #61
NOTE: #704 is a prereq (ish), I use that example in the docs but it's not functionally needed

Would like some feedback from the commenters of #61 on the implementation since it IS a redirect

Checklist:

  • Added a description of the changes to the [Unreleased] section of docs/docs/CHANGELOG.md
  • Added test cases to the relevant parts of the codebase
  • Ran integration tests npm run test:integration (unsupported on Windows, please use WSL)

@JasonLovesDoggo JasonLovesDoggo added enhancement New feature or request help wanted Extra attention is needed labels Jun 21, 2025
@JasonLovesDoggo JasonLovesDoggo changed the title feat: implement DENY_AND_REROUTE action to redirect to tarpits feat: implement a DENY_AND_REROUTE action to redirect to tarpits Jun 21, 2025
check-spelling run (pull_request) for feature/tarpit-express

Signed-off-by: check-spelling-bot <[email protected]>
on-behalf-of: @check-spelling <[email protected]>
@JasonLovesDoggo JasonLovesDoggo marked this pull request as ready for review June 21, 2025 03:52
s.respondWithError(w, r, "Internal Server Error: administrator has misconfigured Anubis. Please contact the administrator and ask them to look for the logs around \"maybeReverseProxy.RuleDenyAndReroute\"")
return true
}
// note for others, would it be better to be reverse proxying here?
Copy link
Member Author

Choose a reason for hiding this comment

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

I just had some concerns related to anubis performance + how constantly the scrapers would end up this path

@JasonLovesDoggo JasonLovesDoggo requested a review from Xe June 26, 2025 19:33
@JasonLovesDoggo
Copy link
Member Author

If someone wants this to be merged, shoot me a ping over

@JasonLovesDoggo JasonLovesDoggo marked this pull request as draft September 8, 2025 00:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request help wanted Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a target for unverified requests

3 participants