Bump HtmlSanitizer

[bdukes]

Summary

In order to avoid NU1902 error, this PR bumps HtmlSanitizer (which requires bumping System.Collections.Immutable).

Locally, this caused issues with the source generator, pushing this to CI to see if it's a local only issue.

[bdukes]

hmmm… to upgrade that, it bumps System.Collections.Immutable to 10.0.0 (from 9.0.2), and that bump causes errors with the source generator for some reason.

error : Generator 'DnnDeprecatedGenerator' failed to initialize. It will not contribute to the output and compilation errors may occur as a result. Exception was of type 'FileNotFoundException' with message 'Could not load file or assembly 'System.Collections.Immutable, Version=10.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified.'.

[mitchelsellers]

This looks good with one question.

Does this upgrade result in a need for a binding redirect?

[bdukes]

It will require an update to the binding redirect. It's already included in the HtmlSanitizer package, but that assembly is included in the bin folder of the upgrade package, so it'll cause an error using the traditional upgrade process.

[donker]

It will require an update to the binding redirect. It's already included in the HtmlSanitizer package, but that assembly is included in the bin folder of the upgrade package, so it'll cause an error using the traditional upgrade process.

So does this mean we already move forward to the new upgrade process? Meaning: not release the upgrade package and point users to the upgrade module instead?

[valadas]

@bdukes my understanding is that this works fine with the new upgrade process and we have adjusted the documentation for that right?

[bdukes]

10.2.3 resolves an issue introduced in 10.2.2 which required using the upgrade process to avoid binding redirect issues. I'm not sure where we landed on requiring the upgrade process for 10.3.0.