Feature branch sync up with main branch

[RyanL1997]

Description

Feature branch sync up with main

Check List

• New functionality includes testing
• New functionality has been documented
• Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

[DarshitChanpura]

Are we still pushing changes to this branch?

[codecov]

Codecov Report

Merging #2847 (d2a5e9d) into feature/extensions (56e77fe) will increase coverage by 0.16%.
The diff coverage is 71.62%.

❗ Current head d2a5e9d differs from pull request most recent head 9af7290. Consider uploading reports for the commit 9af7290 to get more accurate results

@@                   Coverage Diff                    @@
##             feature/extensions    #2847      +/-   ##
========================================================
+ Coverage                 61.49%   61.66%   +0.16%     
- Complexity                 3428     3441      +13     
========================================================
  Files                       269      269              
  Lines                     19096    19124      +28     
  Branches                   3327     3330       +3     
========================================================
+ Hits                      11744    11793      +49     
+ Misses                     5740     5720      -20     
+ Partials                   1612     1611       -1     

Impacted Files	Coverage Δ
.../main/java/com/amazon/dlic/auth/ldap/LdapUser.java	85.71% <ø> (ø)
...c/auth/ldap/backend/LDAPAuthenticationBackend.java	81.73% <ø> (ø)
...ic/auth/ldap/backend/LDAPAuthorizationBackend.java	57.81% <ø> (ø)
...on/dlic/auth/ldap2/LDAPAuthenticationBackend2.java	67.96% <ø> (ø)
.../dlic/auth/ldap2/LDAPConnectionFactoryFactory.java	57.46% <ø> (ø)
...a/org/opensearch/security/DefaultObjectMapper.java	64.55% <ø> (ø)
.../opensearch/security/OpenSearchSecurityPlugin.java	80.31% <ø> (ø)
.../action/configupdate/ConfigUpdateNodeResponse.java	77.27% <ø> (ø)
...tion/configupdate/TransportConfigUpdateAction.java	100.00% <ø> (ø)
.../security/action/whoami/TransportWhoAmIAction.java	33.33% <0.00%> (ø)
... and 107 more

... and 3 files with indirect coverage changes

[cwperks]

@RyanL1997 I am seeing some differences after I did a merge from main -> feature/extensions and resolved conflicts. The file count on this https://github.com/opensearch-project/security/compare/feature/extensions...cwperks:format-feature-extensions-apply?expand=1 is different from this PR.

I checked out your branch and did a merge of main -> this branch and had to resolve the same conflicts from merging main into the branch in the link above.

[RyanL1997]

Are we still pushing changes to this branch?

Hi @DarshitChanpura. Yes this PR needs the formatting changes + library fix to run the test I think

[peternied]

Lets please hold off on these changes , I've still got outstanding commits for this branch that I'd like to land before we pull in all the style changes.

[cwperks]

@peternied When can this change be merged into the feature branch? Its pretty stale at this point and not only needs the formatting changes, but other build breaking fixes as well.

[peternied]

I'm no longer impacted by this changes getting merged. Please use rebase and merge instead of squash and merge via the GitHub UX so it makes it easier to manage the changes.

No longer impacting other changes

[RyanL1997]

Closing this one due to the merge of: #2854