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

chore(connect): update device authenticity config #15407

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

trezor-ci
Copy link
Contributor

@trezor-ci trezor-ci commented Nov 15, 2024

@@ -119,11 +122,11 @@ export const deviceAuthenticityConfig: DeviceAuthenticityConfig = {
},
},
T3W1: {
rootPubKeys: ['you shall not pass'], // TODO T3W1
caPubKeys: ['you shall not pass'], // TODO T3W1
rootPubKeys: ['you shall not pass'],
Copy link
Member

Choose a reason for hiding this comment

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

@MiroslavProchazka it looks like you added here those TODOs but this file is updated semi-automatically so they get deleted. Shouldn't we create a issue or something to update the script that update this once it is necessary?

Copy link
Member

@karliatto karliatto left a comment

Choose a reason for hiding this comment

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

This looks good to me.

@trezor-ci
Copy link
Contributor Author

Closing this PR since we are going to create new one with latest changes in https://data.trezor.io

@trezor-ci trezor-ci closed this Nov 16, 2024
@karliatto karliatto reopened this Nov 17, 2024
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.

2 participants