Skip to content

Conversation

PoneyClairDeLune
Copy link
Contributor

@PoneyClairDeLune PoneyClairDeLune commented Aug 24, 2024

Translated from NGINX configs. Should allow Caddy as a viable alternative to NGINX for deployments.

If this is merged, the corresponding documentation will need to be updated.

@PoneyClairDeLune PoneyClairDeLune marked this pull request as ready for review August 24, 2024 19:09
@PoneyClairDeLune PoneyClairDeLune changed the title Add an example config for Caddy Add example configs for Caddy Aug 25, 2024
@mathilde-cryptpad mathilde-cryptpad changed the base branch from main to staging August 27, 2024 08:35
@mathilde-cryptpad mathilde-cryptpad self-requested a review September 3, 2024 08:22
@mathilde-cryptpad mathilde-cryptpad added the Reverse proxy Web server or reverse proxy issues label Sep 3, 2024
mathilde-cryptpad

This comment was marked as outdated.

@mathilde-cryptpad mathilde-cryptpad added OK to Merge This PR can be merged to staging and removed OK to Merge This PR can be merged to staging labels Sep 3, 2024
Copy link
Contributor

@mathilde-cryptpad mathilde-cryptpad left a comment

Choose a reason for hiding this comment

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

After some discussion with the team, we wish for some small changes. Could you please move those two Caddy configuration files to the docs/community folder?

We would like to make even clearer what is officially supported by the project and what isn't.

Thanks in advance!

@PoneyClairDeLune
Copy link
Contributor Author

I'll start working on this at my next opportunity.

@PoneyClairDeLune
Copy link
Contributor Author

Emails changed and the two example config files have been moved to docs/community.

Copy link
Contributor

@mathilde-cryptpad mathilde-cryptpad left a comment

Choose a reason for hiding this comment

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

Perfect now, many thanks for your contribution!

@mathilde-cryptpad mathilde-cryptpad added the OK to Merge This PR can be merged to staging label Sep 4, 2024
@mathilde-cryptpad mathilde-cryptpad merged commit 879f8a7 into cryptpad:staging Sep 6, 2024
@mathilde-cryptpad mathilde-cryptpad removed the OK to Merge This PR can be merged to staging label Sep 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Reverse proxy Web server or reverse proxy issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants