New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
filtron: init at 0.2.0 #93181
filtron: init at 0.2.0 #93181
Conversation
CC'ing searx's maintainers for review: @matejc @fpletz @globin @danielfullmer |
LGTM--It builds correctly for me, at least. I don't plan to use filtron myself, since my searx instance is private. While a nixos module that works with the existing searx module would be nice, I don't think it should be required before merging. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please us buildGoModule.
I marked this as stale due to inactivity. → More info |
|
||
vendorSha256 = "05q2g591xl08h387mm6njabvki19yih63dfsafgpc9hyk5ydf2n9"; | ||
|
||
doCheck = false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No tests? Please add a comment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have tried to fix the tests with a substituteInPlace but it requires much more work than I anticipated, so I marked upstream tests as obsolete and therefore I turned off the tests.
Motivation for this change
I packaged
filtron
in order to protect asearx
instance from bots and unwanted traffic.I packaged its go dependencies as well.
The code I use to run filtron as a service:
My searx instance running as a service on port 8100 (I use a systemd service because I have two searx instances on the server)
Apache configuration:
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)