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

Lighttpd fastcgi #17

Merged
merged 5 commits into from
Mar 14, 2023
Merged

Lighttpd fastcgi #17

merged 5 commits into from
Mar 14, 2023

Conversation

gerdos82
Copy link
Contributor

@gerdos82 gerdos82 commented Mar 13, 2023

Omschrijving

Multiple changes:

  1. Mapproxy Dockerfile uses lighttpd as the base image instead of buster. A separate lighttpd container is no longer needed.
  2. Because the mapproxy Dockerfile is based on lighttpd we can add the standard configuration to the image. You still need to provide the mapproxy.yaml.
  3. Added functionality to add a include.conf for custom lighttpd configuration.
  4. Moved from gunicorn to fastcgi.
  5. Added the pyproj dependency. This is also addressed in Run on Bullseye to use pyproj as recommended. #15. The lighttpd image is still based on buster. And needs to be upgraded to bullseye separately.
  6. Updated Readme. Added description of environment variables.
  7. Updated examples for new setup.

Type verandering

  • Nieuwe feature
  • Refactor
  • Aanpassing van de configuratie
  • Breaking change

Checklist:

  • Ik heb de code in deze PR zelf nogmaals nagekeken
  • Ik heb mijn code beter achtergelaten dan dat ik het aantrof
  • De code is leesbaar en de moeilijke onderdelen zijn voorzien van commentaar
  • Ik heb de tests toegevoegd/uitgebreid indien nodig
  • Ik heb de tests gedraaid die de werking van mijn wijziging bewijst
  • De PDOK documentatie is bijgewerkt indien nodig.
  • Er zit geen gevoelig informatie in deze PR (wachtwoorden etc)

config/lighttpd.conf Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@gerdos82 gerdos82 merged commit c910366 into master Mar 14, 2023
@gerdos82 gerdos82 deleted the lighttpd_fastcgi branch March 14, 2023 10:44
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.

2 participants