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

Add docker containers #12

Merged

Conversation

KarlsonComplete
Copy link
Collaborator

@KarlsonComplete KarlsonComplete commented Sep 24, 2024

Добавил папку docker внутри которой расположил структуру для запуска контейнеров с nginx, php-fpm и postgres.
Сам файлик docker-compose.yaml расположил в корне библиотеки.

@mesilov mesilov marked this pull request as draft September 29, 2024 14:44
@@ -0,0 +1,20 @@
server {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@KarlsonComplete это из елисеева или примеров симфони?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

У елисеева в первом видео было.

@@ -0,0 +1,11 @@
FROM php:8.2-fpm-alpine
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

а какие экстеншены ты по дефолту доставишь и включишь?

FROM postgres:16.4-alpine

# Устанавливаем переменные окружения для настройки базы данных
ENV POSTGRES_USER=kirill
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

погугли что там по этому поводу находится, закинь сюда варианты.

Add CI step for monitoring leaked api-keys in codebase

Кажись вот тут ребята рассказывали
https://www.youtube.com/watch?v=YFWXV0YrDnE

context: ./docker/php-fpm
volumes:
- ./:/var/www/bitrix24-php-lib
nginx:
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@KarlsonComplete зачем тебе nginx если ты гонишь все в CLI?

<?php
echo 'Hello123';
echo "<br>";
$conn_string = "pgsql:host=localhost;port=5432;dbname=bitrixAppDb;user=kirill;password=bitrix24lib";
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

всё через переменные окружения

@mesilov mesilov marked this pull request as ready for review October 6, 2024 14:14
@mesilov mesilov merged commit 4c2f9b9 into mesilov:add-docker-structure Oct 6, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
2 participants