Skip to content
This repository has been archived by the owner on May 11, 2022. It is now read-only.

fix: simplify address replacement #102

Merged
merged 1 commit into from Feb 25, 2021
Merged

fix: simplify address replacement #102

merged 1 commit into from Feb 25, 2021

Commits on Feb 25, 2021

  1. fix: simplify address replacement

    We only ever want to dial the remote peer's observed address (to prevent
    DoS attacks), so we might as well _always_ replace the reported
    addresses with the observed address.
    
    This also fixes a few issues in the original PR:
    
    * Simpler logic.
    * Replaces the IP address, not the port. That's what we actually care about.
    * Won't accidentally dial "skipped" protocols (e.g., circuit transport).
    Stebalien committed Feb 25, 2021
    Configuration menu
    Copy the full SHA
    aa794f7 View commit details
    Browse the repository at this point in the history