[config] MYSQL configurations to env #232

borazslo · 2024-03-02T23:18:36Z

#230 kapcsán: ez így jobbnak tűnik? Így a MYSQL_USER és egyéb beállításokat az .env.local -ba kell megejteni egyetlen helyen és nincs beégetve. És ha nem csinálom meg a rendes .env.local fájlomat, akkor nem is indul el semmi és a .env az maradhat boldog no-touch zone.

issue: docker-compose.yml -ben a mysql healthcheck része nem működik még mert oda be van égetve a mysql név és jelszó!

route alias nem elegáns így, de legacy-ba elmegy talán

…t container akkor adja hozza a containert

…emplomra

- user login kezeles - routerben lehessen methodus nevre is hivatkozni - user getter / setter halmok - es meg jo sok aprosag

…töltése és symfony alap form téma beállítása (#203)

* [user] regisztráció tesztelések és hibajavítások * [github] node setup es asset build

* [security] lemaradt login type * [security] minimal teszt, hogy latszodjon ha valami nem mukodik

)

* [github] php 8.3 * [docker] php 8.3 * [docker] mlocati php ext installer-et hasznaljuk, egyszerubb

* [user] profil oldal letrehozasa #177 * [user] kedvencek torlese (+turbo hozzaadasa) * cs fixer * [composer] route generalas * [static] statikus oldal routeok felvetele es javitas (#208) * [security] lemaradt LoginType #203 (#210) * [security] lemaradt login type * [security] minimal teszt, hogy latszodjon ha valami nem mukodik * [style] footer alul legyen ha keves a tartalom, de tolja le ha sok (#211) * [docker/qa] frissites: php 83 (#212) * [github] php 8.3 * [docker] php 8.3 * [docker] mlocati php ext installer-et hasznaljuk, egyszerubb * [user/favorite] templom legyen parameter inkabb * kedvenceles kezeles turbo-val * cs fix * [user] saját felhasználói adatok szerkesztése * [cs] void return true * [cs] legacy mappat egyelore kivonjuk az ellenorzes alol

* [ci] codecov integráció * [ci] coverage badge

* [composer] kapjon nevteret a teszt mappa * [api clients] hamvazoszerda es hiba teszteles. timeout 2 secre nott * [api clients] 500-as valaszt is teszteljuk le * [phpunit] tesztek BP timezone-ban fussanak * [composer] local coverage generalas * [api clients] karácsony hétköznap tesztelése (a főünnepe szöveg miatt)

* [ci] adatbázis migrációs teszt környezet * cleanup * [github] database migration workflow es teszt adatok * [composer] lock mostantol nem jatszik * [doctrine] db suffix nem baj ha configuralhato * [ci] db migration test specifikus phpunit config: DAMA-t ki kell kapcsolni * [tests] ki lehessen hagyni az adatbazis letrahozast * [ci] elso migteszt user load * [migrations] javitasok a felmerult hibak fenyeben

… szentek, unnepnaptar, updates táblák törlése #174 (#221)

* [composer] phpstan csomagok megkovetelese * [qa] alap phpstan config * [qa] apro javitasok, hogy a stan zold legyen * [github] workflow config a phpstan-hoz

* [orm] User Repository teszt es Clock bevezetese * [components] BreviarClient fetchCalendarAt argumentuma igazabol barmi interface (stan report) * [entity] null eset kezelese N/A statusszal * [orm] role migrator tesztelese * [qa] phpstan level 5 * [qa] tipusdefiniciok (level 6) * [qa] phpstan level 6 * [php83] konstans típusok * [typo] keruljon a vegere a deprecated * [typo] null set folosleges, contructor allitja be az erteket * [qa] AccessibilityHelper tesztek * [qa] ezt a kettot egyelore ignoreba tesszuk, a tobbit pedig baseline-ba

* [breviarclient] object alapu visszateresi ertek, hogy rendesen tipusos legyen * [breviarclient] teszt javitasa * [breviarclient] phpstan kizolditese

* [docker] gd kiterjesztes telepitese * [docker] curl csomag telepitese. composer gyorsitasa

#230

codecov-commenter · 2024-03-02T23:19:49Z

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 6.30%. Comparing base (8f58b47) to head (f866d63).
Report is 4 commits behind head on master.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff            @@
##           master    #232      +/-   ##
=========================================
- Coverage    6.39%   6.30%   -0.09%     
=========================================
  Files         164     164              
  Lines        7912    7912              
=========================================
- Hits          506     499       -7     
- Misses       7406    7413       +7

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

borazslo · 2024-03-02T23:47:03Z

Gondolom nem teljesen hülye az ötletem. Csak hát a getenv() nem látja az env változókat amit ha terminálba belépek akkor szépen látok. Ez van. Most alszom.

connorhu · 2024-03-03T00:03:17Z

.env

@@ -18,7 +18,7 @@
 # IMPORTANT: You MUST configure your server version, either here or in config/packages/doctrine.yaml
 #
 # DATABASE_URL="sqlite:///%kernel.project_dir%/var/data.db"
- DATABASE_URL="mysql://user:pw@mysql:3306/miserend?serverVersion=8.0.32&charset=utf8mb4"
+ DATABASE_URL="mysql://${MISEREND_MYSQL_USER}:${MISEREND_MYSQL_PASSWORD}@mysql:3306/${MISEREND_MYSQL_DATABASE}?serverVersion=8.0.32&charset=utf8mb4"


Suggested change

DATABASE_URL="mysql://${MISEREND_MYSQL_USER}:${MISEREND_MYSQL_PASSWORD}@mysql:3306/${MISEREND_MYSQL_DATABASE}?serverVersion=8.0.32&charset=utf8mb4"

# DATABASE_URL="mysql://app:[email protected]:3306/app?serverVersion=8.0.32&charset=utf8mb4"

# DATABASE_URL="mysql://app:[email protected]:3306/app?serverVersion=10.11.2-MariaDB&charset=utf8mb4"

DATABASE_URL="postgresql://app:[email protected]:5432/app?serverVersion=16&charset=utf8"

A .env fájl még mindig tabu olyan dolgoknak amik a te környezetedre jellemző. Ha nagyon be akarod égetni akkor inkább a .env.dev, de inkább ne, mert még mindig csak a te környezetedre jellemző. (nálam teljesen más van és a prod és a teszt is teljesen más)

Dehogy megmutassam mire való a .env: Pl itt be lehet állítani a renszerre jellemző időzónát:
APP_TIMEZONE=Europe/Budapest
Ez nem egy támogatott dolog a symfonyban, de például ilyen adat kerülhetne ide. Most attól tekintsünk el, hogy a miserend nem egy globális rendszer és nem változik az időzóna instance-onként.

connorhu · 2024-03-03T00:03:49Z

docker-compose.yml

-      MYSQL_USER: user
-      MYSQL_PASSWORD: pw
+      MYSQL_ROOT_PASSWORD: ${MISEREND_MYSQL_ROOT_PASSWORD}
+      MYSQL_DATABASE:  ${MISEREND_MYSQL_DATABASE}


Suggested change

MYSQL_DATABASE: ${MISEREND_MYSQL_DATABASE}

MYSQL_DATABASE: ${MISEREND_MYSQL_DATABASE}

connorhu · 2024-03-03T00:04:00Z

docker-compose.yml

-      MYSQL_PASSWORD: pw
+      MYSQL_ROOT_PASSWORD: ${MISEREND_MYSQL_ROOT_PASSWORD}
+      MYSQL_DATABASE:  ${MISEREND_MYSQL_DATABASE}
+      MYSQL_USER:  ${MISEREND_MYSQL_USER}


Suggested change

MYSQL_USER: ${MISEREND_MYSQL_USER}

MYSQL_USER: ${MISEREND_MYSQL_USER}

connorhu · 2024-03-03T00:04:08Z

docker-compose.yml

+      MYSQL_ROOT_PASSWORD: ${MISEREND_MYSQL_ROOT_PASSWORD}
+      MYSQL_DATABASE:  ${MISEREND_MYSQL_DATABASE}
+      MYSQL_USER:  ${MISEREND_MYSQL_USER}
+      MYSQL_PASSWORD:  ${MISEREND_MYSQL_PASSWORD}


Suggested change

MYSQL_PASSWORD: ${MISEREND_MYSQL_PASSWORD}

MYSQL_PASSWORD: ${MISEREND_MYSQL_PASSWORD}

connorhu · 2024-03-03T00:04:22Z

docker-compose.yml

@@ -81,14 +81,17 @@ services:
      - "${MISEREND_PORT:-8000}:8000"
    environment:
        MISEREND_WEBAPP_ENVIRONMENT: ${MISEREND_WEBAPP_ENVIRONMENT:-development}
+        MISEREND_MYSQL_DATABASE:  ${MISEREND_MYSQL_DATABASE}


Suggested change

MISEREND_MYSQL_DATABASE: ${MISEREND_MYSQL_DATABASE}

MISEREND_MYSQL_DATABASE: ${MISEREND_MYSQL_DATABASE}

connorhu · 2024-03-03T00:04:32Z

docker-compose.yml

@@ -81,14 +81,17 @@ services:
      - "${MISEREND_PORT:-8000}:8000"
    environment:
        MISEREND_WEBAPP_ENVIRONMENT: ${MISEREND_WEBAPP_ENVIRONMENT:-development}
+        MISEREND_MYSQL_DATABASE:  ${MISEREND_MYSQL_DATABASE}
+        MISEREND_MYSQL_USER:  ${MISEREND_MYSQL_USER}


