move custom REDIS_HOST from Dockerfile to compose

We now use an explicit env var in the compose file. This should make it more clear which hostname the api will try to connect to.
pull/8/head
Angel Garcia 3 years ago
parent 91f82dbb96
commit 7923c4fc22

@ -5,6 +5,8 @@ services:
build: build:
context: jikan context: jikan
dockerfile: Dockerfile dockerfile: Dockerfile
environment:
- REDIS_HOST=jikan-redis
depends_on: depends_on:
- jikan-redis - jikan-redis
ports: ports:

@ -18,7 +18,7 @@ RUN git clone https://github.com/jikan-me/jikan-rest.git -b v3 . \
&& composer update jikan-me/jikan && composer update jikan-me/jikan
RUN sed -i "s/CACHE_DRIVER=.*/CACHE_DRIVER=redis/" .env \ RUN sed -i "s/CACHE_DRIVER=.*/CACHE_DRIVER=redis/" .env \
&& sed -i "s/REDIS_HOST=.*/REDIS_HOST=jikan-redis/" .env \ && sed -i "s/REDIS_HOST=.*/REDIS_HOST=redis/" .env \
&& sed -i "s/CACHE_404_EXPIRE=.*/CACHE_404_EXPIRE=3600/" .env \ && sed -i "s/CACHE_404_EXPIRE=.*/CACHE_404_EXPIRE=3600/" .env \
&& sed -i "s/CACHE_SEARCH_EXPIRE=.*/CACHE_SEARCH_EXPIRE=1000/" .env \ && sed -i "s/CACHE_SEARCH_EXPIRE=.*/CACHE_SEARCH_EXPIRE=1000/" .env \
&& sed -i "s/GITHUB_REPORTING=.*/GITHUB_REPORTING=false/" .env && sed -i "s/GITHUB_REPORTING=.*/GITHUB_REPORTING=false/" .env

Loading…
Cancel
Save