Uploaded image for project: 'C Driver'
  1. C Driver
  2. CDRIVER-3602

Prefix enum constants in OCSP callback

    • Type: Icon: Task Task
    • Resolution: Fixed
    • Priority: Icon: Major - P3 Major - P3
    • 1.17.0-beta, 1.17.0
    • Affects Version/s: None
    • Component/s: tls
    • None

      The enum values introduced in 8d3aa32 for CDRIVER-3408 appear to conflict with existing constants on Windows. I picked this up during an Appveyor build for the PHP driver (see: here).

      I suggest changing these to more unique names. Additionally, there are a few cases within _mongoc_ocsp_tlsext_status_cb where the SUCCESS case in our enum is used to check return values from other TLS functions (e.g. OCSP_basic_verify, X509_check_host). I'd suggest we define explicit constants for those functions rather than re-use our internal enum for the callback's return value.

            Assignee:
            jmikola@mongodb.com Jeremy Mikola
            Reporter:
            jmikola@mongodb.com Jeremy Mikola
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: