Skip to content
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

net/freeradius: EAP-TLS with multiple CAs #4381

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

Conversation

razza-guhl
Copy link

@razza-guhl razza-guhl commented Dec 2, 2024

Description:
This PR enables the configuration of multiple CA certificates for EAP-TLS authentication in FreeRADIUS. This is useful for environments where client devices (e.g., laptops, desktops) use certificates from an internal private CA, while devices like VoIP phones and printers use certificates issued by their vendor's CA.

The configuration aligns with the FreeRADIUS documentation regarding the "ca_file" directive, which supports multiple CA certificates:
[FreeRADIUS Documentation - ca_file](https://networkradius.com/doc/current/raddb/tls/tls-config_tls-common.html)

To implement this, the controller and model for eap was modified. And the "generate_certs.php" script was updated to handle and process multiple refid values when provided.

Changes:

  • Modified "generate_certs.php" to support multiple CA references.
  • Updated logic to accommodate additional CA certificates for EAP-TLS configurations.
  • Modified the UI for multiple CA input.

Testing:

  • Verified functionality with multiple CAs in test environments for both internal and vendor-supplied certificates.
  • Ensured backward compatibility for single CA setups.

Let me know if additional tests or refinements are needed!

@razza-guhl razza-guhl force-pushed the freeradius-eap-tls-multiple-ca branch from b3eb2e0 to 32d4709 Compare December 2, 2024 17:06
@razza-guhl razza-guhl marked this pull request as ready for review December 2, 2024 18:14
Copy link
Author

@razza-guhl razza-guhl left a comment

Choose a reason for hiding this comment

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

Tested and works as aspected

Copy link
Member

@mimugmail mimugmail left a comment

Choose a reason for hiding this comment

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

Yes, this looks good and works on my system, thx. @fichtner there is a second PR open so no version bump here and we'll do it afterwards. ok?

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

Successfully merging this pull request may close these issues.

2 participants