Skip to content

Update certs for cases where support for DES cipher is not available #93096

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

rado17
Copy link
Contributor

@rado17 rado17 commented Jul 14, 2025

No description provided.

@@ -0,0 +1,30 @@
-----BEGIN ENCRYPTED PRIVATE KEY-----

Check failure

Code scanning / SonarCloud

Cryptographic private keys should not be disclosed High test

Make sure this private key gets revoked, changed, and removed from the code. See more on SonarQube Cloud
@@ -0,0 +1,30 @@
-----BEGIN ENCRYPTED PRIVATE KEY-----

Check failure

Code scanning / SonarCloud

Cryptographic private keys should not be disclosed High test

Make sure this private key gets revoked, changed, and removed from the code. See more on SonarQube Cloud
@@ -0,0 +1,30 @@
-----BEGIN ENCRYPTED PRIVATE KEY-----

Check failure

Code scanning / SonarCloud

Cryptographic private keys should not be disclosed High test

Make sure this private key gets revoked, changed, and removed from the code. See more on SonarQube Cloud
krish2718
krish2718 previously approved these changes Jul 14, 2025
boards:
/.*/nrf.*/cpuapp/:
append:
EXTRA_CONF_FILE: wifi-enterprise_nrf.conf
Copy link
Contributor

Choose a reason for hiding this comment

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

overlay-nrf-wifi.conf?

rado17 added 3 commits July 16, 2025 15:45
With NRF security, higher heap requirement is seen for operating
with RSA-3072 based certificates. Add a NRF SoC specific conf in
snippets to handle this.

Signed-off-by: Ravi Dondaputi <ravi.dondaputi@nordicsemi.no>
Add AES-only cipher certificates. Useful for systems which
have only AES support and DES is not enabled.

Signed-off-by: Ravi Dondaputi <ravi.dondaputi@nordicsemi.no>
Add config options required for storing certificates in protected
storage, and to free up space of ROM to accommodate the additional
features that are enabled for TFM.

Signed-off-by: Ravi Dondaputi <ravi.dondaputi@nordicsemi.no>
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
E Security Rating on New Code (required ≥ C)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@@ -0,0 +1,8 @@
# For TLS and X.509 processing MbedTLS needs large heap size and using separate heap
# for MbedTLS gives us more control over the heap size.
CONFIG_MBEDTLS_HEAP_SIZE=75000
Copy link
Contributor

Choose a reason for hiding this comment

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

CONFIG_MBEDTLS_ENABLE_HEAP isn't needed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants