Skip to content

[Compliance] CodeQL alerts due to missing justifications #191

@GrabYourPitchforks

Description

@GrabYourPitchforks

We're seeing three CodeQL policy violations for these lines:

using (var md5 = MD5.Create()) // lgtm [cs/weak-crypto] Not used for crypto

using (var md5 = MD5.Create()) // lgtm [cs/weak-crypto] Not used for crypto

using (var md5 = MD5.Create()) // lgtm [cs/weak-crypto] Not used for crypto

The justification string is insufficient to meet policy requirements. The suppression string must at minimum contain:

  • Why it's not possible to use a different algorithm; and
  • One of:
    • Why we don't think a collision is likely to occur; or
    • Why the consequences of a collision are uninteresting.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions