summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorMax Kellermann <max@musicpd.org>2018-07-06 19:26:11 +0200
committerMax Kellermann <max@musicpd.org>2018-07-06 19:26:11 +0200
commit466625f7ad9066eeb25383b0ad6f63954b1af9f8 (patch)
tree1a1f1b3352ba31d0207c2a11b06a810a6120c153 /src
parentb8259e604ab6f77f8c0c0d27f0ee095a36d8007b (diff)
input/curl: use new class HttpStatusError
This way, IsFileNotFound() can detect status 404.
Diffstat (limited to 'src')
-rw-r--r--src/input/Error.cxx8
-rw-r--r--src/input/plugins/CurlInputPlugin.cxx5
-rw-r--r--src/lib/curl/Error.hxx42
3 files changed, 54 insertions, 1 deletions
diff --git a/src/input/Error.cxx b/src/input/Error.cxx
index ae7b9d7de..3db1f26f1 100644
--- a/src/input/Error.cxx
+++ b/src/input/Error.cxx
@@ -21,6 +21,10 @@
#include "Error.hxx"
#include "system/Error.hxx"
+#ifdef ENABLE_CURL
+#include "lib/curl/Error.hxx"
+#endif
+
#ifdef ENABLE_NFS
#include "lib/nfs/Error.hxx"
#include <nfsc/libnfs-raw-nfs.h>
@@ -33,6 +37,10 @@ IsFileNotFound(std::exception_ptr ep)
std::rethrow_exception(ep);
} catch (const std::system_error &e) {
return IsFileNotFound(e);
+#ifdef ENABLE_CURL
+ } catch (const HttpStatusError &e) {
+ return e.GetStatus() == 404;
+#endif
#ifdef ENABLE_NFS
} catch (const NfsClientError &e) {
return e.GetCode() == NFS3ERR_NOENT;
diff --git a/src/input/plugins/CurlInputPlugin.cxx b/src/input/plugins/CurlInputPlugin.cxx
index 6e1cf42f2..d9887727f 100644
--- a/src/input/plugins/CurlInputPlugin.cxx
+++ b/src/input/plugins/CurlInputPlugin.cxx
@@ -19,6 +19,7 @@
#include "config.h"
#include "CurlInputPlugin.hxx"
+#include "lib/curl/Error.hxx"
#include "lib/curl/Easy.hxx"
#include "lib/curl/Global.hxx"
#include "lib/curl/Request.hxx"
@@ -188,7 +189,9 @@ CurlInputStream::OnHeaders(unsigned status,
assert(!postponed_exception);
if (status < 200 || status >= 300)
- throw FormatRuntimeError("got HTTP status %ld", status);
+ throw HttpStatusError(status,
+ StringFormat<40>("got HTTP status %u",
+ status).c_str());
const std::lock_guard<Mutex> protect(mutex);
diff --git a/src/lib/curl/Error.hxx b/src/lib/curl/Error.hxx
new file mode 100644
index 000000000..4bd68e0c8
--- /dev/null
+++ b/src/lib/curl/Error.hxx
@@ -0,0 +1,42 @@
+/*
+ * Copyright 2003-2018 The Music Player Daemon Project
+ * http://www.musicpd.org
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+ */
+
+#ifndef CURL_ERROR_HXX
+#define CURL_ERROR_HXX
+
+#include <stdexcept>
+
+/**
+ * Thrown when an unsuccessful status was received from the HTTP
+ * server.
+ */
+class HttpStatusError : public std::runtime_error {
+ unsigned status;
+
+public:
+ template<typename M>
+ explicit HttpStatusError(unsigned _status, M &&_msg) noexcept
+ :std::runtime_error(std::forward<M>(_msg)), status(_status) {}
+
+ unsigned GetStatus() const noexcept {
+ return status;
+ }
+};
+
+#endif