Skip to content
Snippets Groups Projects
Commit 8766e700 authored by Adrien Béraud's avatar Adrien Béraud Committed by Anthony Léonard
Browse files

contrib/restbed: handle DNS resolution error


Add patch to fix restbed DNS resolution error,
that was ignored, causing indefinite hangup.

Change-Id: I40fff6a8662e8d204e0d4b813c4a94adadb99b65
Reviewed-by: default avatarAnthony Léonard <anthony.leonard@savoirfairelinux.com>
parent 33349af4
No related branches found
No related tags found
No related merge requests found
From: Adrien Beraud <adrien.beraud@savoirfairelinux.com>
Date: Fri, 2 Jun 2017 11:24:12 +0200
Subject: [PATCH] Handle DNS resolution error
DNS resolution error currently causes
indefinite hangup as the callback is
not called and higher-level layers
are not informed.
This commit bubbles up the error,
allowing the request to fail.
---
source/corvusoft/restbed/detail/socket_impl.cpp | 2 ++
1 file changed, 2 insertions(+)
diff --git a/source/corvusoft/restbed/detail/socket_impl.cpp b/source/corvusoft/restbed/detail/socket_impl.cpp
index 313a421..665cfe7 100644
--- a/source/corvusoft/restbed/detail/socket_impl.cpp
+++ b/source/corvusoft/restbed/detail/socket_impl.cpp
@@ -145,6 +145,8 @@ namespace restbed
m_is_open = true;
callback( error );
} );
+ } else {
+ callback( error );
}
} );
}
--
2.11.0
...@@ -44,6 +44,7 @@ restbed: restbed-$(RESTBED_VERSION).tar.gz ...@@ -44,6 +44,7 @@ restbed: restbed-$(RESTBED_VERSION).tar.gz
$(APPLY) $(SRC)/restbed/CMakeLists.patch $(APPLY) $(SRC)/restbed/CMakeLists.patch
$(APPLY) $(SRC)/restbed/strand.patch $(APPLY) $(SRC)/restbed/strand.patch
$(APPLY) $(SRC)/restbed/uri_cpp.patch $(APPLY) $(SRC)/restbed/uri_cpp.patch
$(APPLY) $(SRC)/restbed/dns-resolution-error.patch
$(MOVE) $(MOVE)
.restbed: restbed toolchain.cmake .restbed: restbed toolchain.cmake
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment