DGNum/dgsi
11
0
Fork 0
Code Issues 6 Pull requests 4 Projects Releases Packages Wiki Activity Actions

Manage the radius secret locally #60

Open
thubrecht wants to merge 10 commits from radius into main
pull from: radius
merge into: DGNum:main
Conversation 1 Commits 10 Files changed 18 +366 -206
thubrecht commented 2026-06-28 00:52:14 +02:00
Owner
Copy link
No description provided.
feat(api/user): Also send the radius secret in the response
Some checks failed
Check that artifacts are already generated / up-to-date (pull_request) Failing after 6s
Details
Build the packages / dgsi-build (pull_request) Successful in 14s
Details
34d8f004c8
thubrecht force-pushed radius from 34d8f004c8
Some checks failed
Check that artifacts are already generated / up-to-date (pull_request) Failing after 6s
Details
Build the packages / dgsi-build (pull_request) Successful in 14s
Details
to 8d47dd5300
Some checks failed
Check that artifacts are already generated / up-to-date (pull_request) Failing after 11s
Details
Build the packages / dgsi-build (pull_request) Successful in 14s
Details
2026-06-28 00:57:45 +02:00 Compare
mdebray reviewed 2026-06-28 09:03:36 +02:00
src/dgsi/views.py Outdated
@ -242,3 +242,3 @@
if not self._user.kanidm.radius_secret:
if not self._user.radius_secret:
message = _("Mot de passe Wi-Fi généré avec succès.")
sync_call("group_add_members", "radius_access", [self._user.username])
mdebray commented 2026-06-28 09:03:36 +02:00
Owner
Copy link

Toujours besoin de ça ?

Toujours besoin de ça ?
thubrecht commented 2026-06-28 10:21:41 +02:00
Author
Owner
Copy link

Tant que j'ai pas mis un champ pour dire qu'une personne est ban du radius oui, ce sera dans un commit suivant

Tant que j'ai pas mis un champ pour dire qu'une personne est ban du radius oui, ce sera dans un commit suivant
thubrecht marked this conversation as resolved 
This allows banning some user for infringing on the rules
chore(dgsi/generate-wifi-password): Simplify logic
Some checks failed
Check that artifacts are already generated / up-to-date (pull_request) Failing after 12s
Details
Build the packages / dgsi-build (pull_request) Successful in 14s
Details
03a3b06a6e
 
The network access checks is done through a boolean in dgsi, no need to
set membership to the corresponding groupe
thubrecht force-pushed radius from 03a3b06a6e
Some checks failed
Check that artifacts are already generated / up-to-date (pull_request) Failing after 12s
Details
Build the packages / dgsi-build (pull_request) Successful in 14s
Details
to af89ed6bfd
Some checks failed
Check that artifacts are already generated / up-to-date (pull_request) Failing after 12s
Details
Build the packages / dgsi-build (pull_request) Successful in 14s
Details
2026-08-16 22:27:04 +02:00 Compare
thubrecht force-pushed radius from af89ed6bfd
Some checks failed
Check that artifacts are already generated / up-to-date (pull_request) Failing after 12s
Details
Build the packages / dgsi-build (pull_request) Successful in 14s
Details
to cfdca89e82
All checks were successful
Check that artifacts are already generated / up-to-date (pull_request) Successful in 11s
Details
Build the packages / dgsi-build (pull_request) Successful in 14s
Details
2026-08-16 22:31:17 +02:00 Compare
thubrecht changed title from WIP: Manage the radius secret locally to Manage the radius secret locally 2026-08-16 22:31:27 +02:00 
It is more noise than useful
instead of saying that they could not be found, it is the job of the
consumer to act on that information
api/user-view: also return the site's information
All checks were successful
Check that artifacts are already generated / up-to-date (pull_request) Successful in 11s
Details
Build the packages / dgsi-build (pull_request) Successful in 16s
Details
c97a4505e4
thubrecht force-pushed radius from c97a4505e4
All checks were successful
Check that artifacts are already generated / up-to-date (pull_request) Successful in 11s
Details
Build the packages / dgsi-build (pull_request) Successful in 16s
Details
to 51c9c677d6
All checks were successful
Build the packages / dgsi-build (pull_request) Successful in 14s
Details
Check that artifacts are already generated / up-to-date (pull_request) Successful in 1m31s
Details
2026-08-16 23:09:46 +02:00 Compare
All checks were successful
Build the packages / dgsi-build (pull_request) Successful in 14s
Details
Check that artifacts are already generated / up-to-date (pull_request) Successful in 1m31s
Details
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin radius:radius
git switch radius
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
  
bug
Something is not working

  
duplicate
This issue or pull request already exists

  
enhancement
New feature

  
help wanted
Need some help

  
invalid
Something is wrong

  
question
More information is needed

  
wontfix
This won't be fixed

No labels
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
DGNum/dgsi!60
No description provided.