Skip to content

Conversation

@D-Triveni
Copy link
Contributor

Add changes to read parameters required to
validate the server certificate.

Fixes SHEL-3880.

@D-Triveni D-Triveni requested a review from a team as a code owner October 24, 2025 06:52
@D-Triveni D-Triveni requested a review from rado17 October 24, 2025 06:52
@NordicBuilder NordicBuilder added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 24, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 24, 2025

CI Information

To view the history of this post, click the 'edited' button above
Build number: 5

Inputs:

Sources:

sdk-nrf: PR head: 8a2533d3ff722a28b6a97f8a59fbd3f0576e3ca6
hostap: PR head: 6086dea5ee7406e1eede7f2ca6dff1b00b0f04e2
zephyr: PR head: 14fb2bff5f15e9b12a70f0b33958daf098602c6c

more details

sdk-nrf:

PR head: 8a2533d3ff722a28b6a97f8a59fbd3f0576e3ca6
merge base: 864c91b8865517af98d7394412771015fb54b443
target head (main): 34d58781fbac05f30ce7128e418cb597603a35e3
Diff

hostap:

PR head: 6086dea5ee7406e1eede7f2ca6dff1b00b0f04e2
merge base: 5abcff1c0ecff65f0f81e0cc086b7f766e5101bf
Diff

zephyr:

PR head: 14fb2bff5f15e9b12a70f0b33958daf098602c6c
merge base: 164b47d3094202c9ac5b756aa2648d2ce5822680
target head (main): b5cf3eaf422232176fe4eb00201c811647e8aa64
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 (29)
modules
│  ├── lib
│  │  ├── hostap
│  │  │  ├── hostapd
│  │  │  │  ├── hostapd_cli_zephyr.c
│  │  │  │  │ hostapd_cli_zephyr.h
│  │  │  ├── src
│  │  │  │  ├── ap
│  │  │  │  │  │ hostapd.h
│  │  │  │  ├── common
│  │  │  │  │  │ wpa_ctrl.h
│  │  │  │  ├── crypto
│  │  │  │  │  │ tls_mbedtls_alt.c
│  │  │  │  ├── drivers
│  │  │  │  │  ├── driver_zephyr.c
│  │  │  │  │  │ driver_zephyr.h
│  │  │  │  ├── utils
│  │  │  │  │  ├── includes.h
│  │  │  │  │  │ os_zephyr.c
│  │  │  ├── wpa_supplicant
│  │  │  │  ├── wpa_cli.c
│  │  │  │  ├── wpa_cli_cmds.c
│  │  │  │  ├── wpa_cli_zephyr.c
│  │  │  │  ├── wpa_cli_zephyr.h
│  │  │  │  ├── wpa_supplicant.c
│  │  │  │  │ wpa_supplicant_i.h
│  ├── wfa-qt
│  │  ├── src
│  │  │  │ indigo_api_callback_dut.c
west.yml
zephyr
│  ├── doc
│  │  ├── connectivity
│  │  │  ├── networking
│  │  │  │  ├── api
│  │  │  │  │  │ wifi.rst
│  ├── drivers
│  │  ├── modem
│  │  │  │ modem_cellular.c
│  ├── include
│  │  ├── zephyr
│  │  │  ├── net
│  │  │  │  │ wifi_mgmt.h
│  ├── modules
│  │  ├── hostap
│  │  │  ├── CMakeLists.txt
│  │  │  ├── Kconfig
│  │  │  ├── src
│  │  │  │  ├── supp_api.c
│  │  │  │  ├── supp_main.h
│  │  │  │  │ wpa_cli.c
│  ├── subsys
│  │  ├── mgmt
│  │  │  ├── mcumgr
│  │  │  │  ├── grp
│  │  │  │  │  ├── img_mgmt
│  │  │  │  │  │  ├── Kconfig
│  │  │  │  │  │  ├── src
│  │  │  │  │  │  │  │ img_mgmt.c
│  │  ├── net
│  │  │  ├── l2
│  │  │  │  ├── wifi
│  │  │  │  │  │ wifi_shell.c
│  │ west.yml

Outputs:

Toolchain

Version: df3cc9d822
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:df3cc9d822_e595b21c39

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ❌ Build twister
    • sdk-nrf test count: 1020
    • sdk-zephyr test count: 767
  • ❌ Integration tests
    • ❌ test-sdk-audio
    • ❌ test-fw-nrfconnect-chip
    • ❌ test-fw-nrfconnect-nfc
    • ❌ test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • ❌ test-fw-nrfconnect-nrf-iot_thingy91
    • ❌ test-fw-nrfconnect-rs
    • ❌ test-fw-nrfconnect-fem
    • ❌ test-fw-nrfconnect-thread-main
    • ❌ test-sdk-find-my
    • ❌ test-fw-nrfconnect-nrf_lrcs_positioning
    • ❌ test-sdk-wifi
    • ❌ test-low-level
    • ❌ test-sdk-mcuboot
    • ❌ test-sdk-dfu
    • ⚠️ test-fw-nrfconnect-nrf-iot_cloud
    • ⚠️ test-fw-nrfconnect-nrf_lrcs_mosh
    • ⚠️ test-fw-nrfconnect-nrf_lrcs_positioning
Disabled integration tests
    • desktop52_verification
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-ps-main
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-tfm
    • test-secdom-samples-public

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

@D-Triveni D-Triveni added the DNM label Oct 27, 2025
@krga2022 krga2022 self-requested a review November 5, 2025 04:11
@D-Triveni D-Triveni requested review from a team as code owners November 7, 2025 08:56
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Nov 7, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@164b47d nrfconnect/sdk-zephyr#3478 nrfconnect/sdk-zephyr#3478/files

DNM label due to: 1 project with PR revision

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@D-Triveni D-Triveni force-pushed the fix_server_valid_configuration branch from 97b98cc to d901c88 Compare November 7, 2025 09:26
@github-actions
Copy link

github-actions bot commented Nov 7, 2025

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-25220/nrf/samples/cellular/tls_ciphersuites/README.html

This change pulls in the server validation using exact
domain match and domain suffix match options.

Signed-off-by: Triveni Danda <triveni.danda@nordicsemi.no>
Add changes to read parameters required to
validate the server certificate.

Also, set the PMF option as required for the realted testcase.

Fixes SHEL-3880.

Signed-off-by: Triveni Danda <triveni.danda@nordicsemi.no>
@D-Triveni D-Triveni force-pushed the fix_server_valid_configuration branch from d901c88 to 91e6bbf Compare November 11, 2025 05:42
Fix build errors that are observed after recent hostap
upmerge.

Signed-off-by: Triveni Danda <triveni.danda@nordicsemi.no>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. DNM manifest manifest-zephyr

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants