X-Git-Url: https://gerrit.o-ran-sc.org/r/gitweb?a=blobdiff_plain;f=webapp-backend%2Fsrc%2Fmain%2Fjava%2Forg%2Foransc%2Fric%2Fportal%2Fdashboard%2Fcontroller%2FAppManagerController.java;h=2e1aaea09606afd5733754ff29b999940cf835c0;hb=847b44ffdd48cd1fa6b201887d3728df54305891;hp=f71c6bd93cd61b6868206584239f52be9810b014;hpb=3f812ea25d352ec33d07f5ffa4c2aa2a77e8e793;p=portal%2Fric-dashboard.git diff --git a/webapp-backend/src/main/java/org/oransc/ric/portal/dashboard/controller/AppManagerController.java b/webapp-backend/src/main/java/org/oransc/ric/portal/dashboard/controller/AppManagerController.java index f71c6bd9..2e1aaea0 100644 --- a/webapp-backend/src/main/java/org/oransc/ric/portal/dashboard/controller/AppManagerController.java +++ b/webapp-backend/src/main/java/org/oransc/ric/portal/dashboard/controller/AppManagerController.java @@ -56,9 +56,12 @@ import org.springframework.web.bind.annotation.RestController; import io.swagger.annotations.ApiOperation; /** - * Proxies calls from the front end to the App Manager API. All methods answer - * 502 on failure:
HTTP server received an invalid response from a - * server it consulted when acting as a proxy or gateway.
+ * Proxies calls from the front end to the App Manager API. + * + * If a method throws RestClientResponseException, it is handled by + * {@link CustomResponseEntityExceptionHandler#handleProxyMethodException(Exception, org.springframework.web.context.request.WebRequest)} + * which returns status 502. All other exceptions are handled by Spring which + * returns status 500. */ @Configuration @RestController @@ -157,7 +160,7 @@ public class AppManagerController { @Secured({ DashboardConstants.ROLE_ADMIN, DashboardConstants.ROLE_STANDARD }) public Object getAvailableXapps() { logger.debug("getAvailableXapps"); - AllDeployableXapps appNames = xappApi.listAllXapps(); + AllDeployableXapps appNames = xappApi.listAllDeployableXapps(); // Answer a collection of structure instead of string // because I expect the AppMgr to be extended with // additional properties for each one. @@ -197,7 +200,7 @@ public class AppManagerController { public void undeployXapp(@PathVariable("xAppName") String xAppName, HttpServletResponse response) { logger.debug("undeployXapp {}", xAppName); xappApi.undeployXapp(xAppName); - response.setStatus(healthApi.getApiClient().getStatusCode().value()); + response.setStatus(xappApi.getApiClient().getStatusCode().value()); } }