Skip to content

net: l2: wifi: Refactor certificates processing code into common file #93119

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 2 commits into
base: main
Choose a base branch
from

Conversation

D-Triveni
Copy link
Contributor

Refactor certificate processing code to eliminate duplication and enable reuse across modules that require enterprise support.

This change is to fix the eloop.h header file path.

Signed-off-by: Triveni Danda <triveni.danda@nordicsemi.no>
@D-Triveni D-Triveni force-pushed the refactor_wifi_certs_process branch from d2e696a to 134caa6 Compare July 15, 2025 11:27
Copy link

github-actions bot commented Jul 15, 2025

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

Name Old Revision New Revision Diff
hostap zephyrproject-rtos/hostap@bc5d22f zephyrproject-rtos/hostap#92 zephyrproject-rtos/hostap#92/files

DNM label due to: 1 project with PR revision

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

@github-actions github-actions bot added manifest manifest-hostap DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Jul 15, 2025
Refactor certificate processing code to eliminate duplication and
enable reuse across modules that require enterprise support.

Signed-off-by: Triveni Danda <triveni.danda@nordicsemi.no>
@D-Triveni D-Triveni force-pushed the refactor_wifi_certs_process branch from 134caa6 to 1ed0285 Compare July 15, 2025 12:47
Copy link
Contributor

@krish2718 krish2718 left a comment

Choose a reason for hiding this comment

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

LGTM, do we need an entry in the changelog?

Copy link

@krish2718
Copy link
Contributor

LGTM, do we need an entry in the changelog?

hold-off on this till it's confirmed that it's part of 4.2 which I doubt as its too close.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Networking area: Wi-Fi Wi-Fi DNM (manifest) This PR should not be merged (controlled by action-manifest) manifest manifest-hostap
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants