Skip to content
This repository has been archived by the owner on Oct 10, 2022. It is now read-only.

Add SPDM 1.0 compatibility #75

Merged
merged 8 commits into from
Aug 10, 2021
Merged

Add SPDM 1.0 compatibility #75

merged 8 commits into from
Aug 10, 2021

Conversation

jklmnn
Copy link
Member

@jklmnn jklmnn commented Aug 3, 2021

  • Allow SPDM 1.0 and add additional messages and refinements where required
  • Fix inconsistencies with spec
  • Add tests

@jklmnn jklmnn requested a review from senier August 3, 2021 09:00
spdm.rflx Outdated Show resolved Hide resolved
spdm.rflx Outdated Show resolved Hide resolved
spdm.rflx Outdated Show resolved Hide resolved
spdm.rflx Show resolved Hide resolved
@jklmnn
Copy link
Member Author

jklmnn commented Aug 9, 2021

I additionally added some hacks in cfa1810 and fixed some field orders and sizes to support Negotiate_Algorithms and Algorithms.

@jklmnn jklmnn requested a review from senier August 9, 2021 12:57
Copy link
Member

@senier senier left a comment

Choose a reason for hiding this comment

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

@jklmnn A test case still fails in the CI.

@jklmnn
Copy link
Member Author

jklmnn commented Aug 10, 2021

@jklmnn A test case still fails in the CI.

Should be fixed in a3f63b9.

@jklmnn jklmnn merged commit e297588 into issue_55 Aug 10, 2021
@jklmnn jklmnn deleted the issue_55_spdm_1_0 branch August 10, 2021 12:54
jklmnn added a commit that referenced this pull request Aug 23, 2021
jklmnn added a commit that referenced this pull request Sep 8, 2021
jklmnn added a commit that referenced this pull request Oct 4, 2021
senier pushed a commit that referenced this pull request Oct 5, 2021
senier pushed a commit that referenced this pull request Oct 7, 2021
senier pushed a commit that referenced this pull request Oct 8, 2021
senier pushed a commit that referenced this pull request Oct 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants