Skip to content

Conversation

vcsjones
Copy link
Member

Summary

The RandomNumberGenerator class is currently documented as a base class which other random number generators should derive from, and that callers are expected to use a derived class.

We've de-emphasized using derived classes for RandomNumberGenerator, going so far as to obsolete the only in-box derived type, RNGCryptoServiceProvider.

These changes attempt to steer people toward using the new static APIs, instead of Create or derived types.

@vcsjones vcsjones requested a review from a team as a code owner April 29, 2022 22:03
@ghost ghost added the area-System.Security Issues related to security practices for .NET developers. label Apr 29, 2022
@ghost
Copy link

ghost commented Apr 29, 2022

Tagging subscribers to this area: @dotnet/area-system-security, @vcsjones
See info in area-owners.md if you want to be subscribed.

Issue Details

Summary

The RandomNumberGenerator class is currently documented as a base class which other random number generators should derive from, and that callers are expected to use a derived class.

We've de-emphasized using derived classes for RandomNumberGenerator, going so far as to obsolete the only in-box derived type, RNGCryptoServiceProvider.

These changes attempt to steer people toward using the new static APIs, instead of Create or derived types.

Author: vcsjones
Assignees: -
Labels:

area-System.Security

Milestone: -

@opbld30
Copy link

opbld30 commented Apr 29, 2022

Docs Build status updates of commit 98ef492:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Security.Cryptography/RandomNumberGenerator.xml ✅Succeeded View

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@gewarren gewarren merged commit 9063bf4 into main May 5, 2022
@gewarren gewarren deleted the fixup-rng branch May 5, 2022 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-System.Security Issues related to security practices for .NET developers.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants