-
Notifications
You must be signed in to change notification settings - Fork 201
build with mbedtls 3.x #198
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
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Glenn Strauss <[email protected]>
ce5b152 to
120c581
Compare
|
To follow this ticket. |
|
@Neustradamus if after this many years, you still have not learned how to click the "Subscribe" button to the right instead of useless posts, you are more annoying than helpful. Please avoid making useless posts. |
|
@gstrauss: Always same, I receive only e-mails if I comment the issues/PRs... I follow a lot of projects... |
|
@Neustradamus you are technologically inept. If you are logged into github, look at the section on the right. There will be a button which says "Subscribe" (if you have not subscribed to this issue/pull request) or "Unsubscribe" (if you have subscribed to this issue/pull request) Above the button it says "Notifications Customize". You can click that to further customize notifications. When you click the Subscribe button in a single issue or pull request, that triggers the same notifications for the issue/pull request that occur as when you comment. Stop commenting garbage just to enable notifications, you buffoon! Instead, click the Subscribe button to the issue/pull request to enable notifications for the specific issue/pull request. |
The basicstation build fails since the change to the new major version 3.x of mbedtls, because of API changes in the new mbedtls version. To fix the compilation for new mbedtls version, the waiting pullrequest is backported as a patch. Thanks to 'Glenn Strauss' to create this PR: lorabasics/basicstation#198 Signed-off-by: Florian Eckert <[email protected]>
The basicstation build fails since the change to the new major version 3.x of mbedtls, because of API changes in the new mbedtls version. To fix the compilation for new mbedtls version, the waiting pullrequest is backported as a patch. Thanks to 'Glenn Strauss' to create this PR: lorabasics/basicstation#198 Signed-off-by: Florian Eckert <[email protected]>
The basicstation build fails since the change to the new major version 3.x of mbedtls, because of API changes in the new mbedtls version. To fix the compilation for new mbedtls version, the waiting pullrequest is backported as a patch. Thanks to 'Glenn Strauss' to create this PR: lorabasics/basicstation#198 Signed-off-by: Florian Eckert <[email protected]>
The basicstation build fails since the change to the new major version 3.x of mbedtls, because of API changes in the new mbedtls version. To fix the compilation for new mbedtls version, the waiting pullrequest is backported as a patch. Thanks to 'Glenn Strauss' to create this PR: lorabasics/basicstation#198 Signed-off-by: Florian Eckert <[email protected]>
|
We are about to get started integrating lorawan into our product. I just finished integrating mbedtls (3.6) and now I see that lorawan depends on mbedtls 2.x. Then i came across this pull request. Any reason why this is not merged? I don't mind starting our integration on this branch, but if there are reasons why this won't get merged into master I was hoping to learn the reason. Many thanks in advance |
The basicstation build fails since the change to the new major version 3.x of mbedtls, because of API changes in the new mbedtls version. To fix the compilation for new mbedtls version, the waiting pullrequest is backported as a patch. Thanks to 'Glenn Strauss' to create this PR: lorabasics/basicstation#198 Signed-off-by: Florian Eckert <[email protected]>
After updating to 3.4.x parsing of certificates returned by the CUPS service of Actility weren't parsed correctly anymore. This fixes the parsing. Issue lorabasics#198
|
When testing 665141d with Actility, parsing of the certificates downloaded via I created a patch that fixes the parsing, but I'm not sure, if there would be a better way to fix it. |
Fix parsing of DER formatted keys while still supporting PEM. The problem arises when using `mbedtls_pk_parse_key()` the caller needs to know beforehand which format the key has to decide whether the `keylen` needs to account for the zero terminator. Since mbedtls knows best how to parse the respective formats, we switch from `mbedtls_pk_parse_key()` to `mbedtls_pk_parse_keyfile()` on latest mbedtls version. Issue lorabasics#198
|
While trimpim@089c817 fixed the handling of DER formatted keys, it introduced a regression on PEM keys. sidhussmann@383fc6e fixes the key handling for DER as well as PEM. |
build with mbedtls 3.x. This patch should also still work with mbedtls 2.28.x.
This patch simply adds
MBEDTLS_PRIVATE()to access now-private mbedtls 3.x struct members.Note: mbedtls 3.6.0 adds some helper functions which would allow
cups_verifySig()to avoid usingMBEDTLS_PRIVATE(), but that change is not included in this patch.