Skip to content

feat(proxy): add anubis support#1122

Open
RedstoneWizard08 wants to merge 1 commit into
tobychui:mainfrom
RedstoneWizard08:main
Open

feat(proxy): add anubis support#1122
RedstoneWizard08 wants to merge 1 commit into
tobychui:mainfrom
RedstoneWizard08:main

Conversation

@RedstoneWizard08

Copy link
Copy Markdown

This adds support for Anubis for native AI crawler and scraper defense. See the website for more.

It integrates support directly into the settings for routes and serves the verification challenge before continuing to proxy.

@RedstoneWizard08

Copy link
Copy Markdown
Author

(Force pushed for GPG signing reasons)

@SeychellesDev

Copy link
Copy Markdown

furries on da TBS... it's peak...

@tobychui

Copy link
Copy Markdown
Owner

Thanks for the furry PR!

I see you have changed something related to the HTTPServe flow and do some tidy up as well, but since all of them are in the same PR, it is really hard to read.

Maybe I will review this in the coming weekend, it is not something I can review within a weekday night.

Also, a few questions:

  1. Might I know do I need to wait for the PR to be merged on the Anubis side before I can merge this?
  2. Is this vibe coded? If yes, please let me know which AI model you used so I don't use the same model for code review

Thanks 👍🏻

@RedstoneWizard08

Copy link
Copy Markdown
Author
  1. Probably, the submodule is on my fork for now, but yeah.
  2. This is absolutely not vibe coded. I did this by hand (hence why it's kinda cursed lmao)

@RedstoneWizard08

Copy link
Copy Markdown
Author

I can try to clean up the code a bit, sorry I was doing this kinda quickly :P

@RedstoneWizard08

Copy link
Copy Markdown
Author

Also I should clarify that the cleanup I did was necessary to make this work - essentially I have to pass around a handler pointer which is kinda cursed but it works

@AnthonyMichaelTDM

Copy link
Copy Markdown
Collaborator

I think part of what's complicating things is that there seem to be a decent number of unrelated changes included in this PR

@RedstoneWizard08

Copy link
Copy Markdown
Author

Hi, sorry, I've been insanely busy with school and everything so I haven't had a ton of time - when I get the chance I'll try to clean up the code a bit. I tried not to modify anything I didn't need to for this, but if there's anything that shouldn't have been touched, I will do my best to revert it.

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.

4 participants