minfrin commented on PR #9:
URL: https://github.com/apache/serf/pull/9#issuecomment-3027210773

   > I entirely get the desire to attach this error to "a unit of work", your 
choices for a unit of work in this code consist of just one structure, 
serf_ssl_context_t, and that is it.
   > 
   > 
https://github.com/apache/serf/blob/trunk/buckets/ssl_buckets.c#L136C8-L136C26
   
   Looking at this closer:
   
   https://github.com/apache/serf/blob/trunk/buckets/ssl_buckets.c#L148
   
   ```
       SSL *ssl;
   ```
   
   The serf_ssl_context_t structure represents one connection by definition.
   
   By passing serf_ssl_context_t as the baton, like we do in 
https://github.com/apache/subversion/pull/31, we tie the error to the 
connection, thus solving your problem.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@serf.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to