-
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
feat(jans-core): use DB document store by default #9267
Conversation
Signed-off-by: Yuriy Movchan <Yuriy.Movchan@gmail.com>
DryRun Security SummaryThe pull request includes changes to the Jans application's configuration, such as transitioning to a database-based document store, updating the SMTP configuration with sensitive information, and configuring an LDAP-based authentication mechanism, all of which require careful security review to ensure proper access controls, encryption, and data protection measures are in place. Expand for full summarySummary: The code changes in this pull request are related to the configuration settings for the Jans application. The key changes include the transition from a local document store to a database-based document store, updates to the SMTP configuration that contain sensitive information, and the configuration of an LDAP-based authentication mechanism. From a security perspective, these changes require careful review to ensure that proper access controls, encryption, and data protection measures are in place for the database storage, the SMTP configuration is properly secured, and the LDAP authentication process is implemented securely, including the use of SSL/TLS. Additionally, the miscellaneous configuration changes should be reviewed to understand their purpose and potential security implications within the overall application architecture. Files Changed:
Code AnalysisWe ran Riskiness🟢 Risk threshold not exceeded. |
|
Signed-off-by: Yuriy Movchan <Yuriy.Movchan@gmail.com> Former-commit-id: 7408954
closes #9179
Closes #9268,