Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add send/receive to spdm_common_lib #2598

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

PrithviAPai
Copy link

Once secure session is established, in application phase both SPDM Requester and SPDM Responder can send/receive messages. Added patch which will enable SPDM Responders also to send message during Appication phase.

Resolves issue: #2571

@jyao1
Copy link
Member

jyao1 commented Feb 29, 2024

@PrithviAPai , please squash your commit.
There is no need to create a patch to fix format issue.

@PrithviAPai PrithviAPai force-pushed the move_send_receive_spdm_common_lib branch from 0ec70df to 3b76148 Compare February 29, 2024 04:48
Once secure session is established, in application phase both SPDM Requester and SPDM Responder can send/receive messages.
Added patch which will enable SPDM Responders also to send message during Appication phase.

Resolves issue: DMTF#2571

Signed-off-by: Prithvi A Pai <prithvi.a.pai@intel.com>
@PrithviAPai PrithviAPai force-pushed the move_send_receive_spdm_common_lib branch from 3b76148 to c9365ea Compare February 29, 2024 05:49
@PrithviAPai
Copy link
Author

@PrithviAPai , please squash your commit. There is no need to create a patch to fix format issue.

Addressed the comment.
Squashed the commits. Thank You

@Wenxing-hou
Copy link
Member

Thanks for your commit. And I compare the change: no code changes, only positional changes. LGTM.
Please submit commit too in spdm-emu if the PR is merged.

@jyao1
Copy link
Member

jyao1 commented Feb 29, 2024

Would you please show an example on how responder uses those APIs?

@PrithviAPai
Copy link
Author

Would you please show an example on how responder uses those APIs?

SPDM Responder(Integrator) daemon can expose APIs for applications to send encrypted data to SPDM EndPoint.
The payload received by the applications can be sent to EID using libspdm_send_data

@PrithviAPai PrithviAPai marked this pull request as draft March 1, 2024 05:45
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.

None yet

3 participants