Skip to content

Update n_ca_digital_signature_not_set citation, notice, and doc comment#998

Merged
christopher-henderson merged 1 commit intozmap:masterfrom
mhyder13:ca-ku-digitalSig
Oct 12, 2025
Merged

Update n_ca_digital_signature_not_set citation, notice, and doc comment#998
christopher-henderson merged 1 commit intozmap:masterfrom
mhyder13:ca-ku-digitalSig

Conversation

@mhyder13
Copy link
Copy Markdown
Contributor

Notes

This is similar to #997, but I broke this one out both because it's not a firm requirement, and because the way the supporting language is written is different. (It's in a weird footnote.) As with the error lints, this one was years out of date and I've added essentially the same version history to it.

Summery

  • Updated the in-program citation strings for n_ca_digital_signature_not_set
  • Added a full history of the locations of this warning to the lint comments
  • Updated the requirement language in the comments to match the current language
  • Added a details string to the result when the notice is raised
    • Because this is a Notice lint warning about a requirement that it's not actually testing, I figured it would be helpful to add something to the result about what it actually means. How useful this actually is probably depends on your integration pattern.

Doc References

@christopher-henderson christopher-henderson merged commit 7dfef30 into zmap:master Oct 12, 2025
4 checks passed
@mhyder13 mhyder13 deleted the ca-ku-digitalSig branch October 12, 2025 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants