> 16 + except httplib2.SSLHandshakeError:
> 17 + raise CommandError(
> 18 + "Certificate verify failed, use --disable-cert-check to disable "
> 19 + "the certificate check.")
>
> s/verify/verification/
I'll trust you on this one… but be aware that "Certificate verify failed" is a wording I took from the existing exception.
> 37 + parser.add_argument(
> 38 + '--disable-cert-check', action='store_true', help=(
> 39 + "Disable SSL certificate check"), default=False)
>
> This is a long option and will get a bit annoying if someone needs to
> repeatedly specify it (e.g. on a dev system), can you make a short one too
> please, e.g. -d
Good point. But '-d' is already taken (--debug) so I'll add '-di'.
Thanks for the review!
> 16 + except httplib2. SSLHandshakeErr or: cert-check to disable " verification/
> 17 + raise CommandError(
> 18 + "Certificate verify failed, use --disable-
> 19 + "the certificate check.")
>
> s/verify/
I'll trust you on this one… but be aware that "Certificate verify failed" is a wording I took from the existing exception.
> 37 + parser. add_argument( cert-check' , action= 'store_ true', help=(
> 38 + '--disable-
> 39 + "Disable SSL certificate check"), default=False)
>
> This is a long option and will get a bit annoying if someone needs to
> repeatedly specify it (e.g. on a dev system), can you make a short one too
> please, e.g. -d
Good point. But '-d' is already taken (--debug) so I'll add '-di'.