Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[release/9.0] Ensure a null PublicKey is supported on assembly refs #108928

Merged
merged 1 commit into from
Oct 18, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Oct 16, 2024

Backport of #108907 to release/9.0.

Issue: #108517

Customer Impact

  • Customer reported
  • Found internally

Certain DispatchProxy scenarios and consumers including WCF are broken in v9 when referencing assemblies that do not have a PublicKey. One potential workaround is to add a PublicKey to the referenced assembly, however that is not viable in all cases since the target assembly may external.

Regression

  • Yes
  • No

See the linked issue - when generating an assembly through DispatchProxy, an assembly reference that has a null PublicKey may instead be generated with a random PublicKey. This will later cause that referenced assembly to not be loaded properly.

Testing

The original issue was verified.

Risk

Low; the regression was made in v9 and the fix is straightforward.

@dotnet-issue-labeler dotnet-issue-labeler bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Oct 16, 2024
@steveharter steveharter added area-VM-coreclr and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Oct 16, 2024
@steveharter steveharter self-assigned this Oct 16, 2024
@steveharter steveharter added this to the 9.0.0 milestone Oct 16, 2024
Copy link
Contributor

Tagging subscribers to this area: @mangod9
See info in area-owners.md if you want to be subscribed.

@steveharter steveharter added the Servicing-consider Issue for next servicing release review label Oct 16, 2024
@jeffschwMSFT jeffschwMSFT added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Oct 16, 2024
@ericstj ericstj merged commit a9d9707 into release/9.0 Oct 18, 2024
90 of 101 checks passed
@steveharter steveharter deleted the backport/pr-108907-to-release/9.0 branch October 18, 2024 17:13
@github-actions github-actions bot locked and limited conversation to collaborators Nov 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants