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

intel-trust-authority-as: add error message log #424

Merged

Conversation

pawelpros
Copy link
Contributor

@pawelpros pawelpros commented Jun 20, 2024

  • Added logging error message for failed appraisal request.
  • Fixed typos

Sample output:

ERROR api_server::http::error] Attestation failed: Attestation request failed: response status=400 Bad Request, message=Invalid nonce and/or run time data provided in the request

@pawelpros pawelpros force-pushed the ita_err_msg_handling branch from 316d670 to a94ea97 Compare June 20, 2024 11:46
- Added logging error message for failed appraisal request.
- Fixed typos

Sample log:

`ERROR api_server::http::error] Attestation failed: Attestation request failed: response status=400 Bad Request, message=Invalid nonce and/or run time data provided in the request`

Signed-off-by: Pawel Proskurnicki <[email protected]>
@pawelpros pawelpros force-pushed the ita_err_msg_handling branch from a94ea97 to 4cb2048 Compare June 21, 2024 05:41
Copy link
Member

@fidencio fidencio left a comment

Choose a reason for hiding this comment

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

lgtm, thanks @pawelpros!

@Xynnn007 Xynnn007 merged commit 215b3f7 into confidential-containers:main Jun 26, 2024
15 checks passed
@pawelpros pawelpros deleted the ita_err_msg_handling branch July 8, 2024 13:14
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.

4 participants