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
magnetico: init package and service #64738
Conversation
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.
Only reviewed the Go package.
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.
Approving the Go package only. I have not reviewed the NixOS module.
@GrahamcOfBorg build magnetico |
I disabled the build on darwin, since ofBorg says the build is failing. |
|
||
web.port = mkOption { | ||
type = types.port; | ||
default = 0; |
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.
This doesn't seem sensible. Wouldn't this cause an arbitrary free port to be selected every time?
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.
Correct, it's what the software does by default. Do you think I should pick an arbitrary port?
I added a small test. |
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.
Other than this thing, this LGTM
Should be ready. |
Thank you all for the review. |
Motivation for this change
Wanted to self-host a DHT crawler.
Things done
nix path-info -S
before and after)I have tested basically everything. the crawler works, the web ui works, also authentication.