From b50dfa6b2dd2d459e0c2746ac9367982b96ffac0 Mon Sep 17 00:00:00 2001 From: David Kalnischkies Date: Wed, 6 Jul 2016 15:10:52 +0200 Subject: report all instead of first error up the acquire chain If we don't give a specific error to report up it is likely that all error currently in the error stack are equally important, so reporting just one could turn out to be confusing e.g. if name resolution failed in a SRV record list. --- apt-pkg/acquire-method.cc | 17 ++++++++++++++--- methods/https.cc | 8 +++++++- 2 files changed, 21 insertions(+), 4 deletions(-) diff --git a/apt-pkg/acquire-method.cc b/apt-pkg/acquire-method.cc index a9fff661b..39d5e61f2 100644 --- a/apt-pkg/acquire-method.cc +++ b/apt-pkg/acquire-method.cc @@ -80,9 +80,20 @@ void pkgAcqMethod::Fail(bool Transient) { string Err = "Undetermined Error"; if (_error->empty() == false) - _error->PopMessage(Err); - _error->Discard(); - Fail(Err,Transient); + { + Err.clear(); + while (_error->empty() == false) + { + std::string msg; + if (_error->PopMessage(msg)) + { + if (Err.empty() == false) + Err.append("\n"); + Err.append(msg); + } + } + } + Fail(Err, Transient); } /*}}}*/ // AcqMethod::Fail - A fetch has failed /*{{{*/ diff --git a/methods/https.cc b/methods/https.cc index 92f786d17..bbdbd8fa9 100644 --- a/methods/https.cc +++ b/methods/https.cc @@ -436,7 +436,13 @@ bool HttpsMethod::Fetch(FetchItem *Itm) break; } #pragma GCC diagnostic pop - return _error->Error("%s", curl_errorstr); + // only take curls technical errors if we haven't our own + // (e.g. for the maximum size limit we have and curls can be confusing) + if (_error->PendingError() == false) + _error->Error("%s", curl_errorstr); + else + _error->Warning("curl: %s", curl_errorstr); + return false; } // server says file not modified -- cgit v1.2.3