Merge branch 'master' into preprod

Author: agallou

.env.dist

@@ -68,8 +68,6 @@ SMTP_PORT=1025
 SMTP_TLS=0
 SMTP_USERNAME=""
 SMTP_PASSWORD=""
-MAILER_FORCE_RECIPIENTS=""
-MAILER_BCC=""
 
 QR_CODE_SALT=""
 # https://symfony.com/doc/current/reference/configuration/framework.html#ide

.gitattributes

@@ -62,7 +62,7 @@ jobs:
           composer-options: "--no-scripts"
 
       - name: PHPStan
-        run: make phpstan
+        run: php ./bin/phpstan
 
   functional:
     name: "Functional tests"
@@ -139,4 +139,4 @@ jobs:
           composer-options: "--no-scripts"
 
       - name: Rector
-        run: make rector
+        run: php ./bin/rector

.github/workflows/ci.yml

@@ -26,6 +26,7 @@ node_modules
 
 /htdocs/assets
 /htdocs/templates/site/css/styles.css
+/htdocs/templates/site/images/
 js_dist
 # uploads
 /htdocs/uploads

.gitignore

@@ -17,16 +17,16 @@
     ->registerCustomFixers(new PhpCsFixerCustomFixers\Fixers())
     ->setRiskyAllowed(true)
     ->setRules([
-        '@PER-CS2.0' => true,
-        '@PHP74Migration:risky' => true,
+        '@PER-CS3x0' => true,
+        '@PHP7x4Migration:risky' => true,
         'no_alias_functions' => true,
         'native_function_casing' => true,
         'phpdoc_no_access' => true,
         'self_accessor' => true,
         'no_unused_imports' => true,
         'method_argument_space' => false,
         'statement_indentation' => false,
-        MultilinePromotedPropertiesFixer::name() => [
+        'multiline_promoted_properties' => [
             'keep_blank_lines' => true,
             'minimum_number_of_parameters' => 2,
         ],

.php-cs-fixer.php

@@ -0,0 +1 @@
+*   @agallou @stakovicz @Mopolo

.styleci.yml

@@ -3,9 +3,14 @@ default: help
 
 # Variables
 CURRENT_UID ?= $(shell id -u)
-DOCKER_UP_OPTIONS ?=
+DOCKER_UP_OPTIONS ?= --detach
 DOCKER_COMPOSE_BIN ?= docker compose
 
+# Exécutables
+DOCKER_COMP = CURRENT_UID=$(CURRENT_UID) $(DOCKER_COMPOSE_BIN)
+PHP_CONT    = $(DOCKER_COMP) exec apachephp
+PHP         = $(PHP_CONT) php
+
 # Colors
 COLOR_RESET = \033[0m
 COLOR_TARGET = \033[32m
@@ -31,7 +36,7 @@ help:
 	} \
 	{ lastLine = $$0 }' $(MAKEFILE_LIST)
 
-.PHONY: install docker-up docker-stop docker-down test hooks vendors db-seed db-migrations reset-db init console phpstan
+.PHONY: install docker-up docker-stop docker-down test hooks vendors db-seed db-migrations reset-db init console phpstan assets
 
 ##@ Setup
 
@@ -47,76 +52,79 @@ init: htdocs/uploads
 
 ### Démarrer les containers
 docker-up: .env var/logs/.docker-build data compose.override.yml
-	CURRENT_UID=$(CURRENT_UID) $(DOCKER_COMPOSE_BIN) up $(DOCKER_UP_OPTIONS)
+	$(DOCKER_COMP) up $(DOCKER_UP_OPTIONS)
 
 ### Stopper les containers
 docker-stop:
-	CURRENT_UID=$(CURRENT_UID) $(DOCKER_COMPOSE_BIN) stop
+	$(DOCKER_COMP) stop
 
 ### Supprimer les containers
 docker-down:
-	CURRENT_UID=$(CURRENT_UID) $(DOCKER_COMPOSE_BIN) down
+	$(DOCKER_COMP) down
 
 ### Démarrer un bash dans le container PHP
 console:
-	CURRENT_UID=$(CURRENT_UID) $(DOCKER_COMPOSE_BIN) exec -u localUser -it apachephp bash
+	$(DOCKER_COMP) exec -u localUser -it apachephp bash
+
+### Voir les logs docker compose
+logs:
+	$(DOCKER_COMP) logs -f --tail 150
 
 ##@ Quality
 
-### (Dans Docker) Tests unitaires
+### Tests unitaires
 test:
-	./bin/phpunit --testsuite unit
-	./bin/php-cs-fixer fix --dry-run -vv
+	$(PHP_CONT) ./bin/phpunit --testsuite unit
+	$(PHP_CONT) ./bin/php-cs-fixer fix --dry-run -vv
 
-### (Dans Docker) Tests d'intégration
+### Tests d'intégration
 test-integration:
-	./bin/phpunit --testsuite integration
+	$(PHP_CONT) ./bin/phpunit --testsuite integration
 
-### (Dans Docker) Tests fonctionnels
+### Tests fonctionnels
 behat:
-	./bin/behat
+	$(PHP_CONT) ./bin/behat
 
-### (Dans Docker) PHP CS Fixer (dry run)
+### PHP CS Fixer (dry run)
 cs-lint:
-	./bin/php-cs-fixer fix --dry-run -vv
+	$(PHP_CONT) ./bin/php-cs-fixer fix --dry-run -vv
 
-### (Dans Docker) PHP CS Fixer (fix)
+### PHP CS Fixer (fix)
 cs-fix:
-	./bin/php-cs-fixer fix -vv
+	$(PHP_CONT) ./bin/php-cs-fixer fix -vv
 
-### (Dans Docker) Rector (dry run)
+### Rector (dry run)
 rector: var/cache/dev/AppKernelDevDebugContainer.xml
-	./bin/rector --dry-run
+	$(PHP_CONT) ./bin/rector --dry-run
 
-### (Dans Docker) Rector (fix)
+### Rector (fix)
 rector-fix: var/cache/dev/AppKernelDevDebugContainer.xml
-	./bin/rector
+	$(PHP_CONT) ./bin/rector
 
 ### Tests fonctionnels
 test-functional: data config htdocs/uploads tmp
-	CURRENT_UID=$(CURRENT_UID) $(DOCKER_COMPOSE_BIN) stop dbtest apachephptest mailcatcher
-	CURRENT_UID=$(CURRENT_UID) $(DOCKER_COMPOSE_BIN) up -d dbtest apachephptest mailcatcher
+	$(DOCKER_COMP) stop dbtest apachephptest mailcatcher
+	$(DOCKER_COMP) up -d dbtest apachephptest mailcatcher
 	make clean-test-deprecated-log
-	CURRENT_UID=$(CURRENT_UID) $(DOCKER_COMPOSE_BIN) run --no-deps --rm -u localUser apachephp ./bin/behat
+	$(DOCKER_COMP) run --no-deps --rm -u localUser apachephptest ./bin/behat
 	make var/logs/test.deprecations_grouped.log
-	CURRENT_UID=$(CURRENT_UID) $(DOCKER_COMPOSE_BIN) stop dbtest apachephptest mailcatcher
+	$(DOCKER_COMP) stop dbtest apachephptest mailcatcher
 
 ### Tests d'intégration avec start/stop des images docker
 test-integration-ci:
-	CURRENT_UID=$(CURRENT_UID) $(DOCKER_COMPOSE_BIN) stop dbtest apachephptest
-	CURRENT_UID=$(CURRENT_UID) $(DOCKER_COMPOSE_BIN) up -d dbtest apachephptest
-	CURRENT_UID=$(CURRENT_UID) $(DOCKER_COMPOSE_BIN) run --no-deps --rm -u localUser apachephp make vendor
-	CURRENT_UID=$(CURRENT_UID) $(DOCKER_COMPOSE_BIN) run --no-deps --rm -u localUser apachephp ./bin/phpunit --testsuite integration
-	CURRENT_UID=$(CURRENT_UID) $(DOCKER_COMPOSE_BIN) stop dbtest apachephptest
+	$(DOCKER_COMP) stop dbtest apachephptest
+	$(DOCKER_COMP) up -d dbtest apachephptest
+	$(DOCKER_COMP) run --no-deps --rm -u localUser apachephptest make vendor
+	$(DOCKER_COMP) run --no-deps --rm -u localUser apachephptest ./bin/phpunit --testsuite integration
+	$(DOCKER_COMP) stop dbtest apachephptest
 
-### (Dans Docker) Analyse PHPStan
+### Analyse PHPStan
 phpstan:
-	./bin/phpstan --memory-limit=-1
+	$(PHP_CONT) ./bin/phpstan --memory-limit=-1
 
 ##@ Frontend
 
 ### Compiler les assets pour la production
-.PHONY: assets
 assets:
 	./node_modules/.bin/webpack -p
 
@@ -162,12 +170,12 @@ node_modules:
 
 init-db:
 	make reset-db
-	CURRENT_UID=$(CURRENT_UID) $(DOCKER_COMPOSE_BIN) run --rm -u localUser apachephp make db-migrations
-	CURRENT_UID=$(CURRENT_UID) $(DOCKER_COMPOSE_BIN) run --rm -u localUser apachephp make db-seed
+	$(DOCKER_COMP) run --rm -u localUser apachephp make db-migrations
+	$(DOCKER_COMP) run --rm -u localUser apachephp make db-seed
 
 config:
-	CURRENT_UID=$(CURRENT_UID) $(DOCKER_COMPOSE_BIN) run --no-deps --rm -u localUser apachephp make vendors
-	CURRENT_UID=$(CURRENT_UID) $(DOCKER_COMPOSE_BIN) run --no-deps --rm -u localUser apachephp make assets
+	$(DOCKER_COMP) run --no-deps --rm -u localUser apachephp make vendors
+	$(DOCKER_COMP) run --no-deps --rm -u localUser apachephp make assets
 
 data:
 	mkdir data

CODEOWNERS

@@ -2,6 +2,15 @@
 
 Si vous découvrez un problème de sécurité dans ce projet, **merci de ne pas ouvrir un ticket public**.
 
-À la place, contactez-nous à `outils[at]afup[dot]org` avec les détails de la vulnérabilité et des étapes pour la reproduire.
+À la place, utilisez le [système intégré à GitHub](https://github.com/afup/web/security/advisories/new) pour créer un [security advisory](https://github.com/afup/web/security/advisories).
+
+Ce système nous permettra de facilement discuter des détails de la vulnérabilité et des étapes pour la reproduire en
+privé tout en gérant un fork privé pour la correction.
+
+---
 
 Si vous êtes adhérent à l'association, vous pouvez également nous contacter via le Slack des membres.
+
+---
+
+Nous avons également un email : `outils[at]afup[dot]org`