-
Notifications
You must be signed in to change notification settings - Fork 85
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
fix(jans-cli-tui): Admin UI roles #8780
Conversation
Signed-off-by: Mustafa Baser <mbaser@mail.com>
Signed-off-by: Mustafa Baser <mbaser@mail.com>
Hi there 👋, @DryRunSecurity here, below is a summary of our analysis and findings.
Note 🟢 Risk threshold not exceeded. Change Summary (click to expand)The following is a summary of changes in this pull request made by me, your security buddy 🤖. Note that this summary is auto-generated and not meant to be a definitive list of security issues but rather a helpful summary from a security perspective. Summary: The provided code changes cover various aspects of the Jans Command Line Interface (CLI) and Text-based User Interface (TUI) application, including improvements to logging, user management functionality, and background task implementation. While the changes do not directly introduce any obvious security vulnerabilities, there are a few security-related considerations that should be addressed:
Files Changed:
Powered by DryRun Security |
|
|
|
|
|
|
|
|
* chore(jans-cli-tui): more logging Signed-off-by: Mustafa Baser <mbaser@mail.com> * fix(jans-cli-tui): User Admin UI roles Signed-off-by: Mustafa Baser <mbaser@mail.com> --------- Signed-off-by: Mustafa Baser <mbaser@mail.com> Former-commit-id: 8a88d6f
closes #8765