Skip to content

Provide more information when Key.verify_signature() fails #1875

Closed

Description

@MVrachev

Current behavior:
If we call Metadata.verify_delegate() and one of the keys fails to be verified the message they got is this:

logger.info("Key %s failed to verify %s", keyid, delegated_role)

No logging is provided about why the key has failed verification.

Expected behavior:
Adding a log about the last exception message could be useful to provide context for what exactly happened.
This could be useful to identify if a keyvalue is malformed.

Activity

  1. self-assigned this
    on Feb 21, 2022
  2. added a commit that references this issue on Feb 21, 2022
    fc1558b
  3. added this to the sprint 18 milestone on Feb 23, 2022
  4. linked a pull request that will close this issueLogging and error message improvements #1876on Mar 1, 2022
  5. modified the milestones: sprint 18, sprint 19 on Mar 9, 2022
  6. added
    backlogIssues to address with priority for current development goals
    on Mar 9, 2022
  7. added a commit that references this issue on May 10, 2022
    7de325d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

backlogIssues to address with priority for current development goals

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions