Remove suppressAccessChecks permission from repository-s3

[dadoonet]

This morning I was looking at a problem reported on discuss and I came to this content:

https://github.com/elastic/elasticsearch/blob/master/plugins/repository-s3/src/main/plugin-metadata/plugin-security.policy#L25-L36

Apparently, we can remove this permission when aws/aws-sdk-java#766 is fixed. And this has been fixed a long time ago. I'm wondering then if this can be removed or if we should change the comment?

@tlrx Do you remember?

[elasticmachine]

Pinging @elastic/es-distributed (Team:Distributed)

[tlrx]

AWS SDK 1.11.749 cannot run without the suppressAccessChecks permission because InternalConfig still uses a Jackson's ObjectMapper with the ALLOW_FINAL_FIELDS_AS_MUTATORS feature which requires this permission.

[dadoonet]

Should aws/aws-sdk-java#766 be reopened or commented then?

[pugnascotia]

@tlrx I just found this issue because I am doing work on plugin installation. What is the path forward here?

[tlrx]

As far as I know this permission is still required, but the annoying logging traces have been muted in 7.10.0 (#62522)