Skip to content
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

Feature/exit policy #4030

Merged
merged 21 commits into from
Oct 26, 2023
Merged

Feature/exit policy #4030

merged 21 commits into from
Oct 26, 2023

Conversation

jstuczyn
Copy link
Contributor

@jstuczyn jstuczyn commented Oct 23, 2023

Description

Closes: #4024

Apart from obviously testing the thing, before it can be merged, we need to make sure the following is adjusted to the correct value (i.e. we need to redeploy the website):

pub const EXIT_POLICY_URL: &str = &*"http://localhost:8001/sample_policy";

edit 25/10/23: it was already updated

what does it do?

as the related ticket describes, it adds a different kind of request filtering to our network requesters. rather than being based on the current allow lists system, it uses a tornull-based exit policy.

right now this is enabled via the with-exit-policy flag (or obviously also via modifying config.toml file), however, soon this will get deprecated and the exit policy will become the default. even further in the future, the allow-list will be completely removed and NRs will all be exclusively running the exit policy.

to see whether particular NR uses the exit policy (and what entires it has there) you can do one of the following:

finally, @tommyv1987 @benedettadavico when testing you can set the EXIT_POLICY environmental variable to any endpoint serving a valid policy file. For my local tests I've just had a dummy sample_policy served with python -m http.server 8001

for the 'real' deployment file, that we will use in mainnet, you can see it here: https://github.com/nymtech/nym-websites/pull/88#issuecomment-1772577284

Checklist:

  • added a changelog entry to CHANGELOG.md

@benedettadavico benedettadavico merged commit 0801b3c into develop Oct 26, 2023
20 checks passed
@benedettadavico benedettadavico deleted the feature/exit-policy branch October 26, 2023 09:35
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.

Add exit node policy from TorNull and Tor Exit Node Policy
2 participants