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

Add cluster_name property to avoid several clusters merging accidentally #36

Merged
merged 5 commits into from
Apr 12, 2022

Commits on Apr 7, 2022

  1. Configuration menu
    Copy the full SHA
    902d730 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    63128f0 View commit details
    Browse the repository at this point in the history

Commits on Apr 8, 2022

  1. return BadCluster on cluster name mismatch

    This will make testing and operations easier, by explicitly
    refusing the message instead of silently dropping it.
    xvello committed Apr 8, 2022
    Configuration menu
    Copy the full SHA
    abefca0 View commit details
    Browse the repository at this point in the history
  2. remove invalid ignore_oversized_payload test

    Introduced in #4, this test has been outdated since the reception
    buffer sized has been increased to the full UDP MTU.
    
    As is, it sends an empty Syn message before sending another valid
    Syn message. The assert checks for an SynAck message, that is the
    response to the first packet instead of the second.
    
    Now that peers check the cluster name property of Syn messages,
    the first packet gets a BadCluster response, which fails the assert.
    xvello committed Apr 8, 2022
    Configuration menu
    Copy the full SHA
    2fa4c91 View commit details
    Browse the repository at this point in the history

Commits on Apr 12, 2022

  1. Configuration menu
    Copy the full SHA
    3980594 View commit details
    Browse the repository at this point in the history