From 0906b69c0678d17f3b0139fe6d01c62f8e734890 Mon Sep 17 00:00:00 2001
From: Edoardo Putti <edoardo.putti@gmail.com>
Date: Thu, 22 Aug 2019 15:37:17 +0200
Subject: [PATCH] [fix] Handle OPDS catalog responses accordingly (#5241)

- handle moved permanently HTTP 301
- handle authentication required HTTP 401
- handle authentication errors HTTP 403
- handle catalog not found HTTP 404
---
 frontend/ui/widget/opdsbrowser.lua | 18 +++++++++++++++++-
 1 file changed, 17 insertions(+), 1 deletion(-)

diff --git a/frontend/ui/widget/opdsbrowser.lua b/frontend/ui/widget/opdsbrowser.lua
index fa65e597b..7a353d038 100644
--- a/frontend/ui/widget/opdsbrowser.lua
+++ b/frontend/ui/widget/opdsbrowser.lua
@@ -298,9 +298,25 @@ function OPDSBrowser:fetchFeed(item_url, username, password, method)
         if xml ~= "" then
             return xml
         end
+    elseif code == 301 then
+        UIManager:show(InfoMessage:new{
+            text = T(_("The catalog has been permanently moved. Please update catalog URL to '%1'."), headers['Location']),
+        })
+    elseif code == 401 then
+        UIManager:show(InfoMessage:new{
+            text = T(_("Authentication required for catalog. Please add a username and password.")),
+        })
+    elseif code == 403 then
+        UIManager:show(InfoMessage:new{
+            text = T(_("Failed to authenticate. Please check your username and password.")),
+        })
+    elseif code == 404 then
+        UIManager:show(InfoMessage:new{
+            text = T(_("Catalog not found.")),
+        })
     else
         UIManager:show(InfoMessage:new{
-            text = T(_("Cannot get catalog. Server code response %1"), code),
+            text = T(_("Cannot get catalog. Server response code %1."), code),
         })
     end
 end