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.

Sign up for GitHub

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

Jump to bottom

Tests for NAT-Traversal and Hole-Punching #381

Merged
merged 15 commits into from
Jun 24, 2022
Merged

Tests for NAT-Traversal and Hole-Punching #381

merged 15 commits into from
Jun 24, 2022

Commits on Jun 24, 2022

  1. test: NAT Traversal testing utils and tests

    @joshuakarp @tegefaulkes
    joshuakarp authored and tegefaulkes committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    18437ce View commit details
    Browse the repository at this point in the history

  2. test: describeIf and testIf utilities for conditional testing

    @emmacasolin @tegefaulkes
    emmacasolin authored and tegefaulkes committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    2110634 View commit details
    Browse the repository at this point in the history

  3. feat: better logging for service handler errors 

    If an error was logged out by a service handler it would previously appear as `createClientService:`, which was too vague. The logger will now state the name of the handler.
    @emmacasolin @tegefaulkes
    emmacasolin authored and tegefaulkes committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    8cf52a4 View commit details
    Browse the repository at this point in the history

  4. fix: PolykeyAgent now stops nodes domains when startup fails

    @emmacasolin @tegefaulkes
    emmacasolin authored and tegefaulkes committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    fc365b4 View commit details
    Browse the repository at this point in the history

  5. fix: ICE protocol uses Promise.any instead of Promise.all

    @emmacasolin @tegefaulkes
    emmacasolin authored and tegefaulkes committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    eebeee3 View commit details
    Browse the repository at this point in the history

  6. tests: fixes from testnet changes 

    The testnet PR brought in some changes that affect the NAT tests, so they needed to be modified slightly.
- Adding a node to the node graph pings the node by default. We want to disable this in the NAT tests so we have more control over the pings.
- Nodes add the details of any node that pings them. We can now remove some additional `nodes add` calls that were required to imitate this previously missing functionality.
    @emmacasolin @tegefaulkes
    emmacasolin authored and tegefaulkes committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    ed2973e View commit details
    Browse the repository at this point in the history

  7. build: upgraded async-init, async-locks and js-logger

    @emmacasolin @tegefaulkes
    emmacasolin authored and tegefaulkes committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    a0adb35 View commit details
    Browse the repository at this point in the history

  8. feat: removing edmSimple configuration 

    Now that nodes add the details of a node that contacts them, we no longer need the `edmSimple` configuration to do this manually.
    @emmacasolin @tegefaulkes
    emmacasolin authored and tegefaulkes committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    8fdde54 View commit details
    Browse the repository at this point in the history

  9. style: linting and other style fixes 

    General linting, using capitals for constants in NAT utils, and lowercase test descriptions
    @emmacasolin @tegefaulkes
    emmacasolin authored and tegefaulkes committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    9800c4a View commit details
    Browse the repository at this point in the history

  10. fix: relay node for hole punch message now re-writes proxy address 

    A relay node for a hole punch message was previously not modifying the proxy address in the message (which is the "return address" used to contact the source node). For nodes behind a NAT, who do not know their own public address, they rely on this overwriting so that nodes do not try to contact them on their private, inaccessible address.
    @emmacasolin @tegefaulkes
    emmacasolin authored and tegefaulkes committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    4f8d17e View commit details
    Browse the repository at this point in the history

  11. feat: added ability to pause refreshBucketQueue 

    This will allow us to disable the queue for testing.
    @tegefaulkes
    tegefaulkes committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    1095f8f View commit details
    Browse the repository at this point in the history

  12. build: package upgrades to aid testing

    @emmacasolin @tegefaulkes
    emmacasolin authored and tegefaulkes committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    520b799 View commit details
    Browse the repository at this point in the history

  13. style: lintfix

    @emmacasolin @tegefaulkes
    emmacasolin authored and tegefaulkes committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    1e35180 View commit details
    Browse the repository at this point in the history

  14. feat: added diagrams to DMZ tests 

    #388
    @emmacasolin @tegefaulkes
    emmacasolin authored and tegefaulkes committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    856388b View commit details
    Browse the repository at this point in the history

  15. fix: async bug with composing connections 

    the composed flag was set at the beginning of the compose function causing another function to throw an error due to an undefined property if it was called at the same time.
    @tegefaulkes
    tegefaulkes committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    7c91ace View commit details
    Browse the repository at this point in the history