diff --git a/components/apimgt/org.wso2.carbon.apimgt.api/src/main/java/org/wso2/carbon/apimgt/api/dto/ClientCertificateDTO.java b/components/apimgt/org.wso2.carbon.apimgt.api/src/main/java/org/wso2/carbon/apimgt/api/dto/ClientCertificateDTO.java index 41f86b34e05e..0d6313f93904 100644 --- a/components/apimgt/org.wso2.carbon.apimgt.api/src/main/java/org/wso2/carbon/apimgt/api/dto/ClientCertificateDTO.java +++ b/components/apimgt/org.wso2.carbon.apimgt.api/src/main/java/org/wso2/carbon/apimgt/api/dto/ClientCertificateDTO.java @@ -30,9 +30,6 @@ public class ClientCertificateDTO { private String tierName; private APIIdentifier apiIdentifier; - public ClientCertificateDTO() { - } - /** * To get the identifier of the API related with client certificate. * diff --git a/components/apimgt/org.wso2.carbon.apimgt.gateway/src/main/java/org/wso2/carbon/apimgt/gateway/InMemoryAPIDeployer.java b/components/apimgt/org.wso2.carbon.apimgt.gateway/src/main/java/org/wso2/carbon/apimgt/gateway/InMemoryAPIDeployer.java index 6c9b55897bf6..dbb9d0ecac21 100644 --- a/components/apimgt/org.wso2.carbon.apimgt.gateway/src/main/java/org/wso2/carbon/apimgt/gateway/InMemoryAPIDeployer.java +++ b/components/apimgt/org.wso2.carbon.apimgt.gateway/src/main/java/org/wso2/carbon/apimgt/gateway/InMemoryAPIDeployer.java @@ -356,7 +356,6 @@ private void unDeployAPI(APIGatewayAdmin apiGatewayAdmin, DeployAPIInGatewayEven .addStringToList(gatewayEvent.getUuid(), gatewayAPIDTO.getLocalEntriesToBeRemove())); apiGatewayAdmin.unDeployAPI(gatewayAPIDTO); DataHolder.getInstance().getApiToCertificatesMap().remove(gatewayEvent.getUuid()); - DataHolder.getInstance().removeKeyManagerToAPIMapping(gatewayAPIDTO.getApiId()); } } @@ -453,9 +452,7 @@ private void setClientCertificatesToRemoveIntoGatewayDTO(GatewayAPIDTO gatewayDT if (StringUtils.isNotEmpty(gatewayDTO.getApiId())) { List certificateAliasListForAPI = DataHolder.getInstance().getCertificateAliasListForAPI(gatewayDTO.getApiId()); - certificateAliasListForAPI.addAll(certificateAliasListForAPI); gatewayDTO.setClientCertificatesToBeRemove(certificateAliasListForAPI.toArray(new String[0])); - } } } diff --git a/components/apimgt/org.wso2.carbon.apimgt.gateway/src/main/java/org/wso2/carbon/apimgt/gateway/handlers/security/APIAuthenticationHandler.java b/components/apimgt/org.wso2.carbon.apimgt.gateway/src/main/java/org/wso2/carbon/apimgt/gateway/handlers/security/APIAuthenticationHandler.java index 4676a5b6b075..e68aca00112d 100644 --- a/components/apimgt/org.wso2.carbon.apimgt.gateway/src/main/java/org/wso2/carbon/apimgt/gateway/handlers/security/APIAuthenticationHandler.java +++ b/components/apimgt/org.wso2.carbon.apimgt.gateway/src/main/java/org/wso2/carbon/apimgt/gateway/handlers/security/APIAuthenticationHandler.java @@ -97,7 +97,6 @@ public class APIAuthenticationHandler extends AbstractHandler implements Managed private String apiKeyHeader; private String apiSecurity; private String apiLevelPolicy; - private String environmentType; private String certificateInformation; private String apiUUID; private String apiType = String.valueOf(APIConstants.ApiTypes.API); // Default API Type @@ -152,24 +151,6 @@ public void setAPILevelPolicy(String apiLevelPolicy) { this.apiLevelPolicy = apiLevelPolicy; } - /** - * To get the environment type (whether production or sandbox). - * - * @return the environment type. - */ - public String getEnvironmentType() { - return environmentType; - } - - /** - * To set the environment type (whether production or sandbox). - * - * @param environmentType the environment type. - */ - public void setEnvironmentType(String environmentType) { - this.environmentType = environmentType; - } - /** * Get type of the API * @return API Type diff --git a/components/apimgt/org.wso2.carbon.apimgt.gateway/src/main/java/org/wso2/carbon/apimgt/gateway/service/APIGatewayAdmin.java b/components/apimgt/org.wso2.carbon.apimgt.gateway/src/main/java/org/wso2/carbon/apimgt/gateway/service/APIGatewayAdmin.java index 566d39bc0a54..e9c66c45601b 100644 --- a/components/apimgt/org.wso2.carbon.apimgt.gateway/src/main/java/org/wso2/carbon/apimgt/gateway/service/APIGatewayAdmin.java +++ b/components/apimgt/org.wso2.carbon.apimgt.gateway/src/main/java/org/wso2/carbon/apimgt/gateway/service/APIGatewayAdmin.java @@ -715,7 +715,6 @@ public boolean deployAPI(GatewayAPIDTO gatewayAPIDTO) throws AxisFault { } } } - if (log.isDebugEnabled()) { log.debug(gatewayAPIDTO.getName() + ":" + gatewayAPIDTO.getVersion() + " client certificates deployed"); log.debug("Start to add vault entries " + gatewayAPIDTO.getName() + ":" + gatewayAPIDTO.getVersion()); @@ -860,7 +859,6 @@ private void unDeployAPI(SequenceAdminServiceProxy sequenceAdminServiceProxy, } } } - if (log.isDebugEnabled()) { log.debug(gatewayAPIDTO.getName() + ":" + gatewayAPIDTO.getVersion() + " client certificates undeployed " + "successfully"); diff --git a/components/apimgt/org.wso2.carbon.apimgt.internal.service/swagger.json b/components/apimgt/org.wso2.carbon.apimgt.internal.service/swagger.json index 8040e385bd35..a7fa59630d63 100644 --- a/components/apimgt/org.wso2.carbon.apimgt.internal.service/swagger.json +++ b/components/apimgt/org.wso2.carbon.apimgt.internal.service/swagger.json @@ -1354,8 +1354,8 @@ "type" : "string", "example" : "EXCHANGED", "description" : "The type of the tokens to be used (exchanged or without exchanged). Accepted values are EXCHANGED, DIRECT or BOTH.", - "default" : "DIRECT", - "enum" : [ "EXCHANGED", "DIRECT", "BOTH" ] + "enum" : [ "EXCHANGED", "DIRECT", "BOTH" ], + "default" : "DIRECT" } } }, diff --git a/components/apimgt/org.wso2.carbon.apimgt.rest.api.common/src/main/resources/admin-api.yaml b/components/apimgt/org.wso2.carbon.apimgt.rest.api.common/src/main/resources/admin-api.yaml index 8dff2ebf0c29..9aa4db9ea6a8 100644 --- a/components/apimgt/org.wso2.carbon.apimgt.rest.api.common/src/main/resources/admin-api.yaml +++ b/components/apimgt/org.wso2.carbon.apimgt.rest.api.common/src/main/resources/admin-api.yaml @@ -1330,18 +1330,6 @@ paths: Retrieves all existing deny policies. parameters: - $ref: '#/components/parameters/Accept' - - name: query - in: query - description: | - **Search condition**. - You can search in attributes by using **"conditionType:"** modifier and **"conditionValue:"** modifier. - Eg. - The entry "conditionType:API" will result in a match with blocking conditions only if the conditionType is "API". Similarly, "conditionValue:test/1.0.0" will result in a match with blocking conditions only if the conditionValue is "test/1.0.0". - When you use "conditionType:API & conditionValue:test/1.0.0" as a combination, it will result in a match with blocking conditions only if both the conditionType is "API" and the conditionValue is "test/1.0.0". - If query attribute is provided, this returns the blocking conditions that match the specified attributes. - Please note that you need to use encoded URL (URL encoding) if you are using a client which does not support URL encoding (such as curl) - schema: - type: string responses: 200: description: |