8000 Removed redirects for some cluster rest handler (#14425) · arangodb/arangodb@7560f36 · GitHub
[go: up one dir, main page]
Skip to content

Commit 7560f36

Browse files
Lars Maierjsteemann
Lars Maier
and
jsteemann
committed
Removed redirects for some cluster rest handler (#14425)
Co-authored-by: jsteemann <jsteemann@users.noreply.github.com>
1 parent b8280fa commit 7560f36

File tree

10 files changed

+19
-147
lines changed

10 files changed

+19
-147
lines changed

‎CHANGELOG‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
11
devel
22
-----
33

4+
* Removed redirects from /_admin/cluster* to /_admin/cluster/*. Adjusted
5+
internal requests to use the new url.
6+
47
* Fix display of running and slow queries in web UI when there are multiple
58
coordinators. Previously, the display order of queries was undefined, which
69
could lead to queries from one coordinator being display on top once and

‎Documentation/DocuBlocks/Rest/Cluster/get_cluster_statistics.md‎

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
@startDocuBlock get_cluster_statistics
33
@brief allows to query the statistics of a DB-Server in the cluster
44

5-
@RESTHEADER{GET /_admin/clusterStatistics, Queries statistics of a DB-Server}
5+
@RESTHEADER{GET /_admin/cluster/statistics, Queries statistics of a DB-Server}
66

77
@RESTQUERYPARAMETERS
88

@@ -13,10 +13,12 @@ Queries the statistics of the given DB-Server
1313

1414
@RESTRETURNCODES
1515

16-
@RESTRETURNCODE{200} is returned when everything went well.
16+
@RESTRETURNCODE{200}
17+
is returned when everything went well.
1718

18-
@RESTRETURNCODE{400} the parameter DBserver was not given or is not the
19-
ID of a DB-Server
19+
@RESTRETURNCODE{400}
20+
the parameter DBserver was not given or is not the ID of a DB-Server
2021

21-
@RESTRETURNCODE{403} server is not a Coordinator.
22+
@RESTRETURNCODE{403}
23+
server is not a DB-Server.
2224
@endDocuBlock

‎arangod/CMakeLists.txt‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -737,7 +737,6 @@ set(LIB_ARANGOSERVER_SOURCES
737737
RestHandler/RestPleaseUpgradeHandler.cpp
738738
RestHandler/RestQueryCacheHandler.cpp
739739
RestHandler/RestQueryHandler.cpp
740-
RestHandler/RestRedirectHandler.cpp
741740
RestHandler/RestShutdownHandler.cpp
742741
RestHandler/RestSimpleHandler.cpp
743742
RestHandler/RestSimpleQueryHandler.cpp

‎arangod/GeneralServer/GeneralServerFeature.cpp‎

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,6 @@
8282
#include "RestHandler/RestPregelHandler.h"
8383
#include "RestHandler/RestQueryCacheHandler.h"
8484
#include "RestHandler/RestQueryHandler.h"
85-
#include "RestHandler/RestRedirectHandler.h"
8685
#include "RestHandler/RestShutdownHandler.h"
8786
#include "RestHandler/RestSimpleHandler.h"
8887
#include "RestHandler/RestSimpleQueryHandler.h"
@@ -638,25 +637,6 @@ void GeneralServerFeature::defineHandlers() {
638637

639638
_handlerFactory->addPrefixHandler("/", RestHandlerCreator<RestActionHandler>::createNoData);
640639

641-
// ...........................................................................
642-
// redirects
643-
// ...........................................................................
644-
645-
// UGLY HACK INCOMING!
646-
647-
#define ADD_REDIRECT(from, to) \
648-
do { \
649-
_handlerFactory->addPrefixHandler( \
650-
from, RestHandlerCreator<RestRedirectHandler>::createData<const char*>, \
651-
(void*)(to)); \
652-
} while (0)
653-
654-
ADD_REDIRECT("/_admin/clusterNodeVersion", "/_admin/cluster/nodeVersion");
655-
ADD_REDIRECT("/_admin/clusterNodeEngine", "/_admin/cluster/nodeEngine");
656-
ADD_REDIRECT("/_admin/clusterNodeStats", "/_admin/cluster/nodeStatistics");
657-
ADD_REDIRECT("/_admin/clusterStatistics", "/_admin/cluster/statistics");
658-
659-
660640
// engine specific handlers
661641
StorageEngine& engine = server().getFeature<EngineSelectorFeature>().engine();
662642
engine.addRestHandlers(*_handlerFactory);

‎arangod/RestHandler/RestRedirectHandler.cpp‎

Lines changed: 0 additions & 44 deletions
This file was deleted.

‎arangod/RestHandler/RestRedirectHandler.h‎

Lines changed: 0 additions & 47 deletions
This file was deleted.

‎js/apps/system/_admin/aardvark/APP/frontend/js/views/clusterView.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@
129129
var stat = new window.Statistics({name: dbserver.id});
130130
stat.url = coord.get('protocol') + '://' +
131131
coord.get('address') +
132-
'/_admin/clusterStatistics?DBserver=' +
132+
'/_admin/cluster/statistics?DBserver=' +
133133
dbserver.get('name');
134134
self.statCollectDBS.add(stat);
135135
});

‎js/apps/system/_admin/aardvark/APP/frontend/js/views/showClusterView.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -331,7 +331,7 @@
331331

332332
stat.url = coord.get('protocol') + '://' +
333333
coord.get('address') +
334-
'/_admin/clusterStatistics?DBserver=' +
334+
'/_admin/cluster/statistics?DBserver=' +
335335
dbserver.get('name');
336336

337337
statCollect.add(stat);

‎tests/js/client/authentication/admin-cluster-apis.js‎

Lines changed: 3 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -51,31 +51,19 @@ function AuthSuite() {
5151
};
5252

5353
let checkClusterNodeVersion = function() {
54-
// this will just redirect to a new endpoint
55-
let result = arango.GET('/_admin/clusterNodeVersion?ServerID=' + servers[0]);
56-
assertEqual(308, result.code);
57-
58-
result = arango.GET('/_admin/cluster/nodeVersion?ServerID=' + servers[0]);
54+
let result = arango.GET('/_admin/cluster/nodeVersion?ServerID=' + servers[0]);
5955
assertTrue(result.hasOwnProperty('server'), result);
6056
assertTrue(result.hasOwnProperty('license'), result);
6157
assertTrue(result.hasOwnProperty('version'), result);
6258
};
6359

6460
let checkClusterNodeEngine = function() {
65-
// this will just redirect to a new endpoint
66-
let result = arango.GET('/_admin/clusterNodeEngine?ServerID=' + servers[0]);
67-
assertEqual(308, result.code);
68-
69-
result = arango.GET('/_admin/cluster/nodeEngine?ServerID=' + servers[0]);
61+
let result = arango.GET('/_admin/cluster/nodeEngine?ServerID=' + servers[0]);
7062
assertTrue(result.hasOwnProperty('name'), result);
7163
};
7264

7365
let checkClusterNodeStats = function() {
74-
// this will just redirect to a new endpoint
75-
let result = arango.GET('/_admin/clusterNodeStats?ServerID=' + servers[0]);
76-
assertEqual(308, result.code);
77-
78-
result = arango.GET('/_admin/cluster/nodeStatistics?ServerID=' + servers[0]);
66+
let result = arango.GET('/_admin/cluster/nodeStatistics?ServerID=' + servers[0]);
7967
assertTrue(result.hasOwnProperty('time'), result);
8068
assertTrue(result.hasOwnProperty('system'), result);
8169
};

‎tests/js/server/shell/shell-readonly-mode-spec.js‎

Lines changed: 4 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -153,10 +153,7 @@ describe('Readonly mode api', function() {
153153
}
154154

155155
let server = setReadOnlyAndGetDBServer();
156-
let resp = download(endpoint + '/_admin/clusterNodeVersion?ServerID=' + server);
157-
expect(resp.code).to.equal(308);
158-
159-
resp = download(endpoint + '/_admin/cluster/nodeVersion?ServerID=' + server);
156+
let resp = download(endpoint + '/_admin/cluster/nodeVersion?ServerID=' + server);
160157
expect(resp.code).to.equal(200);
161158
});
162159

@@ -166,23 +163,17 @@ describe('Readonly mode api', function() {
166163
}
167164

168165
let server = setReadOnlyAndGetDBServer();
169-
let resp = download(endpoint + '/_admin/clusterNodeEngine?ServerID=' + server);
170-
expect(resp.code).to.equal(308);
171-
172-
resp = download(endpoint + '/_admin/cluster/nodeEngine?ServerID=' + server);
166+
let resp = download(endpoint + '/_admin/cluster/nodeEngine?ServerID=' + server);
173167
expect(resp.code).to.equal(200);
174168
});
175169

176-
it('can still access cluster/nodeStats API when readonly', function() {
170+
it('can still access cluster/nodeStatistics API when readonly', function() {
177171
if (!isCluster) {
178172
return;
179173
}
180174

181175
let server = setReadOnlyAndGetDBServer();
182-
let resp = download(endpoint + '/_admin/clusterNodeStats?ServerID=' + server);
183-
expect(resp.code).to.equal(308);
184-
185-
resp = download(endpoint + '/_admin/cluster/nodeStatistics?ServerID=' + server);
176+
let resp = download(endpoint + '/_admin/cluster/nodeStatistics?ServerID=' + server);
186177
expect(resp.code).to.equal(200);
187178
});
188179

0 commit comments

Comments
 (0)
0