Suggested change

MISEREND_MYSQL_USER: ${MISEREND_MYSQL_USER}

MISEREND_MYSQL_USER: ${MISEREND_MYSQL_USER}

connorhu · 2024-03-03T00:04:40Z

docker-compose.yml

@@ -81,14 +81,17 @@ services:
      - "${MISEREND_PORT:-8000}:8000"
    environment:
        MISEREND_WEBAPP_ENVIRONMENT: ${MISEREND_WEBAPP_ENVIRONMENT:-development}
+        MISEREND_MYSQL_DATABASE:  ${MISEREND_MYSQL_DATABASE}
+        MISEREND_MYSQL_USER:  ${MISEREND_MYSQL_USER}
+        MISEREND_MYSQL_PASSWORD:  ${MISEREND_MYSQL_PASSWORD}


Suggested change

MISEREND_MYSQL_PASSWORD: ${MISEREND_MYSQL_PASSWORD}

MISEREND_MYSQL_PASSWORD: ${MISEREND_MYSQL_PASSWORD}

connorhu · 2024-03-03T00:05:03Z

docker-compose.yml

    networks:
      inner:
        aliases:
          - miserend
    depends_on:
      - mysql
      - mailcatcher
-      - build
+      - miserend-build


Suggested change

- miserend-build

- build

igyekezzünk ne keverni a dolgokat

#233 megjavítja

connorhu · 2024-03-03T07:07:22Z

Kis olvasnivaló a témában:
https://docs.docker.com/compose/use-secrets/
https://symfony.com/doc/current/configuration/env_var_processors.html
https://symfony.com/doc/current/configuration/secrets.html

Kicsit hosszabb olvasnivaló:
https://cheatsheetseries.owasp.org/cheatsheets/Secrets_Management_Cheat_Sheet.html

tldr:
Kétféle megközelítés van: vagy a docker kezeli vagy a symfony. Mivel több helyen is meg kell jelennie, ezért a docker kezeli, szóval azt a megoldást kell használni. env_var-ban van processzor ami fájlból hozza az értéket. Nem a dsn-t kell matatni, hanem a megfelelő beállítás értéket kell megadni:

# config/packages/doctrine.yaml
doctrine:
    dbal:
        password: '%env(file:MISEREND_MYSQL_PASSWORD_FILE)%'
        # ...
    # ...

connorhu and others added 30 commits January 10, 2024 17:47

fájlok átszervezése és routing alapok

7c5db9f

symfony integráció

dd1baf8

új asset build container

a2abe6d

localhost-ra fallbackolunk ha nincs hostname

03d7a08

psr-4 átnevezés

5264b59

code style fix

4f90ebf

php kodok javitasa

c71f9dc

bootstrap 5 portolás

2f9557c

stat portolasa #177

beb1011

