fix(api): remove automatic backend failure for Down endpoints (#3160)

pull/3181/head
Anthony Lapenna 2019-09-20 16:13:58 +12:00 committed by GitHub
parent a5e5983c28
commit 07505fabcc
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 5 deletions

View File

@ -25,10 +25,6 @@ func (handler *Handler) proxyRequestsToDockerAPI(w http.ResponseWriter, r *http.
return &httperror.HandlerError{http.StatusInternalServerError, "Unable to find an endpoint with the specified identifier inside the database", err}
}
if endpoint.Type != portainer.EdgeAgentEnvironment && endpoint.Status == portainer.EndpointStatusDown {
return &httperror.HandlerError{http.StatusServiceUnavailable, "Unable to query endpoint", errors.New("Endpoint is down")}
}
err = handler.requestBouncer.AuthorizedEndpointOperation(r, endpoint, true)
if err != nil {
return &httperror.HandlerError{http.StatusForbidden, "Permission denied to access endpoint", err}

View File

@ -18,7 +18,8 @@ angular.module('portainer.app')
}
checkEndpointStatus(endpoint)
.then(function success() {
.then(function success(data) {
endpoint = data;
return switchToDockerEndpoint(endpoint);
}).catch(function error(err) {
Notifications.error('Failure', err, 'Unable to verify endpoint status');
@ -59,6 +60,7 @@ angular.module('portainer.app')
EndpointService.updateEndpoint(endpoint.Id, { Status: status })
.then(function success() {
endpoint.Status = status;
deferred.resolve(endpoint);
}).catch(function error(err) {
deferred.reject({ msg: 'Unable to update endpoint status', err: err });