Skip to content

Add support for TLVs #3

Open
Open

Description

@emersion

The PP2_TYPE_ALPN TLV is necessary to negotiate HTTP/2.

Activity

  1. mastercactapus commented on Mar 12, 2023

    @mastercactapus
    Owner

    I started this in the tlv branch, and want to test it with some in-the-wild software before merging & releasing.

  2. vmihailenco commented on Aug 22, 2023

    @vmihailenco

    @mastercactapus I am about to try this so I wonder what were your conclusions after testing TLV support?

  3. Embraser01 commented on Oct 10, 2023

    @Embraser01

    I started this in the tlv branch, and want to test it with some in-the-wild software before merging & releasing.

    Hi @mastercactapus, after having issue with AWS Network Load Balancer (that add TLV), I tested your tlv branch with caddy v2.7.4 and I'm happy to say that it worked perfectly 💯

    Would it be possible to release a new version with the TLV feature?

    PS : For those who want to test it with Caddy, you can do it like this:

    xcaddy build \
      --with github.com/mastercactapus/proxyprotocol=github.com/mastercactapus/proxyprotocol@tlv
    
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions