Skip to content
GitLab
  • Menu
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • Arti Arti
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 141
    • Issues 141
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 14
    • Merge requests 14
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • The Tor Project
  • Core
  • ArtiArti
  • Merge requests
  • !390

Add DNS resolver support to arti

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged trinity-1686a requested to merge trinity-1686a/arti:dns into main Mar 06, 2022
  • Overview 39
  • Commits 7
  • Pipelines 1
  • Changes 23

I used trust-dns-proto with no default features because other trust-dns-* crates or proto with its default feature depend on tokio, and Arti try to be runtime independent.

I had an issue with TorClient::resolve returning a BadPort error, it seems it try to construct a TorAddr with a port set to 0, which TorAddr does not allow.

Before merging I should add some tests on resolve so the bug doesn't come back, as well as add some integration test for the new DNS port.

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: dns