[reSIProcate] Proposed change to Connection

["Derek MacDonald" <derek@xxxxxxxxxxxxxxx>]

Right now, SSL handshake errors are not propogated into the 503 the stack returns.

 

Changing Connection::read to return an enum instead of an integer(the # bytes read is not used; it is just tested for success/failure) .  Transport::fail would also take this enum as a parameter.  This would allow more information in the 503, such as a certificate mismatch.

 

So far, the enum values I have are:

 

Success,

Failure,

CertificateMismatch

 

In the case where different errors were encountered when trying alternatives discovered using SRV, the highest enum encountered will be used(could aggregate if desired.

 

This change would only affect people that have written their own custom transports.

 

--Derek