Ensure DataProtection can be used in trimmed apps.

[eerhardt]

• Add a DynamicDependency to ensure Aes decryption works in EncryptedXmlDecryptor
• Suppress the warnings from EncryptedXml
• Add trimming tests to ensure these scenarios work correctly.
• Remove the RequiresUnreferencedCode attribute on AddAuthentication, since this is the only thing in that method that has warnings.

Fix #47695

[mitchdenny]

More of a question than a blocking issue. Seems we are copying a type definition that already exists in code analysis?

[wtgodbe]

Build bits seem reasonable

[eerhardt]

Anyone have any feedback about the DataProtection pieces?

[amcasey]

I have limited knowledge of this code, but the change seems low-risk to me. Nits below.

[amcasey]

LGTM. Thanks for the offline explanation!

[JamesNK]

Nice!

Does this mean we can move authentication/authorization to the AOT supported column?

[eerhardt]

Does this mean we can move authentication/authorization to the AOT supported column?

Some things still don't work. For example, the JWT bearer handler depends on System.IdentityModel.Tokens.Jwt, which is not AOT compatible.

For the other authentication modes, are there other 3rd party dependencies? Or are they all contained in this repo?

[ghost]

Hi @eerhardt. It looks like you just commented on a closed PR. The team will most probably miss it. If you'd like to bring something important up to their attention, consider filing a new issue and add enough details to build context.