remove unrelated modifications

RusJaI · Jul 10, 2024 · ee8e8be · ee8e8be
1 parent 335fda3
commit ee8e8be
Show file tree

Hide file tree

Showing 6 changed files with 2 additions and 41 deletions.
diff --git a/....carbon.apimgt.api/src/main/java/org/wso2/carbon/apimgt/api/dto/ClientCertificateDTO.java b/....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/...rbon.apimgt.gateway/src/main/java/org/wso2/carbon/apimgt/gateway/InMemoryAPIDeployer.java b/...rbon.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<String> certificateAliasListForAPI =
                         DataHolder.getInstance().getCertificateAliasListForAPI(gatewayDTO.getApiId());
-                certificateAliasListForAPI.addAll(certificateAliasListForAPI);
                 gatewayDTO.setClientCertificatesToBeRemove(certificateAliasListForAPI.toArray(new String[0]));
-
             }
         }
     }

diff --git a/.../main/java/org/wso2/carbon/apimgt/gateway/handlers/security/APIAuthenticationHandler.java b/.../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/....apimgt.gateway/src/main/java/org/wso2/carbon/apimgt/gateway/service/APIGatewayAdmin.java b/....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
@@ -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
@@ -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: |