You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by al...@apache.org on 2018/08/08 14:18:59 UTC

[mesos] branch master updated: Removed long ago deprecated endpoints.

This is an automated email from the ASF dual-hosted git repository.

alexr pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mesos.git


The following commit(s) were added to refs/heads/master by this push:
     new 42551cb  Removed long ago deprecated endpoints.
42551cb is described below

commit 42551cb5290b7b04101f7d800b4b8fd573e47b91
Author: Alexander Rukletsov <al...@apache.org>
AuthorDate: Wed Aug 8 16:18:48 2018 +0200

    Removed long ago deprecated endpoints.
---
 src/files/files.cpp   | 19 -------------------
 src/master/master.cpp | 30 ------------------------------
 src/slave/slave.cpp   | 21 ---------------------
 3 files changed, 70 deletions(-)

diff --git a/src/files/files.cpp b/src/files/files.cpp
index a64613c..f200d5e 100644
--- a/src/files/files.cpp
+++ b/src/files/files.cpp
@@ -222,25 +222,6 @@ FilesProcess::FilesProcess(
 
 void FilesProcess::initialize()
 {
-    // TODO(ijimenez): Remove these endpoints at the end of the
-    // deprecation cycle on 0.26.
-    route("/browse.json",
-          authenticationRealm,
-          FilesProcess::BROWSE_HELP,
-          &FilesProcess::loggedBrowse);
-    route("/read.json",
-          authenticationRealm,
-          FilesProcess::READ_HELP,
-          &FilesProcess::loggedRead);
-    route("/download.json",
-          authenticationRealm,
-          FilesProcess::DOWNLOAD_HELP,
-          &FilesProcess::loggedDownload);
-    route("/debug.json",
-          authenticationRealm,
-          FilesProcess::DEBUG_HELP,
-          &FilesProcess::loggedDebug);
-
     route("/browse",
           authenticationRealm,
           FilesProcess::BROWSE_HELP,
diff --git a/src/master/master.cpp b/src/master/master.cpp
index 21de973..400a83e 100644
--- a/src/master/master.cpp
+++ b/src/master/master.cpp
@@ -937,16 +937,6 @@ void Master::initialize()
           logRequest(request);
           return http.reserve(request, principal);
         });
-  // TODO(ijimenez): Remove this endpoint at the end of the
-  // deprecation cycle on 0.26.
-  route("/roles.json",
-        READONLY_HTTP_AUTHENTICATION_REALM,
-        Http::ROLES_HELP(),
-        [this](const process::http::Request& request,
-               const Option<Principal>& principal) {
-          logRequest(request);
-          return http.roles(request, principal);
-        });
   route("/roles",
         READONLY_HTTP_AUTHENTICATION_REALM,
         Http::ROLES_HELP(),
@@ -971,16 +961,6 @@ void Master::initialize()
           logRequest(request);
           return http.slaves(request, principal);
         });
-  // TODO(ijimenez): Remove this endpoint at the end of the
-  // deprecation cycle on 0.26.
-  route("/state.json",
-        READONLY_HTTP_AUTHENTICATION_REALM,
-        Http::STATE_HELP(),
-        [this](const process::http::Request& request,
-               const Option<Principal>& principal) {
-          logRequest(request);
-          return http.state(request, principal);
-        });
   route("/state",
         READONLY_HTTP_AUTHENTICATION_REALM,
         Http::STATE_HELP(),
@@ -997,16 +977,6 @@ void Master::initialize()
           logRequest(request);
           return http.stateSummary(request, principal);
         });
-  // TODO(ijimenez): Remove this endpoint at the end of the
-  // deprecation cycle.
-  route("/tasks.json",
-        READONLY_HTTP_AUTHENTICATION_REALM,
-        Http::TASKS_HELP(),
-        [this](const process::http::Request& request,
-               const Option<Principal>& principal) {
-          logRequest(request);
-          return http.tasks(request, principal);
-        });
   route("/tasks",
         READONLY_HTTP_AUTHENTICATION_REALM,
         Http::TASKS_HELP(),
diff --git a/src/slave/slave.cpp b/src/slave/slave.cpp
index f603016..78e8666 100644
--- a/src/slave/slave.cpp
+++ b/src/slave/slave.cpp
@@ -793,17 +793,6 @@ void Slave::initialize()
 
         return resourceProviderManager->api(request, principal);
       });
-
-  // TODO(ijimenez): Remove this endpoint at the end of the
-  // deprecation cycle on 0.26.
-  route("/state.json",
-        READONLY_HTTP_AUTHENTICATION_REALM,
-        Http::STATE_HELP(),
-        [this](const http::Request& request,
-               const Option<Principal>& principal) {
-          logRequest(request);
-          return http.state(request, principal);
-        });
   route("/state",
         READONLY_HTTP_AUTHENTICATION_REALM,
         Http::STATE_HELP(),
@@ -833,16 +822,6 @@ void Slave::initialize()
           logRequest(request);
           return http.statistics(request, principal);
         });
-  // TODO(ijimenez): Remove this endpoint at the end of the
-  // deprecation cycle on 0.26.
-  route("/monitor/statistics.json",
-        READONLY_HTTP_AUTHENTICATION_REALM,
-        Http::STATISTICS_HELP(),
-        [this](const http::Request& request,
-               const Option<Principal>& principal) {
-          logRequest(request);
-          return http.statistics(request, principal);
-        });
   route("/containers",
         READONLY_HTTP_AUTHENTICATION_REALM,
         Http::CONTAINERS_HELP(),