Include port in "Host" header when constructing KMS requests

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Unresolved
    • Priority: Minor - P4
    • None
    • Affects Version/s: None
    • Component/s: None
    • None
    • Not Needed
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      Background & Motivation

      The AWS, Azure, and GCP requests include the Host header as part of the request, but do not include a port number. There are no reported issues, but this may be a discrepancy from the expectation in https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Host

      The Host request header specifies the host and port number of the server to which the request is being sent.

      If no port is included, the default port for the service requested is implied (e.g., 443 for an HTTPS URL, and 80 for an HTTP URL).

      Scope

      • Update each of the KMS HTTP requests (AWS, Azure, and GCP) in mongocrypt-kms-ctx.c to include the port as the argument used to supply the "Host" header.

            Assignee:
            Unassigned
            Reporter:
            Kevin Albertson
            Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: