8000 boards: tut: dynamic-apps: Add credential checking by bradjc · Pull Request #4459 · tock/tock · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

boards: tut: dynamic-apps: Add credential checking #4459

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

Merged
merged 3 commits into from
Jun 11, 2025

Conversation

bradjc
Copy link
Contributor
@bradjc bradjc commented Jun 5, 2025

Pull Request Overview

Now that we have multiple credential key checking support, this adds credential checking to the tutorial board.

Because we want to support apps that are not signed as well, this adds a small wrapper that tells the credential checking machine that its ok to load apps without a valid credential.

Testing Strategy

Running the libtock-c apps for the tutorial.

TODO or Help Wanted

n/a

Documentation Updated

  • Updated the relevant files in /docs, or no updates are required.

Formatting

  • Ran make prepush.

Now that we have multiple credential key checking support, this adds
credential checking to the tutorial board.

Because we want to support apps that are not signed as well, this adds a
small wrapper that tells the credential checking machine that its ok to
load apps without a valid credential.
alevy
alevy previously approved these changes Jun 11, 2025
@alevy alevy enabled auto-merge June 11, 2025 17:23
@alevy alevy added this pull request to the merge queue Jun 11, 2025
auto-merge was automatically disabled June 11, 2025 17:37

Pull Request is not mergeable

Merged via the queue into master with commit 2e0b3a1 Jun 11, 2025
17 of 21 checks passed
@alevy alevy deleted the tut-dynamic-credential-checking branch June 11, 2025 17:46
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.

3 participants
0