Skip to content

Commit b0e42dd

Browse files
committed
DDF-2644 Changed revert endpoint path to allow security (#42)
1 parent d8204ed commit b0e42dd

File tree

6 files changed

+16
-8
lines changed

6 files changed

+16
-8
lines changed

catalog/ui/catalog-ui-search/src/main/java/org/codice/ddf/catalog/ui/metacard/MetacardApplication.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -251,7 +251,7 @@ public void init() {
251251
return util.getJson(response);
252252
});
253253

254-
get("/history/:id/revert/:revertid", (req, res) -> {
254+
get("/history/revert/:id/:revertid", (req, res) -> {
255255
String id = req.params(":id");
256256
String revertId = req.params(":revertid");
257257

catalog/ui/catalog-ui-search/src/main/java/org/codice/ddf/catalog/ui/metacard/enumerations/ExperimentalEnumerationExtractor.java

+1-2
Original file line numberDiff line numberDiff line change
@@ -81,8 +81,7 @@ public Map<String, Set<String>> getEnumerations(@Nullable String metacardType) {
8181
.stream()
8282
.flatMap(ad -> attributeValidatorRegistry.getValidators(ad.getName())
8383
.stream()
84-
.map(av -> av.validate(new AttributeImpl(ad.getName(),
85-
"null"))))
84+
.map(av -> av.validate(new AttributeImpl(ad.getName(), "null"))))
8685
.filter(Optional::isPresent)
8786
.map(Optional::get)
8887
.filter(avr -> !avr.getSuggestedValues()

catalog/ui/catalog-ui-search/src/main/webapp/component/metacard-archive/metacard-archive.view.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -119,8 +119,9 @@ define([
119119
var loadingView = new LoadingView();
120120
$.whenAll.apply(this, this.model.map(function(result) {
121121
return $.get('/search/catalog/internal/history/' +
122+
'revert/' +
122123
result.get('metacard').get('properties').get('metacard.deleted.id') +
123-
'/revert/' +
124+
'/' +
124125
result.get('metacard').get('properties').get('metacard.deleted.version')).then(function(response) {
125126
ResultUtils.refreshResult(result);
126127
}.bind(this));

catalog/ui/catalog-ui-search/src/main/webapp/component/metacard-history/metacard-history.view.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ define([
9696
revertToSelectedVersion: function(){
9797
LoadingCompanionView.beginLoading(this);
9898
var self = this;
99-
$.get('/search/catalog/internal/history/'+this.model.get('metacard').get('id')+'/revert/'+selectedVersion).then(function(response){
99+
$.get('/search/catalog/internal/history/revert/'+this.model.get('metacard').get('id')+'/'+selectedVersion).then(function(response){
100100
self.model.get('metacard').get('properties').set(response.metacards[0]);
101101
}).always(function(){
102102
setTimeout(function(){ //let solr flush

catalog/ui/catalog-ui-search/src/main/webapp/component/metacard-restore/metacard-restore.view.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ define([
6666

6767
var historyId = model.get('metacard').get('properties').get('metacard.version.id');
6868
var metacardId = model.get('metacard').get('id');
69-
var revert = '/search/catalog/internal/history/' + historyId + '/revert/' + metacardId;
69+
var revert = '/search/catalog/internal/history/revert/' + historyId + '/' + metacardId;
7070

7171
$.get(revert).then(function () {
7272
model.set({

catalog/ui/catalog-ui-search/src/main/webapp/main.js

+10-2
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,17 @@ require([
4141
}
4242

4343
if (!settings.customErrorHandling) {
44+
var defaultTitle = 'Server Error';
45+
var defaultMessage = 'Unknown error.';
46+
47+
if (jqxhr.status === 403) {
48+
defaultTitle = 'Forbidden';
49+
defaultMessage = 'Not Authorized';
50+
}
51+
4452
announcement.announce({
45-
title: 'Server Error',
46-
message: message || 'Unknown error.',
53+
title: defaultTitle,
54+
message: message || defaultMessage,
4755
type: 'error'
4856
});
4957
}

0 commit comments

Comments
 (0)