diff options
| author | David Kalnischkies <david@kalnischkies.de> | 2017-09-26 19:27:30 +0200 |
|---|---|---|
| committer | David Kalnischkies <david@kalnischkies.de> | 2017-09-26 19:32:15 +0200 |
| commit | f3e34838d95132e5f318e85525326decbfb19e36 (patch) | |
| tree | eb090957cefb8bfb751ee3911ea9df4b5be4ead1 /methods | |
| parent | 5e770a07c8fd649340e83725f6d07b94c361e87c (diff) | |
proper error reporting for v3 onion services
APT connects just fine to any .onion address given, only if the connect
fails somehow it will perform checks on the sanity of which in this case
is checking the length as they are well defined and as the strings are
arbitrary a user typing them easily mistypes which apt should can be
slightly more helpful in figuring out by saying the onion hasn't the
required length.
Diffstat (limited to 'methods')
| -rw-r--r-- | methods/connect.cc | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/methods/connect.cc b/methods/connect.cc index 949cd3f99..6a7b71c0b 100644 --- a/methods/connect.cc +++ b/methods/connect.cc @@ -523,13 +523,13 @@ bool UnwrapSocks(std::string Host, int Port, URI Proxy, std::unique_ptr<MethodFd else if (errcode == 0x01) { auto const prevdot = Host.rfind('.', lastdot - 1); - if (lastdot == 16 && prevdot == std::string::npos) + if (prevdot == std::string::npos && (lastdot == 16 || lastdot == 56)) ; // valid .onion address - else if (prevdot != std::string::npos && (lastdot - prevdot) == 17) + else if (prevdot != std::string::npos && ((lastdot - prevdot) == 17 || (lastdot - prevdot) == 57)) ; // valid .onion address with subdomain(s) else { - errstr = "Invalid hostname: onion service name must be 16 characters long"; + errstr = "Invalid hostname: onion service name must be either 16 or 56 characters long"; Owner->SetFailReason("SOCKS"); } } |