Skip to content
This repository has been archived by the owner on Mar 8, 2024. It is now read-only.

Add missing mandatory values to error message #245

Merged
merged 1 commit into from
Aug 31, 2023
Merged

Add missing mandatory values to error message #245

merged 1 commit into from
Aug 31, 2023

Conversation

mbrehler
Copy link

Makes it easier to update the ASN.1 message.

Change-Id: If2362729cef3fa52b67244d983377c78c76c317c

Makes it easier to update the ASN.1 message.

Change-Id: If2362729cef3fa52b67244d983377c78c76c317c
Copy link
Contributor

@p1-bmu p1-bmu left a comment

Choose a reason for hiding this comment

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

Nice change. using set() does not change a lot the performance of the test, compared to using a list comprehension. Thanks !

@p1-bmu p1-bmu merged commit eacb54d into P1sec:master Aug 31, 2023
16 checks passed
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.

2 participants