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

Fix global protocol assignment special case #7

Merged
merged 1 commit into from Nov 12, 2021

Conversation

camshaft
Copy link
Contributor

The special cases should be in 192.0.0.0/24, not 192.168.0.0/24

From https://www.rfc-editor.org/rfc/rfc7723.html#section-4.1

   +----------------------+-------------------------------------------+
   | Attribute            | Value                                     |
   +----------------------+-------------------------------------------+
   | Address Block        | 192.0.0.9/32                              |
   | Name                 | Port Control Protocol Anycast             |
   | RFC                  | RFC 7723 (this document)                  |
   | Allocation Date      | October 2015                              |
   | Termination Date     | N/A                                       |
   | Source               | True                                      |
   | Destination          | True                                      |
   | Forwardable          | True                                      |
   | Global               | True                                      |
   | Reserved-by-Protocol | False                                     |
   +----------------------+-------------------------------------------+

From https://www.rfc-editor.org/rfc/rfc8155.html#section-8.1

    +----------------------+-------------------------------------------+
    | Attribute            | Value                                     |
    +----------------------+-------------------------------------------+
    | Address Block        | 192.0.0.10/32                             |
    | Name                 | Traversal Using Relays around NAT Anycast |
    | RFC                  | RFC 8155                                  |
    | Allocation Date      | 2017-02                                   |
    | Termination Date     | N/A                                       |
    | Source               | True                                      |
    | Destination          | True                                      |
    | Forwardable          | True                                      |
    | Global               | True                                      |
    | Reserved-by-Protocol | False                                     |
    +----------------------+-------------------------------------------+

The special cases should be in `192.0.0.0/24`, not `192.168.0.0/24`

From https://www.rfc-editor.org/rfc/rfc7723.html#section-4.1

```
   +----------------------+-------------------------------------------+
   | Attribute            | Value                                     |
   +----------------------+-------------------------------------------+
   | Address Block        | 192.0.0.9/32                              |
   | Name                 | Port Control Protocol Anycast             |
   | RFC                  | RFC 7723 (this document)                  |
   | Allocation Date      | October 2015                              |
   | Termination Date     | N/A                                       |
   | Source               | True                                      |
   | Destination          | True                                      |
   | Forwardable          | True                                      |
   | Global               | True                                      |
   | Reserved-by-Protocol | False                                     |
   +----------------------+-------------------------------------------+
```

From https://www.rfc-editor.org/rfc/rfc8155.html#section-8.1

```
    +----------------------+-------------------------------------------+
    | Attribute            | Value                                     |
    +----------------------+-------------------------------------------+
    | Address Block        | 192.0.0.10/32                             |
    | Name                 | Traversal Using Relays around NAT Anycast |
    | RFC                  | RFC 8155                                  |
    | Allocation Date      | 2017-02                                   |
    | Termination Date     | N/A                                       |
    | Source               | True                                      |
    | Destination          | True                                      |
    | Forwardable          | True                                      |
    | Global               | True                                      |
    | Reserved-by-Protocol | False                                     |
    +----------------------+-------------------------------------------+
```
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.07%) to 86.999% when pulling 760b527 on camshaft:patch-1 into a0966e9 on JakubOnderka:master.

@JakubOnderka JakubOnderka merged commit 79bcaf5 into JakubOnderka:master Nov 12, 2021
@JakubOnderka
Copy link
Owner

Thanks, nice catch!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants