Radius 2.1.0#689
Conversation
…ment Update Deploy-UserCertificate Functions to generate certs on SHA match
Fix hyphen user generation and EmailSAN certificate replacement
…unctions Get User to Device Reports with SDK Functions
There was a problem hiding this comment.
Pull Request Overview
Release of the 2.1.0 Radius Module, adding end-to-end PowerShell automation for certificate generation and distribution.
- Refactored README to use
Set-JCRConfigcmdlet for all configuration settings - Introduced
multi_group_radius.ps1for scheduled multi-group automation - Generated placeholder documentation and help files for new module cmdlets
Reviewed Changes
Copilot reviewed 121 out of 139 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| scripts/automation/Radius/readme.md | Updated requirements and moved to Set-JCRConfig usage |
| scripts/automation/Radius/multi_group_radius.ps1 | New script for automating certificate workflows |
| scripts/automation/Radius/en-Us/about_JumpCloud.Radius.help.txt | Added help topic stub with placeholders |
Comments suppressed due to low confidence (2)
scripts/automation/Radius/en-Us/about_JumpCloud.Radius.help.txt:18
- Replace the placeholder text with an actual short description to provide meaningful help content for the cmdlet.
{{ Short Description Placeholder }}
scripts/automation/Radius/multi_group_radius.ps1:34
- The
Write-ToLogfunction references$Levelbut does not define it as a parameter. Consider adding a-Levelparameter with a default (e.g., 'Info') so log entries include valid severity labels.
Switch ($Level) {
jworkmanjc
left a comment
jworkmanjc
left a comment
There was a problem hiding this comment.
Ready for review @kmaranionjc
gweinjc
left a comment
gweinjc
left a comment
There was a problem hiding this comment.
Tested and everything worked good. Was able to generate a Root CA and subsequent user certs and deploy a cert to a user successfully.
Found one minor thing in the readme about the current JumpCloud powershell module pointing to an older version but everything else read well and was understandable.
| @@ -63,46 +64,148 @@ After installing PowerShell 7.x.x, install the [JumpCloud PowerShell Module](htt | |||
|
|
|||
| At the time of this writing JumpCloud Module 2.1.3 was the latest version. Please ensure you are at least running this version of the PowerShell Module. | |||
There was a problem hiding this comment.
May want to update this to point to the current version of the PWSH module (2.18.1)
kmaranionjc
left a comment
kmaranionjc
left a comment
There was a problem hiding this comment.
Looks good on my end.
Issues
What does this solve?
Release of the 2.1.0 Radius Module, an update to the 1.0.7 version which includes functionality for those looking to auto generate certificates through a PowerShell Module.
Is there anything particularly tricky?
How should this be tested?
Screenshots