http backend: Parse the correct argument when loading --tls-client-cert

Previously, the function read from ARGV[1] (hardcoded) rather than the
value passed to it, the command-line argument as it exists in globalOptions.

Resolves #1745
This commit is contained in:
Bryce Chidester 2018-04-30 15:05:06 -07:00
parent 84f82dae1a
commit e9f1721678
2 changed files with 8 additions and 2 deletions

View File

@ -0,0 +1,7 @@
Bugfix: Correctly parse the argument to --tls-client-cert
Previously, the --tls-client-cert method attempt to read ARGV[1] (hardcoded)
instead of the argument that was passed to it. This has been corrected.
https://github.com/restic/restic/issues/1745
https://github.com/restic/restic/pull/1746

View File

@ -7,7 +7,6 @@ import (
"io/ioutil"
"net"
"net/http"
"os"
"strings"
"time"
@ -28,7 +27,7 @@ type TransportOptions struct {
// readPEMCertKey reads a file and returns the PEM encoded certificate and key
// blocks.
func readPEMCertKey(filename string) (certs []byte, key []byte, err error) {
data, err := ioutil.ReadFile(os.Args[1])
data, err := ioutil.ReadFile(filename)
if err != nil {
return nil, nil, errors.Wrap(err, "ReadFile")
}