remark modul portolasa (#177)

07af4be

route alias nem elegáns így, de legacy-ba elmegy talán

templom entity es controller alapok

9b6cbfe

meg tobb templom mezo, view es repository

42db897

cleanup

4732cd9

kell az expression az expr miatt

f0806b8

templom view nézet sablonjai

5e0d678

user, holder es tovabbi church entitasok

35f9994

alap legacy service-k es container logika

e06ca7d

slug alapok

fef8281

slug alapok

59e08fd

lett

d381d7b

legyen futtathato

807e6eb

adatbazis mindig toltson be, container mindig keszuljon el, ha van se…

e898644

…t container akkor adja hozza a containert

legacy url atiranyitas alapok. itt pl a templom id eseten iranyit a t…

7d0d5f7

…emplomra

- token kezeles

c6e239d

- user login kezeles - routerben lehessen methodus nevre is hivatkozni - user getter / setter halmok - es meg jo sok aprosag

has-szel vizsgálunk

335ad84

typo

e7398a3

templom 404 esetek kezelese

a091bec

magikus metodusok

31a3006

az oreg zold emberket letiltjuk. ugyesen vagyuk, nem csinalunk butasagot

8de04e2

fix

d6c195b

connorhu and others added 23 commits February 12, 2024 13:13

[security] login oldal kinézetének javítása, bootstrap form alapok be…

f36c0e9

…töltése és symfony alap form téma beállítása (#203)

[users] Regisztráció összerakása (#204)

97286e5

* [user] regisztráció tesztelések és hibajavítások * [github] node setup es asset build

[static] statikus oldal routeok felvetele es javitas (#208)

1db2a0d

[security] lemaradt LoginType #203 (#210)

445306e

* [security] lemaradt login type * [security] minimal teszt, hogy latszodjon ha valami nem mukodik

[style] footer alul legyen ha keves a tartalom, de tolja le ha sok (#211

044c671

)

[docker/qa] frissites: php 83 (#212)

a1be0c0

* [github] php 8.3 * [docker] php 8.3 * [docker] mlocati php ext installer-et hasznaljuk, egyszerubb

[docker] node 20-ra frissites (#213)

7e7dd22

[cleanup] coverage report nem fut ha ez itt van, szoval kuka (#216)

c771ca5

[ci] codecov integráció (#215)

25701c1

* [ci] codecov integráció * [ci] coverage badge

[docker] build futtassa le a migraciot inditaskor (#220)

df5eb81

modulok, oldalkeret, osm_tags, events, igenaptar, lnaptar, nevnaptar,…

b12d238

… szentek, unnepnaptar, updates táblák törlése #174 (#221)

[qa] statikus analizálás phpstannal (#222)

201bedf

* [composer] phpstan csomagok megkovetelese * [qa] alap phpstan config * [qa] apro javitasok, hogy a stan zold legyen * [github] workflow config a phpstan-hoz

[composer] precommit hivja az analysis-t is (#225)

c234fbb

[api clients] breviar kliens tömb visszatérési érték kivezetése (#226)

8f58b47

* [breviarclient] object alapu visszateresi ertek, hogy rendesen tipusos legyen * [breviarclient] teszt javitasa * [breviarclient] phpstan kizolditese

[docker] hianyzo php kiterjesztesek telepitese (#228)

67346db

* [docker] gd kiterjesztes telepitese * [docker] curl csomag telepitese. composer gyorsitasa

[docker] buildernek is kell inner network (#229)

c71846f

[config] default DATABASE_URL

29dc0a2

#230

[docker] build csak mysql utan, app csak build utan (#231)

40fb2a7

[config] MYSQL configurations to env

fac2ebc

borazslo marked this pull request as draft March 2, 2024 23:19

[config] getenv() -> $_ENV

f866d63

connorhu suggested changes Mar 3, 2024

View reviewed changes

borazslo force-pushed the master branch from 2769674 to 3627aa5 Compare January 2, 2025 16:40

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[config] MYSQL configurations to env #232

[config] MYSQL configurations to env #232

borazslo commented Mar 2, 2024

codecov-commenter commented Mar 2, 2024 •

edited

Loading

borazslo commented Mar 2, 2024

connorhu Mar 3, 2024

connorhu Mar 3, 2024

connorhu Mar 3, 2024

connorhu Mar 3, 2024

connorhu Mar 3, 2024

connorhu Mar 3, 2024

connorhu Mar 3, 2024

connorhu Mar 3, 2024

connorhu Mar 3, 2024

connorhu Mar 3, 2024

connorhu Mar 3, 2024

connorhu commented Mar 3, 2024

- DATABASE_URL="mysql://${MISEREND_MYSQL_USER}:${MISEREND_MYSQL_PASSWORD}@mysql:3306/${MISEREND_MYSQL_DATABASE}?serverVersion=8.0.32&charset=utf8mb4"
+# DATABASE_URL="mysql://app:[email protected]:3306/app?serverVersion=8.0.32&charset=utf8mb4"
+# DATABASE_URL="mysql://app:[email protected]:3306/app?serverVersion=10.11.2-MariaDB&charset=utf8mb4"
+DATABASE_URL="postgresql://app:[email protected]:5432/app?serverVersion=16&charset=utf8"

	MYSQL_DATABASE: ${MISEREND_MYSQL_DATABASE}
	MYSQL_DATABASE: ${MISEREND_MYSQL_DATABASE}

	MYSQL_USER: ${MISEREND_MYSQL_USER}
	MYSQL_USER: ${MISEREND_MYSQL_USER}

	MYSQL_PASSWORD: ${MISEREND_MYSQL_PASSWORD}
	MYSQL_PASSWORD: ${MISEREND_MYSQL_PASSWORD}

	MISEREND_MYSQL_DATABASE: ${MISEREND_MYSQL_DATABASE}
	MISEREND_MYSQL_DATABASE: ${MISEREND_MYSQL_DATABASE}

	MISEREND_MYSQL_USER: ${MISEREND_MYSQL_USER}
	MISEREND_MYSQL_USER: ${MISEREND_MYSQL_USER}

	MISEREND_MYSQL_PASSWORD: ${MISEREND_MYSQL_PASSWORD}
	MISEREND_MYSQL_PASSWORD: ${MISEREND_MYSQL_PASSWORD}

[config] MYSQL configurations to env #232

Are you sure you want to change the base?

[config] MYSQL configurations to env #232

Conversation

borazslo commented Mar 2, 2024

codecov-commenter commented Mar 2, 2024 • edited Loading

Codecov Report

borazslo commented Mar 2, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

connorhu commented Mar 3, 2024

codecov-commenter commented Mar 2, 2024 •

edited

Loading