search

sympa-community / sympa-community.github.io

Incubating the new Sympa documentation site
https://www.sympa.community
Other
11 stars 34 forks source link

Smalls addons for nginx config #77

Closed dcaillibaud closed 2 years ago

dcaillibaud commented 3 years ago

Adding some fastcgi parameter to nginx config (and a link to wwsympa.service sample) for spawn-fcgi

ikedas commented 3 years ago

As of 6.2.56, wwsympa became splitting URI by itself: See sympa-community/sympa#879 for details. That's why description about split_path_info was removed from example: It was moved to the note below.

As you wrote, CSS path may be required. I'll check it.

Thanks for inputs!

dcaillibaud commented 3 years ago

Sorry, I didn't see theses notes (I'm still in 6.2.40 with debian buster and these lines are required, as note says). I don't know if you can exclude the second commit from this PR, to keep the link added in the first commit.

If not I let you close the PR without merge, sorry for the noise.

racke commented 3 years ago

The CSS part is only needed for Debian installations.

ikedas commented 3 years ago

Hi @dcaillibaud ,

As you wrote, CSS path may be required. I'll check it.

$CSSDIR and $PICTURESDIR were subdirectories of $STATICDIR by default, on version prior to 6.2.26. However recently they are vary by environments (check the links of three). For purpose of example, I think all of these three would be better to appear in description.

BTW, to include your changes, could you please include addition of your name to /AUTHORS.md file into this PR? It is required by the license applied to this documentation (CC-BY-SA 4.0).

ikedas commented 2 years ago

Hi @dcaillibaud , thanks for proposal of improvements. I included most of them into my PR and merged it. If you noticed anything, please submit changes again.

Thank you for input!