Qpid Proton
  1. Qpid Proton
  2. PROTON-396

Proton Messenger should configure the SNI headers on it's TLS connections so that those connections can be routed by SNI aware proxies.

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.4
    • Fix Version/s: 0.5
    • Component/s: proton-c
    • Labels:
      None
    1. PROTON-396-v2.patch
      0.5 kB
      Hiram Chirino

      Activity

      Hide
      Hiram Chirino added a comment -

      If you want to keep pn_ssl_t opaque, then I think that the signature on pn_ssl_init needs to change to pass the hostname that's being connected to. pn_ssl_init can then set the ssl->peer_hostname field.

      Show
      Hiram Chirino added a comment - If you want to keep pn_ssl_t opaque, then I think that the signature on pn_ssl_init needs to change to pass the hostname that's being connected to. pn_ssl_init can then set the ssl->peer_hostname field.
      Hide
      Hiram Chirino added a comment -

      Attaching a patch that fixes this issue.

      Show
      Hiram Chirino added a comment - Attaching a patch that fixes this issue.
      Hide
      Hiram Chirino added a comment -

      Attaching simpler fix.

      Show
      Hiram Chirino added a comment - Attaching simpler fix.
      Hide
      Hiram Chirino added a comment -

      calling pn_ssl_set_peer_hostname after init seem to work fine.

      Show
      Hiram Chirino added a comment - calling pn_ssl_set_peer_hostname after init seem to work fine.
      Hide
      ASF subversion and git services added a comment -

      Commit 1511914 from Ken Giusti in branch 'proton/trunk'
      [ https://svn.apache.org/r1511914 ]

      PROTON-396: set the peer hostname in the SSL context

      Show
      ASF subversion and git services added a comment - Commit 1511914 from Ken Giusti in branch 'proton/trunk' [ https://svn.apache.org/r1511914 ] PROTON-396 : set the peer hostname in the SSL context
      Hide
      Ken Giusti added a comment -

      Applied Hiram's patch.

      Show
      Ken Giusti added a comment - Applied Hiram's patch.

        People

        • Assignee:
          Ken Giusti
          Reporter:
          Hiram Chirino
        • Votes:
          0 Vote for this issue
          Watchers:
          2 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved:

            Development