Skip to content

Conversation

@pk-72
Copy link

@pk-72 pk-72 commented Apr 17, 2024

Added Dockerfile for building in docker

and initial version of nginx.conf to run behind the nginx web server

@dima117
Copy link
Contributor

dima117 commented Apr 18, 2024

в PR всё супер! есть предложение:
можем ли собрать вместо двух образов (фронт/бэк) один, который содержит и то и другое?
т.е. смержить содержимое Dockerfile и положить Dockerfile в репозиторий https://github.com/spec-box/install

@dima117
Copy link
Contributor

dima117 commented Apr 18, 2024

и еще кажется, если в одном образе будет и фронт и бэк, то nginx будет не нужен, т.к. статику может раздавать тот же сервер, который обрабатывает запросы на бэк

@pk-72
Copy link
Author

pk-72 commented Apr 18, 2024

nginx дает больше настроек, ну там кеширование еще, сертификаты можно прикрутить и тд, не настолько тяжелый вообщем-то.
в один образ наверное можно засунуть, это будет dotnet developer сервер раздавать статику ?

@dima117
Copy link
Contributor

dima117 commented Apr 18, 2024

в один образ наверное можно засунуть, это будет dotnet developer сервер раздавать статику ?

ну там сервер, который можно использовать и для разработки, и для прода
он довольно быстрый

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.

3 participants