Skip to content

Fixes for version compatibility#40

Merged
leongross merged 8 commits intobuildupfrom
mgudel/fix/versions
Mar 12, 2026
Merged

Fixes for version compatibility#40
leongross merged 8 commits intobuildupfrom
mgudel/fix/versions

Conversation

@embediver
Copy link
Copy Markdown

  • Fixes for compatibility with SPDM versions < v1.3
  • Minor cleanup

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

license-eye has checked 72 files.

Valid Invalid Ignored Fixed
0 57 15 0
Click to see the invalid file list
  • examples/platform/cert_store.rs
  • examples/platform/certs.rs
  • examples/platform/crypto.rs
  • examples/platform/evidence.rs
  • examples/platform/mod.rs
  • examples/platform/socket_transport.rs
  • examples/spdm_requester.rs
  • examples/spdm_responder.rs
  • src/cert_store.rs
  • src/chunk_ctx.rs
  • src/codec.rs
  • src/commands/algorithms/mod.rs
  • src/commands/algorithms/request.rs
  • src/commands/algorithms/response.rs
  • src/commands/capabilities/mod.rs
  • src/commands/capabilities/request.rs
  • src/commands/capabilities/response.rs
  • src/commands/certificate/mod.rs
  • src/commands/certificate/request.rs
  • src/commands/certificate/response.rs
  • src/commands/challenge/mod.rs
  • src/commands/challenge/request.rs
  • src/commands/challenge/response.rs
  • src/commands/chunk_get_rsp.rs
  • src/commands/digests/mod.rs
  • src/commands/digests/request.rs
  • src/commands/digests/response.rs
  • src/commands/error_rsp.rs
  • src/commands/measurements_rsp.rs
  • src/commands/mod.rs
  • src/commands/version/mod.rs
  • src/commands/version/request.rs
  • src/commands/version/response.rs
  • src/context.rs
  • src/error.rs
  • src/lib.rs
  • src/measurements/common.rs
  • src/measurements/freeform_manifest.rs
  • src/measurements/mod.rs
  • src/platform/evidence.rs
  • src/platform/hash.rs
  • src/platform/mod.rs
  • src/platform/rng.rs
  • src/platform/transport.rs
  • src/protocol/algorithms.rs
  • src/protocol/capabilities.rs
  • src/protocol/certs.rs
  • src/protocol/common.rs
  • src/protocol/mod.rs
  • src/protocol/signature.rs
  • src/protocol/version.rs
  • src/requester.rs
  • src/state.rs
  • src/test.rs
  • src/transcript.rs
  • src/transport.rs
  • tests/spdm_validator_host.rs
Use this command to fix any missing license headers
```bash

docker run -it --rm -v $(pwd):/github/workspace apache/skywalking-eyes header fix

</details>

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

license-eye has checked 72 files.

Valid Invalid Ignored Fixed
0 57 15 0
Click to see the invalid file list
  • examples/platform/cert_store.rs
  • examples/platform/certs.rs
  • examples/platform/crypto.rs
  • examples/platform/evidence.rs
  • examples/platform/mod.rs
  • examples/platform/socket_transport.rs
  • examples/spdm_requester.rs
  • examples/spdm_responder.rs
  • src/cert_store.rs
  • src/chunk_ctx.rs
  • src/codec.rs
  • src/commands/algorithms/mod.rs
  • src/commands/algorithms/request.rs
  • src/commands/algorithms/response.rs
  • src/commands/capabilities/mod.rs
  • src/commands/capabilities/request.rs
  • src/commands/capabilities/response.rs
  • src/commands/certificate/mod.rs
  • src/commands/certificate/request.rs
  • src/commands/certificate/response.rs
  • src/commands/challenge/mod.rs
  • src/commands/challenge/request.rs
  • src/commands/challenge/response.rs
  • src/commands/chunk_get_rsp.rs
  • src/commands/digests/mod.rs
  • src/commands/digests/request.rs
  • src/commands/digests/response.rs
  • src/commands/error_rsp.rs
  • src/commands/measurements_rsp.rs
  • src/commands/mod.rs
  • src/commands/version/mod.rs
  • src/commands/version/request.rs
  • src/commands/version/response.rs
  • src/context.rs
  • src/error.rs
  • src/lib.rs
  • src/measurements/common.rs
  • src/measurements/freeform_manifest.rs
  • src/measurements/mod.rs
  • src/platform/evidence.rs
  • src/platform/hash.rs
  • src/platform/mod.rs
  • src/platform/rng.rs
  • src/platform/transport.rs
  • src/protocol/algorithms.rs
  • src/protocol/capabilities.rs
  • src/protocol/certs.rs
  • src/protocol/common.rs
  • src/protocol/mod.rs
  • src/protocol/signature.rs
  • src/protocol/version.rs
  • src/requester.rs
  • src/state.rs
  • src/test.rs
  • src/transcript.rs
  • src/transport.rs
  • tests/spdm_validator_host.rs
Use this command to fix any missing license headers
```bash

docker run -it --rm -v $(pwd):/github/workspace apache/skywalking-eyes header fix

</details>

@embediver
Copy link
Copy Markdown
Author

Verification for v1.1 still fails.
Somehow the signature verification for the CHALLENGE_AUTH response fails.
M1/M2 hash seems to be the same on both requester and responder side though.
Versions <= v1.1 don't prepend the special context string into the signed data, but our example already handles that correctly...

Maybe you @leongross can take a quick look?

Signed-off-by: leongross <leon.gross@9elements.com>
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

license-eye has checked 72 files.

Valid Invalid Ignored Fixed
0 57 15 0
Click to see the invalid file list
  • examples/platform/cert_store.rs
  • examples/platform/certs.rs
  • examples/platform/crypto.rs
  • examples/platform/evidence.rs
  • examples/platform/mod.rs
  • examples/platform/socket_transport.rs
  • examples/spdm_requester.rs
  • examples/spdm_responder.rs
  • src/cert_store.rs
  • src/chunk_ctx.rs
  • src/codec.rs
  • src/commands/algorithms/mod.rs
  • src/commands/algorithms/request.rs
  • src/commands/algorithms/response.rs
  • src/commands/capabilities/mod.rs
  • src/commands/capabilities/request.rs
  • src/commands/capabilities/response.rs
  • src/commands/certificate/mod.rs
  • src/commands/certificate/request.rs
  • src/commands/certificate/response.rs
  • src/commands/challenge/mod.rs
  • src/commands/challenge/request.rs
  • src/commands/challenge/response.rs
  • src/commands/chunk_get_rsp.rs
  • src/commands/digests/mod.rs
  • src/commands/digests/request.rs
  • src/commands/digests/response.rs
  • src/commands/error_rsp.rs
  • src/commands/measurements_rsp.rs
  • src/commands/mod.rs
  • src/commands/version/mod.rs
  • src/commands/version/request.rs
  • src/commands/version/response.rs
  • src/context.rs
  • src/error.rs
  • src/lib.rs
  • src/measurements/common.rs
  • src/measurements/freeform_manifest.rs
  • src/measurements/mod.rs
  • src/platform/evidence.rs
  • src/platform/hash.rs
  • src/platform/mod.rs
  • src/platform/rng.rs
  • src/platform/transport.rs
  • src/protocol/algorithms.rs
  • src/protocol/capabilities.rs
  • src/protocol/certs.rs
  • src/protocol/common.rs
  • src/protocol/mod.rs
  • src/protocol/signature.rs
  • src/protocol/version.rs
  • src/requester.rs
  • src/state.rs
  • src/test.rs
  • src/transcript.rs
  • src/transport.rs
  • tests/spdm_validator_host.rs
Use this command to fix any missing license headers
```bash

docker run -it --rm -v $(pwd):/github/workspace apache/skywalking-eyes header fix

</details>

@leongross leongross marked this pull request as ready for review March 12, 2026 14:37
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

license-eye has checked 72 files.

Valid Invalid Ignored Fixed
0 57 15 0
Click to see the invalid file list
  • examples/platform/cert_store.rs
  • examples/platform/certs.rs
  • examples/platform/crypto.rs
  • examples/platform/evidence.rs
  • examples/platform/mod.rs
  • examples/platform/socket_transport.rs
  • examples/spdm_requester.rs
  • examples/spdm_responder.rs
  • src/cert_store.rs
  • src/chunk_ctx.rs
  • src/codec.rs
  • src/commands/algorithms/mod.rs
  • src/commands/algorithms/request.rs
  • src/commands/algorithms/response.rs
  • src/commands/capabilities/mod.rs
  • src/commands/capabilities/request.rs
  • src/commands/capabilities/response.rs
  • src/commands/certificate/mod.rs
  • src/commands/certificate/request.rs
  • src/commands/certificate/response.rs
  • src/commands/challenge/mod.rs
  • src/commands/challenge/request.rs
  • src/commands/challenge/response.rs
  • src/commands/chunk_get_rsp.rs
  • src/commands/digests/mod.rs
  • src/commands/digests/request.rs
  • src/commands/digests/response.rs
  • src/commands/error_rsp.rs
  • src/commands/measurements_rsp.rs
  • src/commands/mod.rs
  • src/commands/version/mod.rs
  • src/commands/version/request.rs
  • src/commands/version/response.rs
  • src/context.rs
  • src/error.rs
  • src/lib.rs
  • src/measurements/common.rs
  • src/measurements/freeform_manifest.rs
  • src/measurements/mod.rs
  • src/platform/evidence.rs
  • src/platform/hash.rs
  • src/platform/mod.rs
  • src/platform/rng.rs
  • src/platform/transport.rs
  • src/protocol/algorithms.rs
  • src/protocol/capabilities.rs
  • src/protocol/certs.rs
  • src/protocol/common.rs
  • src/protocol/mod.rs
  • src/protocol/signature.rs
  • src/protocol/version.rs
  • src/requester.rs
  • src/state.rs
  • src/test.rs
  • src/transcript.rs
  • src/transport.rs
  • tests/spdm_validator_host.rs
Use this command to fix any missing license headers
```bash

docker run -it --rm -v $(pwd):/github/workspace apache/skywalking-eyes header fix

</details>

@leongross leongross merged commit 58fb82b into buildup Mar 12, 2026
8 of 11 checks passed
@leongross leongross deleted the mgudel/fix/versions branch March 12, 2026 17:33
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