From 58bd96ce8f1e054e10e769506660ccb103c22a44 Mon Sep 17 00:00:00 2001
From: webzwo0i <webzwo0i@c3d2.de>
Date: Sat, 3 Jul 2021 03:58:33 +0200
Subject: [PATCH] padreadonly: Remove dead `/ro/:id` handling

Read-only pads are accessed using `/p/` path since commit
ba3430ebb7347e378cfb5512f24dd2c9968e2210.
---
 src/ep.json                           |  6 ------
 src/node/hooks/express/padreadonly.js | 27 ---------------------------
 2 files changed, 33 deletions(-)
 delete mode 100644 src/node/hooks/express/padreadonly.js

diff --git a/src/ep.json b/src/ep.json
index 5642f8c12..b917aa1f3 100644
--- a/src/ep.json
+++ b/src/ep.json
@@ -50,12 +50,6 @@
         "expressCreateServer": "ep_etherpad-lite/node/hooks/express/padurlsanitize"
       }
     },
-    {
-      "name": "padreadonly",
-      "hooks": {
-        "expressCreateServer": "ep_etherpad-lite/node/hooks/express/padreadonly"
-      }
-    },
     {
       "name": "webaccess",
       "hooks": {
diff --git a/src/node/hooks/express/padreadonly.js b/src/node/hooks/express/padreadonly.js
deleted file mode 100644
index 4dda67b1f..000000000
--- a/src/node/hooks/express/padreadonly.js
+++ /dev/null
@@ -1,27 +0,0 @@
-'use strict';
-
-const readOnlyManager = require('../../db/ReadOnlyManager');
-const hasPadAccess = require('../../padaccess');
-const exporthtml = require('../../utils/ExportHtml');
-
-exports.expressCreateServer = (hookName, args, cb) => {
-  // serve read only pad
-  args.app.get('/ro/:id', async (req, res) => {
-    // translate the read only pad to a padId
-    const padId = await readOnlyManager.getPadId(req.params.id);
-    if (padId == null) {
-      res.status(404).send('404 - Not Found');
-      return;
-    }
-
-    // we need that to tell hasPadAcess about the pad
-    req.params.pad = padId;
-
-    if (await hasPadAccess(req, res)) {
-      // render the html document
-      const html = await exporthtml.getPadHTMLDocument(padId, null);
-      res.send(html);
-    }
-  });
-  return cb();
-};