MINOR: backend: Don't release SI endpoint anymore in connect_server()

Thanks to the previous patch (822decfd "BUG/MAJOR: stream-int: Release SI
endpoint on server side ASAP on retry"), it is now useless to release any
existing connection in connect_server() because it was already done in
back_handle_st_cer() if necessary.

This patch is not a CLEANUP because it may introduce some bugs in edge
cases. There is no reason to backport it for now except if it is required to
fix a bug.
This commit is contained in:
Christopher Faulet 2021-06-01 15:54:49 +02:00
parent f822decfda
commit e9106d69cb

View File

@ -1334,11 +1334,6 @@ int connect_server(struct stream *s)
hash = conn_calculate_hash(&hash_params);
/* This will catch some corner cases such as lying connections resulting from
* retries or connect timeouts but will rarely trigger.
*/
si_release_endpoint(&s->si[1]);
/* do not reuse if mode is not http */
if (!IS_HTX_STRM(s))
goto skip_reuse;