launchdarkly / cpp-sdks

C++ Client/Server SDKs
Other
5 stars 2 forks source link

docs: adding SLSA verification steps for cpp-server-sdk #356

Closed rsoberano-ld closed 7 months ago

rsoberano-ld commented 8 months ago

Drafting up documentation for how consumers may use the SLSA framework to verify SDK packages published with provenance to improve supply chain security.

rsoberano-ld commented 8 months ago

Starting off with a single SDK for now. If this looks good I'll add this to the other SDK READMEs in this monorepo. Let me know if what we've got so far looks good, namely:

rsoberano-ld commented 8 months ago

Appreciate all the feedback here @cwaldren-ld! Definitely valuable having the developer's perspective here, will make revisions and resubmit for review.

rsoberano-ld commented 8 months ago

I'm wondering if this should be in a dedicated file - say, existing SECURITY.md or PROVENANCE.md or something.

Mainly because it leans towards being a reference, rather than a quickstart like "run tool X on file Y", and our READMEs are already pretty dense.

I think this is a really fair point, we wouldn't want to detract from the main usage of the README with this large section (unless we're able to condense it down significantly). @mmrj, any suggestions on what might be an appropriate location for these instructions?

mmrj commented 8 months ago

I'm wondering if this should be in a dedicated file - say, existing SECURITY.md or PROVENANCE.md or something. Mainly because it leans towards being a reference, rather than a quickstart like "run tool X on file Y", and our READMEs are already pretty dense.

I think this is a really fair point, we wouldn't want to detract from the main usage of the README with this large section (unless we're able to condense it down significantly). @mmrj, any suggestions on what might be an appropriate location for these instructions?

The repo has an existing SECURITY.md, so maybe there? You could re-title that file to just "Security" and put the existing content in one header, and have the "Validating SDK packages" as a second header.