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

Fix #845 and dockerfile cleanup #848

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Nov 28, 2022

  1. fix blindsidenetworks#845 and cleanup

    There were more issues with the alpine nginx images than I first realised.
    
    This patch:
    - Moves nginx/conf.d to nginx/http.d because that is how the directory
      is now called within images
    - Fixes the nginx/start script to work with /etc/nginx/http.d instead of
      the deprecated and missing /etc/nginx/conf.d
    - Fixes alpine-based Dockerfiles to copy config files from/to the correct
      locations
    - Fixes amazonlinux-based Dockerfiles to also store config in
      /etc/nginx/http.d so nginx/start works as expeted
    - Harmonized files in dockerfiles/v1/. Lots of changes in the
      past were not ported to all of these files. They should only differ
      in the base image used (alpine/amazonlinux) and the bbb-playback target.
      Base image versions or other details should be the same.
    defnull committed Nov 28, 2022
    Configuration menu
    Copy the full SHA
    76481a5 View commit details
    Browse the repository at this point in the history