Add PMD suppressions with reason where appropriate.

Author: phipag

powertools-logging/src/main/java/software/amazon/lambda/powertools/logging/internal/LambdaLoggingAspect.java

@@ -142,6 +142,7 @@ public Object around(ProceedingJoinPoint pjp,
                     isOnRequestStreamHandler);
         }
 
+        @SuppressWarnings("PMD.CloseResource") // Lambda-owned stream, not ours to close
         OutputStream backupOutputStream = null;
         if (isOnRequestStreamHandler) {
             // To log the result of a RequestStreamHandler (OutputStream), we need to do the following:
@@ -155,7 +156,7 @@ public Object around(ProceedingJoinPoint pjp,
         Object lambdaFunctionResponse;
         try {
             lambdaFunctionResponse = pjp.proceed(proceedArgs);
-        } catch (Throwable t) {
+        } catch (Throwable t) { // NOPMD - AspectJ proceed() throws Throwable
             handleException(pjp, logging, t);
             throw t;
         } finally {

powertools-logging/src/test/java/software/amazon/lambda/powertools/logging/internal/KeyBufferTest.java

@@ -334,9 +334,9 @@ void shouldUseDefaultWarningLoggerWhenNotProvided() {
         // Capture System.err output
         ByteArrayOutputStream errCapture = new ByteArrayOutputStream();
         PrintStream originalErr = System.err;
-        System.setErr(new PrintStream(errCapture));
+        try (PrintStream newErr = new PrintStream(errCapture)) {
+            System.setErr(newErr);
 
-        try {
             KeyBuffer<String, String> defaultBuffer = new KeyBuffer<>(5, String::length);
 
             // Cause overflow