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

chore(jans-linux-setup): restart jans services in post-setup #9400

Merged
merged 2 commits into from
Sep 6, 2024

Conversation

devrimyatar
Copy link
Contributor

closes #9392

  • I confirm that there is no impact on the docs due to the code changes in this PR.

Signed-off-by: Mustafa Baser <mbaser@mail.com>
@devrimyatar devrimyatar added the comp-jans-linux-setup Component affected by issue or PR label Sep 6, 2024
Copy link

dryrunsecurity bot commented Sep 6, 2024

DryRun Security Summary

The provided code change updates the post_installation message in the messages.py file of the jans-linux-setup/jans_setup/setup_app module to provide a specific command for restarting the Janssen Services, and the rest of the file contents provide various messages and labels used throughout the Janssen Server installation process.

Expand for full summary

Summary:

The provided code change appears to be an update to the messages.py file in the jans-linux-setup/jans_setup/setup_app module. The change updates the post_installation message to provide a specific command for restarting the Janssen Services. From an application security perspective, this change does not introduce any obvious security concerns, as it simply provides clearer instructions to the user on how to complete the installation process.

The rest of the file contents provide various messages and labels used throughout the Janssen Server installation process, covering a wide range of topics, including system information, service selection, storage options, and installation descriptions. While not directly related to security, these messages play an important role in guiding the user through the installation and configuration of the Janssen Server. Overall, this code change seems to be a routine update to the user-facing messages and does not raise any immediate application security concerns.

Files Changed:

  • jans-linux-setup/jans_setup/setup_app/messages.py: This file has been updated to modify the post_installation message, providing a specific command for restarting the Janssen Services. The rest of the file contents provide various messages and labels used throughout the Janssen Server installation process, covering a wide range of topics, such as system information, service selection, storage options, and installation descriptions.

Code Analysis

We ran 9 analyzers against 1 file and 0 analyzers had findings. 9 analyzers had no findings.

Riskiness

🟢 Risk threshold not exceeded.

View PR in the DryRun Dashboard.

@mo-auto mo-auto added the kind-dependencies Pull requests that update a dependency file label Sep 6, 2024
Copy link

sonarqubecloud bot commented Sep 6, 2024

Copy link

sonarqubecloud bot commented Sep 6, 2024

@yuriyz yuriyz merged commit 314f247 into main Sep 6, 2024
12 checks passed
@yuriyz yuriyz deleted the jans-linux-setup-post-setup-message-9392 branch September 6, 2024 12:43
yuriyz pushed a commit that referenced this pull request Nov 7, 2024
Signed-off-by: Mustafa Baser <mbaser@mail.com>
Former-commit-id: 314f247
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp-jans-linux-setup Component affected by issue or PR kind-dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

chore(jans-linux-setup): restart services in post-setup message
4 participants