feat(extractors/services): Add support for Tor #71
Conversation
| mkIf _tor.enable { | ||
| name = "Tor"; | ||
| icon = "services.tor"; | ||
| info = optionalString _tor.relay.enable "Role: ${_tor.relay.role}"; |
There was a problem hiding this comment.
can we make this an mkIf then users can set info without mkForce if the relay is not enabled
There was a problem hiding this comment.
I tried to replicate erroneous behavior when setting info, while the relay option is disabled, however everything worked correctly even without mkForce. So there is no need?
What is interesting though is I found out that the strings get concatenated when the relay is enabled.
Let's say:
topology.self.services.tor = "lol";
In the node running tor, where a relay is enabled as role = "relay";
would lead to the info being Role: relay lol". Is that intended?
I don't think you pushed anything 😅 |
Yeah, Sorry. It was a misclick actually 😅 |
Simple support for Tor.
This should cover most needs.
Might be improved later on with more details extracting self-hosted onion Services if preferred.