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.

Sign up for GitHub

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

Jump to bottom

[FL-3950] Update mbedtls & expose AES #4092

Merged
merged 4 commits into from
Feb 10, 2025
Merged

[FL-3950] Update mbedtls & expose AES #4092

merged 4 commits into from
Feb 10, 2025

Conversation

portasynthinca3
Copy link
Member

@portasynthinca3 portasynthinca3 commented Jan 31, 2025 •
edited by skotopes
Loading

What's new

Verification

N/A

Checklist (For Reviewer)

  • PR has description of feature/bug or link to Confluence/Jira task
  • Description contains actions to verify feature/bugfix
  • I've built this code, uploaded it to the device and verified feature/bugfix

@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 31, 2025 •
edited
Loading

Compiled f7 firmware for commit 5eaffc59:

@portasynthinca3 portasynthinca3 marked this pull request as ready for review January 31, 2025 11:41
@hedger hedger self-requested a review February 3, 2025 18:17
@hedger hedger added New Feature Contains an IMPLEMENTATION of a new feature App Loader FAP loader + API-related labels Feb 3, 2025
@hedger
Copy link
Member

hedger commented Feb 3, 2025

@bettse could you please try the SDK from this branch and see if it successfully links with your app?

@bettse
Copy link
Contributor

bettse commented Feb 3, 2025

@hedger will do!

@bettse
Copy link
Contributor

bettse commented Feb 3, 2025

Successfully compiles when I include the header and reference the aes methods 👍

@skotopes skotopes merged commit 7a92fd3 into dev Feb 10, 2025
11 checks passed
@skotopes skotopes deleted the portasynthinca3/3950-update-mbedtls branch February 10, 2025 11:57
@portasynthinca3 portasynthinca3 mentioned this pull request Feb 13, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@skotopes skotopes skotopes approved these changes

@DrZlo13 DrZlo13 Awaiting requested review from DrZlo13 DrZlo13 is a code owner

@gsurkov gsurkov Awaiting requested review from gsurkov gsurkov is a code owner

@nminaylov nminaylov Awaiting requested review from nminaylov nminaylov is a code owner

@hedger hedger Awaiting requested review from hedger hedger is a code owner

Assignees
No one assigned
Labels
App Loader FAP loader + API-related New Feature Contains an IMPLEMENTATION of a new feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@portasynthinca3 @hedger @bettse @skotopes