rnewson commented on code in PR #4736:
URL: https://github.com/apache/couchdb/pull/4736#discussion_r1303541026
##########
src/chttpd/src/chttpd_util.erl:
##########
@@ -110,3 +118,125 @@ get_db_info(DbName) ->
catch
_Tag:Error -> {error, Error}
end.
+
+mochiweb_socket_set(Sock) ->
+ put(?MOCHIWEB_SOCKET, Sock).
+
+mochiweb_socket_get() ->
+ get(?MOCHIWEB_SOCKET).
+
+mochiweb_socket_clean() ->
+ erase(?MOCHIWEB_SOCKET).
+
+mochiweb_socket_check_msec() ->
+ MSec = config:get_integer("chttpd", "disconnect_check_msec",
?DISCONNECT_CHECK_MSEC),
+ % Add some jitter to avoid a stampede in case of a larger number of
concurrent connection
+ MSec + rand:uniform(MSec).
+
+stop_client_process_if_disconnected(Pid, Sock) ->
+ case is_mochiweb_socket_closed(Sock) of
+ true ->
+ couch_log:warning("client socket ~p disconnected, terminating ~p",
[Sock, Pid]),
+ exit(Pid, {shutdown, client_disconnected});
+ false ->
+ ok
+ end.
+
+is_mochiweb_socket_closed(undefined) ->
+ false;
+is_mochiweb_socket_closed(Sock) ->
Review Comment:
we maybe need an
```
is_mochiweb_socket_closed({ssl, Sock}) ->
is_mochiweb_socket_closed(Sock);
```
clause for couchdb's configured for native TLS?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
