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

API documentation debt - System.Security #43861

Closed
Tracked by #43849
carlossanlop opened this issue Oct 26, 2020 · 5 comments
Closed
Tracked by #43849

API documentation debt - System.Security #43861

carlossanlop opened this issue Oct 26, 2020 · 5 comments
Assignees
Labels
area-System.Security documentation Documentation bug or enhancement, does not impact product or test code help wanted [up-for-grabs] Good issue for external contributors
Milestone

Comments

@carlossanlop
Copy link
Member

Area owners: @bartonjs @GrabYourPitchforks @krwq

The APIs in the list below are missing some or all of their documentation. Please add the missing documentation directly in triple slash comments in source. We will make sure it gets ported to dotnet-api-docs after it's merged.

Make sure to follow the documentation guidelines defined in the dotnet-api-docs wiki:
https://github.com/dotnet/dotnet-api-docs/wiki

Also please add me as a PR reviewer.

Community contributions are welcome.

System.Security
DocId Summary Parameters TypeParameters ReturnValue
P:System.Security.Cryptography.Rfc2898DeriveBytes.HashAlgorithm Missing NA NA NA
M:System.Security.Cryptography.RSA.TrySignHash
(System.ReadOnlySpan{System.Byte},System.Span{System.Byte},System.Security.Cryptography.HashAlgorithmName,
System.Security.Cryptography.RSASignaturePadding,System.Int32@)
Present Missing NA Present
P:System.Security.Cryptography.TripleDESCng.LegalKeySizes Missing NA NA NA
M:System.Security.Cryptography.Pkcs.Rfc3161TimestampToken.VerifySignatureForHash
(System.ReadOnlySpan{System.Byte},System.Security.Cryptography.Oid,
System.Security.Cryptography.X509Certificates.X509Certificate2@,
System.Security.Cryptography.X509Certificates.X509Certificate2Collection)
Present Missing NA Present
M:System.Security.Cryptography.Pkcs.Rfc3161TimestampTokenInfo.#ctor
(System.Security.Cryptography.Oid,System.Security.Cryptography.Oid,System.ReadOnlyMemory{System.Byte},
System.ReadOnlyMemory{System.Byte},System.DateTimeOffset,System.Nullable{System.Int64},System.Boolean,
System.Nullable{System.ReadOnlyMemory{System.Byte}},System.Nullable{System.ReadOnlyMemory{System.Byte}},
System.Security.Cryptography.X509Certificates.X509ExtensionCollection)
Missing Missing NA NA
P:System.Security.Cryptography.Pkcs.Rfc3161TimestampTokenInfo.AccuracyInMicroseconds Missing NA NA NA
M:System.Security.Cryptography.Pkcs.Rfc3161TimestampTokenInfo.Encode Missing NA NA Missing
M:System.Security.Cryptography.Pkcs.Rfc3161TimestampTokenInfo.GetExtensions Missing NA NA Missing
M:System.Security.Cryptography.Pkcs.Rfc3161TimestampTokenInfo.GetMessageHash Missing NA NA Missing
M:System.Security.Cryptography.Pkcs.Rfc3161TimestampTokenInfo.GetNonce Missing NA NA Missing
M:System.Security.Cryptography.Pkcs.Rfc3161TimestampTokenInfo.GetSerialNumber Missing NA NA Missing
M:System.Security.Cryptography.Pkcs.Rfc3161TimestampTokenInfo.GetTimestampAuthorityName Missing NA NA Missing
P:System.Security.Cryptography.Pkcs.Rfc3161TimestampTokenInfo.HasExtensions Missing NA NA NA
P:System.Security.Cryptography.Pkcs.Rfc3161TimestampTokenInfo.HashAlgorithmId Missing NA NA NA
P:System.Security.Cryptography.Pkcs.Rfc3161TimestampTokenInfo.IsOrdering Missing NA NA NA
P:System.Security.Cryptography.Pkcs.Rfc3161TimestampTokenInfo.PolicyId Missing NA NA NA
T:System.Security.Cryptography.Pkcs.Rfc3161TimestampTokenInfo Missing NA NA NA
P:System.Security.Cryptography.Pkcs.Rfc3161TimestampTokenInfo.Timestamp Missing NA NA NA
M:System.Security.Cryptography.Pkcs.Rfc3161TimestampTokenInfo.TryDecode
(System.ReadOnlyMemory{System.Byte},System.Security.Cryptography.Pkcs.Rfc3161TimestampTokenInfo@,System.Int32@)
Missing Missing NA Missing
M:System.Security.Cryptography.Pkcs.Rfc3161TimestampTokenInfo.TryEncode(System.Span{System.Byte},System.Int32@) Missing Missing NA Missing
P:System.Security.Cryptography.Pkcs.Rfc3161TimestampTokenInfo.Version Missing NA NA NA
M:System.Security.Permissions.KeyContainerPermissionAccessEntryCollection.#ctor Missing NA NA NA
M:System.Security.Permissions.KeyContainerPermissionAccessEntryEnumerator.#ctor Missing NA NA NA
@carlossanlop carlossanlop added documentation Documentation bug or enhancement, does not impact product or test code area-System.Security help wanted [up-for-grabs] Good issue for external contributors labels Oct 26, 2020
@carlossanlop carlossanlop added this to the 6.0.0 milestone Oct 26, 2020
@ghost
Copy link

ghost commented Oct 26, 2020

Tagging subscribers to this area: @bartonjs, @vcsjones, @krwq, @jeffhandley
See info in area-owners.md if you want to be subscribed.

@Dotnet-GitSync-Bot Dotnet-GitSync-Bot added the untriaged New issue has not been triaged by the area owner label Oct 26, 2020
@bartonjs bartonjs removed the untriaged New issue has not been triaged by the area owner label Oct 26, 2020
@vcsjones
Copy link
Member

I guess just to ask, some of these look like they already have documentation on the docs site.

System.Security.Cryptography.RSA.TrySignHash
(System.ReadOnlySpan{System.Byte},System.Span{System.Byte},System.Security.Cryptography.HashAlgorithmName,
System.Security.Cryptography.RSASignaturePadding,System.Int32@)

https://docs.microsoft.com/en-us/dotnet/api/system.security.cryptography.rsa.trysignhash?view=netcore-3.1

Looks like that one and a few others are documented correctly, including the parameters.

Assuming the API is properly documented, I assume we don't want to document them again with XML comments?

@carlossanlop
Copy link
Member Author

@vcsjones that API hasn't been fully documented. That API is missing a description in the "destination" parameter: It has the default boilerplate text "To be added", which needs to be changed.

@vcsjones
Copy link
Member

"destination" parameter: It has the default boilerplate text "To be added", which needs to be changed.

Doh! I glossed over that one. Thanks.

@krwq
Copy link
Member

krwq commented Nov 24, 2020

I believe this is complete now since both PRs are merged.

@krwq krwq closed this as completed Nov 24, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Dec 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-System.Security documentation Documentation bug or enhancement, does not impact product or test code help wanted [up-for-grabs] Good issue for external contributors
Projects
None yet
Development

No branches or pull requests

5 participants