code review 116360043: crypto/tls: Add new DigestSigner and KeyAlgorithm types...

crypto/tls: Add new DigestSigner and KeyAlgorithm types and fields to Config, implement defaults for each, and use inside doFullHandshake to produce the certificate verify message.

[jdeprez, 2014-07-28 02:27:05]

Hello agl@golang.org (cc: golang-codereviews@googlegroups.com,
jsing@google.com),

I'd like you to review this change to
https://code.google.com/p/go

[agl1, 2014-07-28 17:44:23]

NACK.

I do hope to support this in 1.4, but it's not specific to crypto/tls.

[jdeprez, 2014-07-30 01:23:39]

On 2014/07/28 17:44:23, agl1 wrote:
> NACK.
> 
> I do hope to support this in 1.4, but it's not specific to crypto/tls.

Anything I can do to assist?

[agl1, 2014-08-01 20:38:41]

Please see whether https://codereview.appspot.com/114680043 meets your
needs. If so, I'll send it out for review.

On Tue, Jul 29, 2014 at 6:23 PM,  <jdeprez@google.com> wrote:
> On 2014/07/28 17:44:23, agl1 wrote:
>>
>> NACK.
>
>
>> I do hope to support this in 1.4, but it's not specific to crypto/tls.
>
>
> Anything I can do to assist?
>
> https://codereview.appspot.com/116360043/

[agl1, 2014-08-01 20:48:33]

R=close

[jdeprez, 2014-08-04 00:24:30]

*** Abandoned ***