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

There's no file such as data/conf/nginx/listen_* anymore. #751

Merged
merged 3 commits into from
Nov 6, 2024

Conversation

shahzisme
Copy link
Contributor

In the Step 5 of this documentation:

https://docs.mailcow.email/post_installation/firststeps-disable_ipv6/

The following line refers to a non-existent (or no longer existent) file:
sed -i '/::/d' data/conf/nginx/listen_*

The only file is in the data/conf/nginx/templates/listen* which is the next line.

…a/conf/nginx/templates/listen* which is the next line
@DerLinkman
Copy link
Member

Please remove it for the DE Version too

…a/conf/nginx/templates/listen* which is the next line. Updated for DE docs too.
@shahzisme
Copy link
Contributor Author

Updated for DE version too

.DS_Store Outdated
Copy link
Member

Choose a reason for hiding this comment

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

What are those? Please remove them from your branch!

docs/.DS_Store Outdated
Copy link
Member

Choose a reason for hiding this comment

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

What are those? Please remove them from your branch

@Jniklas2
Copy link
Contributor

Jniklas2 commented Sep 16, 2024

Those files (listen_*) get created, as soon as you start mailcow. I just created a test instance for me, and there are two matching files for that regex.

And those .DS_Store files are typical Mac OS files. Maybe something for a .gitignore file, so those files are ignored automatically, @DerLinkman ?

@DerLinkman
Copy link
Member

Yeah sure why not

Copy link
Member

@DerLinkman DerLinkman left a comment

Choose a reason for hiding this comment

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

Looks fine now!

@DerLinkman DerLinkman merged commit 0ff6e59 into mailcow:master Nov 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants