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

add how-to-keep-your-hetzner-server-safe tutorial #866

Merged

Conversation

osbulbul
Copy link
Contributor

@osbulbul osbulbul commented Jul 3, 2024

I have read and understood the Contributor's Certificate of Origin available at the end of
https://raw.githubusercontent.com/hetzneronline/community-content/master/tutorial-template.md
and I hereby certify that I meet the contribution criteria described in it.
Signed-off-by: Oguzhan Selcuk Bulbul [email protected]

@svenja11 svenja11 added the review wanted Request a review label Jul 8, 2024
@svenja11
Copy link
Collaborator

Thank you for your contribution @osbulbul

I read through your tutorial and "Step 1" is pretty similar to an existing tutorial:

In general, we try to avoid duplicates.

Before you start writing your tutorial, make sure there isn't already a tutorial at community.hetzner.com about the exact same topic. source

I would suggest to remove everything that is already in the other tutorial from "Step 1" and link to the other tutorial instead.



With new tutorials, we want to use "Du" instead of "Sie", so it would be great if you could update that too.



In "Monitor logs", you say set up daily reports but you only mention this command:

sudo logwatch --detail High --mailto [email protected] --service all --range today

Do you need a cronjob or something like this to be "daily"?

@svenja11 svenja11 added needs action Something has to be updated and removed review wanted Request a review labels Jul 19, 2024
@osbulbul
Copy link
Contributor Author

Hey @svenja11 thanks for your help and edits. I made the necessary changes, and changed Monitor logs section a little bit.

@svenja11
Copy link
Collaborator

The tutorial that is linked in the German and English text for the initial server setup is not available in Turkish, so I readded the steps you removed from the Turkish file. I also added bullet point to make it easier to follow. Other than that it looks good now and is ready to merge.

@svenja11 svenja11 added ready and removed needs action Something has to be updated labels Jul 26, 2024
@svenja11 svenja11 merged commit cd6d3d9 into hetzneronline:master Jul 26, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants