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.

Sign up for GitHub

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

Jump to bottom

nginx folder from Docker-compose to docker-example #158

Closed
francesco-filicetti opened this issue Sep 13, 2024 · 1 comment · Fixed by #160
Closed

nginx folder from Docker-compose to docker-example #158

francesco-filicetti opened this issue Sep 13, 2024 · 1 comment · Fixed by #160

Comments

@francesco-filicetti
Copy link
Member

The files in this folder are used to customize the Nginx configuration (certificates, templates, uwsgi conf).
Maybe it would be more useful to have it in docker-example (moving the one with the example files to /example) and not in Docker-compose.
@MdreW
Copy link
Collaborator

MdreW commented Sep 15, 2024

Hi @francesco-filicetti , in PR #152 I'm tidying up some parts of the project.
The satosa-nginx service is configured with as a standard virtual host, default.conf, and I think that old example file can be removed.
Also the docs can be moved to docs directory.
Only uwsgy.ini is required but can go out from uwsgy_setup directory and we can delete the directory.
Look the PR and let me known that you think.

Many talks about this PR are in the Satosa slack channel You can find more information here.

MdreW added a commit that referenced this issue Sep 16, 2024
@MdreW
Local mount + docs + fix #156 , #158, #134
1080c92
@MdreW MdreW mentioned this issue Sep 16, 2024
Merged
peppelinux added a commit that referenced this issue Sep 27, 2024
@MdreW @peppelinux
local mount, profiles and new docs (#160)
6af704e 
* Local mount + docs + fix #156 , #158, #134

* fix #159

* example env file must be not an hidden file

* updated compose istruction

* update index

* Fixed title and added change directory in LONG WAY

* Another index fix

* SystemD doc moved to /docs

* uWSGI config doc with example

* fixed doc

* Add TZ env to NGINX

* add new docs and removed old examples

* workflow with new path

* path with new schema

* Update Docker-compose/README.md

Co-authored-by: Giuseppe De Marco <[email protected]>

* Update Docker-compose/README.md

Co-authored-by: Giuseppe De Marco <[email protected]>

* Update docs/satosa-nginx_compose_service.md

Co-authored-by: Giuseppe De Marco <[email protected]>

* Update docs/satosa-nginx_compose_service.md

Co-authored-by: Giuseppe De Marco <[email protected]>

* Update docs/satosa-nginx_compose_service.md

Co-authored-by: Giuseppe De Marco <[email protected]>

* Update docs/satosa-nginx_compose_service.md

Co-authored-by: Giuseppe De Marco <[email protected]>

---------

Co-authored-by: Giuseppe De Marco <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

local mount, profiles and new docs italia/Satosa-Saml2Spid
2 participants
@MdreW @francesco-filicetti