From 84b349e5a071c13f0cac447e6a0617004411cb77 Mon Sep 17 00:00:00 2001
From: Andrii Nikitin <anikitin@suse.de>
Date: Mon, 9 Oct 2023 16:21:54 +0200
Subject: [PATCH] Forbid server editing if MIRROR_PROVIDER is configured

---
 assets/javascripts/admintable.js                  |  6 +++++-
 lib/MirrorCache/WebAPI/Controller/App/Myserver.pm | 12 +++++++++++-
 lib/MirrorCache/WebAPI/Controller/App/Server.pm   | 12 +++++++++++-
 t/environ/01-smoke-mirror-provider.sh             |  3 ---
 templates/app/myserver/index.html.ep              |  7 ++++++-
 templates/app/server/index.html.ep                |  7 ++++++-
 6 files changed, 39 insertions(+), 8 deletions(-)

diff --git a/assets/javascripts/admintable.js b/assets/javascripts/admintable.js
index 6ddcd56f..2fc05b22 100644
--- a/assets/javascripts/admintable.js
+++ b/assets/javascripts/admintable.js
@@ -325,7 +325,11 @@ function renderAdminTableActions(data, type, row, meta) {
         return '';
     }
     var url = $("#admintable_api_url").val();
-    res = '<button type="submit" class="btn" alt="Edit" title="Edit" onclick="setEditingAdminTableRow(this.parentElement, true, false);"><i class="far fa-edit"></i></button>';
+    var noActions = document.getElementById("regionmirrorproviderpanel");
+    var res = '';
+    if (! noActions ) {
+        res = '<button type="submit" class="btn" alt="Edit" title="Edit" onclick="setEditingAdminTableRow(this.parentElement, true, false);"><i class="far fa-edit"></i></button>';
+    }
     if (url == '/rest/server' && data) {
         return res + '<button type="submit" class="btn" alt="UpdateLocation" title="Update Location" onclick="submitServerLocationRow(this.parentElement, ' + data + ');"><i class="far fa-bookmark"></i></button>';
     } else {
diff --git a/lib/MirrorCache/WebAPI/Controller/App/Myserver.pm b/lib/MirrorCache/WebAPI/Controller/App/Myserver.pm
index df733000..3309ae40 100644
--- a/lib/MirrorCache/WebAPI/Controller/App/Myserver.pm
+++ b/lib/MirrorCache/WebAPI/Controller/App/Myserver.pm
@@ -17,7 +17,17 @@ package MirrorCache::WebAPI::Controller::App::Myserver;
 use Mojo::Base 'MirrorCache::WebAPI::Controller::App::Table';
 
 sub index {
-    shift->SUPER::admintable('myserver');
+    my $c = shift;
+    my $mirror_provider = $c->mcconfig->mirror_provider;
+    if (my $url = $c->mcconfig->mirror_provider) {
+        $url =~ s!^https?://(?:www\.)?!!i;
+        $url =~ s!/.*!!;
+        $url =~ s/[\?\#\:].*//;
+        my $mirror_provider_url = 'https://' . $url . '/app/server';
+        $c->stash( mirror_provider_url => $mirror_provider_url );
+    }
+
+    $c->SUPER::admintable('myserver');
 }
 
 sub update {
diff --git a/lib/MirrorCache/WebAPI/Controller/App/Server.pm b/lib/MirrorCache/WebAPI/Controller/App/Server.pm
index c85214c5..8a722d29 100644
--- a/lib/MirrorCache/WebAPI/Controller/App/Server.pm
+++ b/lib/MirrorCache/WebAPI/Controller/App/Server.pm
@@ -17,7 +17,17 @@ package MirrorCache::WebAPI::Controller::App::Server;
 use Mojo::Base 'MirrorCache::WebAPI::Controller::App::Table';
 
 sub index {
-    shift->SUPER::admintable('server');
+    my $c = shift;
+    my $mirror_provider = $c->mcconfig->mirror_provider;
+    if (my $url = $c->mcconfig->mirror_provider) {
+        $url =~ s!^https?://(?:www\.)?!!i;
+        $url =~ s!/.*!!;
+        $url =~ s/[\?\#\:].*//;
+        my $mirror_provider_url = 'https://' . $url . '/app/server';
+        $c->stash( mirror_provider_url => $mirror_provider_url );
+    }
+
+    $c->SUPER::admintable('server');
 }
 
 sub update {
diff --git a/t/environ/01-smoke-mirror-provider.sh b/t/environ/01-smoke-mirror-provider.sh
index 6ee61b92..851f6081 100755
--- a/t/environ/01-smoke-mirror-provider.sh
+++ b/t/environ/01-smoke-mirror-provider.sh
@@ -8,9 +8,6 @@ mcnaeast=$(environ mc2 $(pwd))
 ap8=$(environ ap8)
 ap7=$(environ ap7)
 
-$mcmirror/gen_env MIRRORCACHE_MODE=mirror_provider
-
-
 $mcmirror/start
 
 $mcmirror/sql "insert into server(hostname,urldir,enabled,country,region) select '$($ap7/print_address)','','t','us','na'"
diff --git a/templates/app/myserver/index.html.ep b/templates/app/myserver/index.html.ep
index f8fbf336..9123d88b 100644
--- a/templates/app/myserver/index.html.ep
+++ b/templates/app/myserver/index.html.ep
@@ -17,6 +17,11 @@
         </div>
 
         %= include 'layouts/info'
+        % if (eval ('$mirror_provider_url')) {
+        <div id="regionmirrorproviderpanel" class="card">
+          <span>This view is readonly, it is imported from <a href=<%= eval('$mirror_provider_url') %>>mirror provider</a></span>
+        </div>
+        % }
 
         <table id="myservers" class="admintable table table-striped">
             <thead>
@@ -38,7 +43,7 @@
             <tbody>
             </tbody>
         </table>
-        % if (current_user) {
+        % if (current_user && !eval('$mirror_provider_url')) {
         <div class="text-center">
             <input value="New mirror" onclick="addAdminTableRow();" type="button" class="btn btn-default"/>
         </div>
diff --git a/templates/app/server/index.html.ep b/templates/app/server/index.html.ep
index daec1daf..51abb4ba 100644
--- a/templates/app/server/index.html.ep
+++ b/templates/app/server/index.html.ep
@@ -17,6 +17,11 @@
         </div>
 
         %= include 'layouts/info'
+        % if (eval ('$mirror_provider_url')) {
+        <div id="regionmirrorproviderpanel" class="card">
+          <span>This view is readonly, it is imported from <a href=<%= eval('$mirror_provider_url') %>>mirror provider</a></span>
+        </div>
+        % }
 
         <table id="servers" class="admintable table table-striped">
             <thead>
@@ -38,7 +43,7 @@
             <tbody>
             </tbody>
         </table>
-        % if (is_admin) {
+        % if (is_admin && !eval('$mirror_provider_url')) {
         <div class="text-center">
             <input value="New mirror" onclick="addAdminTableRow();" type="button" class="btn btn-default"/>
         </div>