nrf_security: fix undefined Kconfig option references

[tomi-font]

See commits.

[NordicBuilder]

CI Information

^{To view the history of this post, clich the 'edited' button above}
Build number: 5

Inputs:

Sources:

sdk-nrf: PR head: abefe9159e519ec51377af0ebd04f587804d269d

more details

sdk-nrf:

PR head: abefe9159e519ec51377af0ebd04f587804d269d
merge base: bc493723e2be19cfc0447fc03be5bca55a59dc59
target head (main): 377b5d3ba65f3288f93020e2f51166fa85947e19
Diff

Github labels

Enabled	Name	Description
	ci-disabled	Disable the ci execution
	ci-all-test	Run all of ci, no test spec filtering will be done
	ci-force-downstream	Force execution of downstream even if twister fails
	ci-run-twister	Force run twister
	ci-run-zephyr-twister	Force run zephyr twister

List of changed files detected by CI (11)

doc
│  ├── _utils
│  │  │ redirects.py
│  ├── nrf
│  │  ├── app_dev
│  │  │  ├── device_guides
│  │  │  │  ├── nrf53
│  │  │  │  │  │ logging_nrf5340.rst
│  │  ├── libraries
│  │  │  ├── security
│  │  │  │  ├── nrf_security
│  │  │  │  │  ├── doc
│  │  │  │  │  │  │ mbed_tls_header.rst
│  │  │  │  │  │ index.rst
│  │  ├── security
│  │  │  ├── tfm
│  │  │  │  │ tfm_building.rst
modules
│  ├── trusted-firmware-m
│  │  ├── tfm_boards
│  │  │  ├── common
│  │  │  │  │ crypto_keys.c
samples
│  ├── bootloader
│  │  ├── socs
│  │  │  ├── nrf54l05_cpuapp.conf
│  │  │  │ nrf54l10_cpuapp.conf
subsys
│  ├── nrf_security
│  │  ├── cmake
│  │  │  │ generate_configs.cmake
│  │  ├── tfm
│  │  │  │ CMakeLists.txt
tests
│  ├── crypto
│  │  ├── src
│  │  │  │ common_test.h

Outputs:

Toolchain

Version: 187d9757da
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:187d9757da_bba2ea5f2e

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

• ◻️ Toolchain - Skipped: existing toolchain is used
• ✅ Build twister - Skipped: Skipping Build & Test as it succeeded in a previous run: 4
• ✅ Integration tests
  • ✅ test-fw-nrfconnect-boot - Skipped: Job was skipped as it succeeded in a previous run
  • ✅ test_ble_nrf_config - Skipped: Job was skipped as it succeeded in a previous run
  • ✅ test-fw-nrfconnect-chip - Skipped: Job was skipped as it succeeded in a previous run
  • ✅ test-fw-nrfconnect-nrf-iot_cloud - Skipped: Job was skipped as it succeeded in a previous run
  • ✅ test-fw-nrfconnect-nrf_crypto
  • ✅ test-fw-nrfconnect-rs - Skipped: Job was skipped as it succeeded in a previous run
  • ✅ test-fw-nrfconnect-tfm - Skipped: Job was skipped as it succeeded in a previous run
  • ✅ test-sdk-find-my - Skipped: Job was skipped as it succeeded in a previous run
  • ✅ test-sdk-mcuboot - Skipped: Job was skipped as it succeeded in a previous run
  • ⚠️ test-fw-nrfconnect-fw-update
  • ⚠️ test-sdk-dfu

Disabled integration tests

• • test-fw-nrfconnect-nrf_lrcs_mosh
  • test-fw-nrfconnect-nrf_lrcs_positioning
  • desktop52_verification
  • doc-internal
  • test-fw-nrfconnect-apps
  • test-fw-nrfconnect-ble_mesh
  • test-fw-nrfconnect-ble_samples
  • test-fw-nrfconnect-fem
  • test-fw-nrfconnect-nfc
  • test-fw-nrfconnect-nrf-iot_libmodem-nrf
  • test-fw-nrfconnect-nrf-iot_lwm2m
  • test-fw-nrfconnect-nrf-iot_samples
  • test-fw-nrfconnect-nrf-iot_serial_lte_modem
  • test-fw-nrfconnect-nrf-iot_thingy91
  • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
  • test-fw-nrfconnect-ps-main
  • test-fw-nrfconnect-rpc
  • test-fw-nrfconnect-thread-main
  • test-low-level
  • test-sdk-audio
  • test-sdk-pmic-samples
  • test-sdk-wifi
  • test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

[greg-fer]

We don't need this page?
If not, please also remove this line https://github.com/tomi-font/ncs/blob/security_fix_undefined_kconfig_references/doc/nrf/libraries/security/nrf_security/index.rst?plain=1#L29

[tomi-font]

No, as said in the commit message this information is outdated, it's not possible to do that anymore judging by our current Kconfig/Cmake logic.
And thanks for the link! I only searched for nrf_security_tls_header which didn't yield any result and so thought that the page was included automatically in the TOC.

[greg-fer]

To fix the build issue, change this line here https://github.com/nrfconnect/sdk-nrf/blob/main/doc/_utils/redirects.py#L556 to:

("libraries/nrf_security/doc/mbed_tls_header", "libraries/security/nrf_security/index"), # User-provided Mbed TLS configuration header (removed in v3.1.0)
("libraries/security/nrf_security/doc/mbed_tls_header", "libraries/security/nrf_security/index")

(Two lines instead of one, both pointing to index for redirects of any old links)

[github-actions]

You can find the documentation preview for this PR here.

Preview links for modified nRF Connect SDK documents:

https://ncsdoc.z6.web.core.windows.net/PR-23168/nrf/app_dev/device_guides/nrf53/logging_nrf5340.html
https://ncsdoc.z6.web.core.windows.net/PR-23168/nrf/libraries/security/nrf_security/index.html
https://ncsdoc.z6.web.core.windows.net/PR-23168/nrf/security/tfm/tfm_building.html

[tomi-font]

@magnev @nrfconnect/ncs-doc-leads please review