Skip to content

Commit

Permalink
Merge pull request #2604 from cloudfoundry-incubator/investigate-2589
Browse files Browse the repository at this point in the history
Fix timing issue resulting in an endpoint warning
  • Loading branch information
KlapTrap authored Jul 3, 2018
2 parents 4b2f0f0 + d4a66b2 commit 0d81548
Showing 1 changed file with 13 additions and 4 deletions.
17 changes: 13 additions & 4 deletions src/frontend/app/store/effects/api.effects.ts
Original file line number Diff line number Diff line change
Expand Up @@ -101,10 +101,19 @@ export class APIEffect {
}

options.url = `/pp/${proxyAPIVersion}/proxy/${cfAPIVersion}/${options.url}`;
options.headers = this.addBaseHeaders(
apiAction.endpointGuid ||
state.requestData.endpoint, options.headers
);

const availableEndpoints = apiAction.endpointGuid || state.requestData.endpoint;
if (typeof availableEndpoints !== 'string') {
// Filter out endpoints that are currently being disconnected
const disconnectedEndpoints = Object.keys(availableEndpoints).
filter(endpointGuid => {
const updating = state.request.endpoint[endpointGuid].updating;
return !!updating.disconnecting && updating.disconnecting.busy;
});
disconnectedEndpoints.forEach(guid => delete availableEndpoints[guid]);
}

options.headers = this.addBaseHeaders(availableEndpoints, options.headers);

if (paginatedAction.flattenPagination) {
options.params.set('page', '1');
Expand Down

0 comments on commit 0d81548

Please sign in to comment.