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

Update to the windows deployment script #3000

Merged
merged 1 commit into from
Dec 4, 2023

Conversation

coloradosarge
Copy link
Contributor

This PR updates the deploy_timesketch.ps1 script to include a cryptographically Windows CRNG function for generating the POSTGRES_PASSWORD and SECRET_KEY variables. It also corrects the 'sed' (Get-Content) lines to update OPENSEARCH_HOST and OPENSEARCH_PORT in the timesketch.conf file instead of ELASTIC_HOST and ELASTIC_PORT variables which have changed in the current timesketch.conf file. The incorrectly named variables were causing the installation to fail on Windows without manual fixes.

Checks

  • All tests succeed.
  • Unit tests added.
  • e2e tests added.
  • Documentation updated. N/A

Closing issues
closes #2986
closes #2985

…anged 'sed' function to correctly update timesketch.conf OPENSEARCH_HOST and OPENSEARCH_PORT paramters
Copy link

google-cla bot commented Dec 4, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@jkppr jkppr self-requested a review December 4, 2023 15:14
@jkppr jkppr added Code Health Code health improvements Bug labels Dec 4, 2023
@jkppr jkppr added this to the Release: 20231206 milestone Dec 4, 2023
@jkppr jkppr changed the title Added function to generate cryptographically random characters and ch… Update to the windows deployment script Dec 4, 2023
Copy link
Collaborator

@jkppr jkppr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm
Thanks for contributing to Timesketch

@jkppr jkppr merged commit e36a4ee into google:master Dec 4, 2023
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Code Health Code health improvements
Projects
None yet
2 participants