{"action":"created","issue":{"url":"https://api.github.com/repos/pastly/simple-bw-scanner/issues/146","repository_url":"https://api.github.com/repos/pastly/simple-bw-scanner","labels_url":"https://api.github.com/repos/pastly/simple-bw-scanner/issues/146/labels{/name}","comments_url":"https://api.github.com/repos/pastly/simple-bw-scanner/issues/146/comments","events_url":"https://api.github.com/repos/pastly/simple-bw-scanner/issues/146/events","html_url":"https://github.com/pastly/simple-bw-scanner/issues/146","id":319474397,"number":146,"title":"Use sensible tor options, and document why each one is used","user":{"login":"teor2345","id":8951843,"avatar_url":"https://avatars0.githubusercontent.com/u/8951843?v=4","gravatar_id":"","url":"https://api.github.com/users/teor2345","html_url":"https://github.com/teor2345","followers_url":"https://api.github.com/users/teor2345/followers","following_url":"https://api.github.com/users/teor2345/following{/other_user}","gists_url":"https://api.github.com/users/teor2345/gists{/gist_id}","starred_url":"https://api.github.com/users/teor2345/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/teor2345/subscriptions","organizations_url":"https://api.github.com/users/teor2345/orgs","repos_url":"https://api.github.com/users/teor2345/repos","events_url":"https://api.github.com/users/teor2345/events{/privacy}","received_events_url":"https://api.github.com/users/teor2345/received_events","type":"User","site_admin":false},"labels":[{"id":873405717,"url":"https://api.github.com/repos/pastly/simple-bw-scanner/labels/enhancement","name":"enhancement","color":"a2eeef","default":true}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2018-05-02T09:15:49Z","updated_at":"2018-05-02T13:33:55Z","closed_at":"2018-05-02T13:20:48Z","author_association":"COLLABORATOR","body":" @juga0 mentioned some tor options from bwscanner on #tor-dev. bwscanner took them from chutney and torflow. Let's think about whether we need each one:\r\n\r\n    'LearnCircuitBuildTimeout': 0,  # Disable adaptive circuit timeouts.\r\n    'CircuitBuildTimeout': 20,\r\n\r\nYes, we should have a set timeout, to avoid biasing results.\r\n\r\n    'UseEntryGuards': 0,  # Disable UseEntryGuards to avoid PathBias warnings.\r\n\r\nYes, we don't need entry guards, we won't be using them.\r\n\r\n    'UseMicroDescriptors': 0,\r\n    'FetchUselessDescriptors': 1,\r\n\r\nWe shouldn't need these options, if Stem and sbws are coded correctly. Or, we can choose to work with them on or off, see #144.\r\n\r\n    'FetchDirInfoEarly': 1,\r\n    'FetchDirInfoExtraEarly': 1,\r\n\r\nIf we want to prioritise measuring new relays, we should set these options. But we should be prepared for exits to fail our circuits, because they don't have the new relay's descriptor yet.\r\n\r\n    'SafeLogging': 0,\r\n    'LogTimeGranularity': 1,\r\n\r\nThese are useful logging options for clients that don't care about anonymity."},"comment":{"url":"https://api.github.com/repos/pastly/simple-bw-scanner/issues/comments/385979041","html_url":"https://github.com/pastly/simple-bw-scanner/issues/146#issuecomment-385979041","issue_url":"https://api.github.com/repos/pastly/simple-bw-scanner/issues/146","id":385979041,"user":{"login":"teor2345","id":8951843,"avatar_url":"https://avatars0.githubusercontent.com/u/8951843?v=4","gravatar_id":"","url":"https://api.github.com/users/teor2345","html_url":"https://github.com/teor2345","followers_url":"https://api.github.com/users/teor2345/followers","following_url":"https://api.github.com/users/teor2345/following{/other_user}","gists_url":"https://api.github.com/users/teor2345/gists{/gist_id}","starred_url":"https://api.github.com/users/teor2345/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/teor2345/subscriptions","organizations_url":"https://api.github.com/users/teor2345/orgs","repos_url":"https://api.github.com/users/teor2345/repos","events_url":"https://api.github.com/users/teor2345/events{/privacy}","received_events_url":"https://api.github.com/users/teor2345/received_events","type":"User","site_admin":false},"created_at":"2018-05-02T13:33:55Z","updated_at":"2018-05-02T13:33:55Z","author_association":"COLLABORATOR","body":"Sure. That also seems like a good design choice. If sbws fetches on a client schedule, then all the relays it uses should have updated consensuses.\r\n\r\nWe only vote once an hour anyway. And it takes clients 1-2 hours to get new bandwidths after they are in a consensus.\r\n\r\nSo when fetching early, the best case delay is 3 hours, and the worst case is about 4.5 hours.\r\nWhen fetching on the client schedule, the best case delay is about 4 hours, and the worse case is about 6.\r\n\r\nYou're right, the extra load isn't worth it."}}
{"actor":{"display_login":"teor2345"}}