Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • Q quickfix
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 48
    • Issues 48
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 25
    • Merge requests 25
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • quickfixgo
  • quickfix
  • Merge requests
  • !457

Make use of SocketCAFile config

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Administrator requested to merge github/fork/hyde-zhang/main into main Aug 24, 2021
  • Overview 8
  • Commits 3
  • Pipelines 0
  • Changes 2

Created by: hyde-zhang

I think the SocketUseSSL config actually makes the code skips the SocketCAFile config when it is set to Y.

Since we have SocketCAFile option to use a supplied CA certificate to verify server identity, we still need the part of the code to add the supplied CA certificate to the CertPool and other functionalities. Skipping client authentication shouldn't skip everything.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/hyde-zhang/main