Remove usage of setAccessible

[scottgerring]

We use java.lang.reflect.AcessibleObject#setAccessible to reflectively access fields in both the matrics and SQS utilities. We should try and remove this, if we can.

What were you trying to accomplish?

Expected Behavior

No need to reflectively access private object information in the libraries themselves. In tests this is fine.

Current Behavior

Examples:

powertools-lambda-java/powertools-metrics/src/main/java/software/amazon/cloudwatchlogs/emf/model/MetricsLoggerHelper.java

Line 36 in 4d3484f

f.setAccessible(true);

powertools-lambda-java/powertools-sqs/src/main/java/software/amazon/lambda/powertools/sqs/SqsUtils.java

Line 619 in 4d3484f

constructor.setAccessible(true);

powertools-lambda-java/powertools-metrics/src/main/java/software/amazon/lambda/powertools/metrics/internal/LambdaMetricsAspect.java

Line 56 in 4d3484f

f.setAccessible(true);

Possible Solution

We can either 1/ find a way to avoid the need or 2/ say "this is too hard, we will remove it in V2 by changing the interface". However we should be confident that there is no way forward with the existing interface.

Steps to Reproduce (for bugs)

Environment

• Powertools for AWS Lambda (Java) version used: 1.17.0

[scottgerring]

For the metrics one, what are we doing in the v2 rewrite #1405 ? We certainly shouldn't introduce this again.

[scottgerring]

For the SQS one, we might be able to ignore it; we have deprecated it and introduced the new SQS libraries #1317