Skip to content
This repository has been archived by the owner on Oct 17, 2023. It is now read-only.

check if cacert string is path to a file #334

Merged
merged 1 commit into from
Mar 24, 2017
Merged

Conversation

jipperinbham
Copy link
Contributor

@jipperinbham jipperinbham commented Mar 24, 2017

the following change will first test if the provided string is a path to a file containing the certificate and if so, read the cert from the file, it os.Stat returns an error, it is assumed to be the raw cert string

fixes #332

  • CHANGELOG.md updated

@codecov
Copy link

codecov bot commented Mar 24, 2017

Codecov Report

Merging #334 into master will increase coverage by 0.1%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #334     +/-   ##
=========================================
+ Coverage   83.74%   83.85%   +0.1%     
=========================================
  Files          49       49             
  Lines        2559     2576     +17     
=========================================
+ Hits         2143     2160     +17     
  Misses        287      287             
  Partials      129      129
Impacted Files Coverage Δ
adaptor/mongodb/client.go 100% <100%> (ø) ⬆️
adaptor/rabbitmq/client.go 100% <100%> (ø) ⬆️
adaptor/rethinkdb/client.go 68.14% <100%> (+1.48%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f3bb72e...133e262. Read the comment docs.

the following change will first test if the provided string is a path to a file containing the certificate and if so, read the cert from the file, it os.Stat returns an error, it is assumed to be the raw cert string

fixes #332
@jipperinbham jipperinbham merged commit 4242b9b into master Mar 24, 2017
@jipperinbham jipperinbham deleted the 332-cacert-fix branch March 24, 2017 20:09
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Parameter cacerts does not accept filename
1 participant