fixes #81 add pre-commit hook and apply it (#82)

Author: stevehu

.pre-commit-config.yaml

@@ -0,0 +1,15 @@
+repos:
+-   repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v3.2.0
+    hooks:
+    -   id: trailing-whitespace
+    -   id: end-of-file-fixer
+    -   id: check-yaml
+        args: ["--unsafe"]
+    -   id: check-added-large-files
+-   repo: https://github.com/networknt/pre-commit-hook-keyword
+    rev: f17c4de14fc24420f6768c19cad06ba03af06d86
+    hooks:
+    -   id: keywordscan
+        args: ["--keywords=c3VubGlmZQ==,Y2liYw==,c3VuIGxpZmU="]
+        types: ["text"]

README.md

@@ -3,7 +3,7 @@ Aws lambda authorizer, middleware handlers and extensions for cross-cutting conc
 
 ### authorizer
 
-The authorizer is a lambda function and it will be configured on the AWS API Gateway to be invoked as a custom authorizer. 
+The authorizer is a lambda function and it will be configured on the AWS API Gateway to be invoked as a custom authorizer.
 
 
 ### middleware handlers
@@ -29,6 +29,3 @@ This is a Lambda extersion that will collect the logs of the Lambda function and
 ##### Metrics
 
 This is a Lambda extension that will collect the metrics of the Lambda function and send to a time series database.
-
-
-

ccc-lambda/README.md

@@ -1,2 +1 @@
 This is a Lambda function that address cross-cutting concerns for business Lambda functions. It is a Lambda function that can be used to build business Lambda functions.
-

ccc-lambda/middleware-function/README.md

@@ -1,6 +1,6 @@
 ### Test with Main
 
-We have provided a Main class in the src folder for testing all the middleware handlers with a sample request. You can run the Main class within your IDE with debug mode to see how each middleware handler works. 
+We have provided a Main class in the src folder for testing all the middleware handlers with a sample request. You can run the Main class within your IDE with debug mode to see how each middleware handler works.
 
 Before running the Main class, you need to disable the DynamoDbCacheManager as it depends on the AWS environment. To do that you need to common out the service.yml section in the values.yml
 
@@ -20,7 +20,7 @@ client.tokenKeyServerUrl: https://networknt.oktapreview.com
 client.tokenKeyUri: /oauth2/aus66u5cybTrCsbZs1d6/v1/keys
 ```
 
-If everything works fine, you should see the following error that indicate all middleware handlers are passed and the AWS Lambda invocation is failed. This is because we are testing without AWS environment. 
+If everything works fine, you should see the following error that indicate all middleware handlers are passed and the AWS Lambda invocation is failed. This is because we are testing without AWS environment.
 
 ```
 22:38:36.103 [main] DEBUG software.amazon.awssdk.auth.credentials.AwsCredentialsProviderChain -- Unable to load credentials from SystemPropertyCredentialsProvider(): Unable to load credentials from system settings. Access key must be specified either via environment variable (AWS_ACCESS_KEY_ID) or system property (aws.accessKeyId).

ccc-lambda/middleware-function/pom.xml

@@ -12,7 +12,7 @@
         <maven.compiler.source>${java.version}</maven.compiler.source>
         <maven.compiler.target>${java.version}</maven.compiler.target>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
-        <version.light-4j>2.1.31-SNAPSHOT</version.light-4j>
+        <version.light-4j>2.1.32-SNAPSHOT</version.light-4j>
         <version.http-client>1.0.6</version.http-client>
         <version.jackson>2.15.2</version.jackson>
         <version.slf4j>2.0.9</version.slf4j>

ccc-lambda/middleware-function/resource-config.json

@@ -8,5 +8,5 @@
     ],
     "excludes": [
     ]
-  } 
+  }
 }

ccc-lambda/middleware-function/src/main/resources/config/lambda-proxy.yml

@@ -7,4 +7,4 @@ metricsName: ${lambda-proxy.metricsName:}
 lambdaAppId: ${lambda-proxy.lambdaAppId:}
 requestChain: ${lambda-proxy.requestChain:}
 responseChain: ${lambda-proxy.responseChain:}
-enableDynamoDbCache: ${lambda-proxy.enableDynamoDbCache:false}
+enableDynamoDbCache: ${lambda-proxy.enableDynamoDbCache:false}

ccc-lambda/middleware-function/src/main/resources/config/pooled-chain-executor.yml

@@ -3,4 +3,4 @@ corePoolSize: ${pooled-chain-executor.corePoolSize:10}
 keepAliveTime: ${pooled-chain-executor.keepAliveTime:0}
 maxChainSize: ${pooled-chain-executor.maxChainSize:20}
 forceSynchronousExecution: ${pooled-chain-executor.forceSynchronousExecution:false}
-exitOnMiddlewareInstanceCreationFailure: ${pooled-chain-executor.exitOnMiddlewareInstanceCreationFailure:true}
+exitOnMiddlewareInstanceCreationFailure: ${pooled-chain-executor.exitOnMiddlewareInstanceCreationFailure:true}

ccc-lambda/middleware-function/src/test/java/com/networknt/aws/lambda/TestContext.java

@@ -132,4 +132,4 @@ public void log(byte[] bytes) {
             System.err.println(new String(bytes, StandardCharsets.UTF_8));
         }
     }
-}
+}

ccc-lambda/middleware-function/src/test/java/com/networknt/aws/lambda/middleware/TestAsynchronousMiddleware.java

@@ -39,4 +39,3 @@ public void getCachedConfigurations() {
 
     }
 }
-