chore: more precommit hooks

.gitignore

@@ -34,4 +34,4 @@ web-webinaire.env
 web-local.env
 *.swp
 
-messages.mo
+messages.mo

.pre-commit-config.yaml

@@ -1,10 +1,27 @@
 repos:
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v4.4.0
+    hooks:
+      - id: fix-byte-order-marker
+      - id: trailing-whitespace
+      - id: end-of-file-fixer
+      - id: check-toml
+  - repo: https://github.com/asottile/reorder_python_imports
+    rev: v3.10.0
+    hooks:
+      - id: reorder-python-imports
+        args: ["--application-directories", "canaille"]
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: 'v0.0.276'
+    rev: 'v0.0.284'
     hooks:
       - id: ruff
         args: [--fix, --exit-non-zero-on-fix]
+  - repo: https://github.com/asottile/pyupgrade
+    rev: v3.10.1
+    hooks:
+      - id: pyupgrade
+        args: ["--py38-plus"]
   - repo: https://github.com/psf/black
-    rev: 22.12.0
+    rev: 23.7.0
     hooks:
     - id: black

Makefile

@@ -5,6 +5,7 @@
 install-dev:
 	# Install dependencies for local development usage
 	poetry env use 3.8 ; poetry install --with dev ; poetry shell
+	pre-commit install
 
 export-base-dependencies:
 	# Update requirements file for web service

bigbluebutton/create_bbb.sh

@@ -189,7 +189,7 @@ subjectAltName = @alt_names
 DNS.1 = ${HOSTNAME}
 EOF
 
-    openssl req -nodes -newkey rsa:2048 -keyout ${HOSTNAME}.key -out ${HOSTNAME}.csr -subj "/C=CA/ST=BBB/L=BBB/O=BBB/OU=BBB/CN=${HOSTNAME}" -addext "subjectAltName = DNS:${HOSTNAME}" 
+    openssl req -nodes -newkey rsa:2048 -keyout ${HOSTNAME}.key -out ${HOSTNAME}.csr -subj "/C=CA/ST=BBB/L=BBB/O=BBB/OU=BBB/CN=${HOSTNAME}" -addext "subjectAltName = DNS:${HOSTNAME}"
     openssl x509 -req -in ${HOSTNAME}.csr -CA bbb-dev-ca.crt -CAkey bbb-dev-ca.key -CAcreateserial -out ${HOSTNAME}.crt -days 825 -sha256 -passin file:bbb-dev-ca.pass -extfile ${HOSTNAME}.ext
 
     cd $HOME/$NAME/

docker-compose.preprod.yml

@@ -13,4 +13,3 @@ services:
       - FLASK_DEBUG=1
     env_file:
       - web.env
-

docker-compose.yml

@@ -1,7 +1,7 @@
 version: '3.7'
 
   # These containers are the one needed to make the app work
-  # Other container are available in docker-compose.override.yml if needed  
+  # Other container are available in docker-compose.override.yml if needed
   # see documentation [link] for more info
 
 services:

documentation/meetingFiles.md

@@ -72,7 +72,7 @@ Il se déroule alors la chose suivante :
 2. si le salon doit être créé, il est créé ( eh ouais ), il se passe alors 2 choses au niveau des fichiers associés
   - Le fichier noté comme étant celui par défaut est DIRECTEMENT associé au salon, via un fichier XML inséré dans l'appel API 'create' ( doc de référence : https://docs.bigbluebutton.org/development/api#pre-upload-slides )
   - Les autres fichiers sont poussés dans une job queue via celery ( qui communique avec B3desk via redis ). Une requête au BBB va ensuite être lancée sur l'endpoint insertDocument ( doc de référence : https://docs.bigbluebutton.org/development/api#insertdocument )
- 
+
  Le résultat obtenu est une association des fichiers fonctionnelle, un salon qui se lance rapidement, et des fichiers 'secondaires' qui s'uploadent ensuite en background sans bloquer le déroulé du cours. Des notifications dans le salon apparaissent à chaque ajout de fichier.
 
 **Troubleshooting : Si la création du salon part en timeout, il y a probablement un problème de communication entre B3desk et Celery et Redis, bien vérifier les variables d'environnement correspondantes**

keycloak.env

@@ -16,5 +16,3 @@ SERVICE_TITLE=Webinaire
 SERVICE_TAGLINE=Le service de webinaire pour les agents de l’État
 SERVICE_CONTACT_TITLE=Nous contacter
 SERVICE_CONTACT=Vous pouvez prendre contact avec l'équipe de la direction du numérique pour l’éducation en charge du projet Services Numériques Partagés en nous écrivant à l'adresse : apps [at] education.gouv.fr
-
-

keycloak/dsfr_configuration/domain.xml

@@ -502,12 +502,12 @@
             </subsystem>
         </profile>
         <!--
-          ~ 
+          ~
           ~            Clustering authentication server setup.
-          ~ 
+          ~
           ~            You must configure a remote shared external database like PostgreSQL or MySql if you want this to be
           ~            able to work on multiple machines.
-          ~         
+          ~
           -->
         <profile name="auth-server-clustered">
             <subsystem xmlns="urn:jboss:domain:logging:8.0">
@@ -1018,10 +1018,10 @@
             </subsystem>
         </profile>
         <!--
-          ~ 
+          ~
           ~            This is a profile for the built-in Underto Loadbalancer
           ~            It should be removed in production systems and replaced with a better software or hardware based one
-          ~          
+          ~
           -->
         <profile name="load-balancer">
             <subsystem xmlns="urn:jboss:domain:logging:8.0">
@@ -1088,13 +1088,13 @@
         </profile>
     </profiles>
     <!--
-      ~ 
+      ~
       ~          Named interfaces that can be referenced elsewhere in the configuration. The configuration
       ~          for how to associate these logical names with an actual network interface can either
       ~          be specified here or can be declared on a per-host basis in the equivalent element in host.xml.
-      ~ 
+      ~
       ~          These default configurations require the binding specification to be done in host.xml.
-      ~     
+      ~
       -->
     <interfaces>
         <interface name="management"/>

keycloak/dsfr_configuration/standalone-ha.xml

@@ -672,4 +672,4 @@
             <remote-destination host="localhost" port="25"/>
         </outbound-socket-binding>
     </socket-binding-group>
-</server>
+</server>

keycloak/dsfr_configuration/standalone.xml

@@ -611,4 +611,4 @@
             <remote-destination host="localhost" port="25"/>
         </outbound-socket-binding>
     </socket-binding-group>
-</server>
+</server>

keycloak/themes/dsfr/account/account.ftl

@@ -28,7 +28,7 @@
 
         <div class="form-group ${messagesPerField.printIfExists('email','has-error')}">
             <div class="col-sm-2 col-md-2">
-            <label for="email" class="control-label">${msg("email")}</label> 
+            <label for="email" class="control-label">${msg("email")}</label>
             </div>
 
             <div class="col-sm-10 col-md-10">
@@ -38,7 +38,7 @@
 
         <div class="form-group ${messagesPerField.printIfExists('firstName','has-error')}">
             <div class="col-sm-2 col-md-2">
-                <label for="firstName" class="control-label">${msg("firstName")}</label> 
+                <label for="firstName" class="control-label">${msg("firstName")}</label>
             </div>
 
             <div class="col-sm-10 col-md-10">
@@ -48,7 +48,7 @@
 
         <div class="form-group ${messagesPerField.printIfExists('lastName','has-error')}">
             <div class="col-sm-2 col-md-2">
-                <label for="lastName" class="control-label">${msg("lastName")}</label> 
+                <label for="lastName" class="control-label">${msg("lastName")}</label>
             </div>
 
             <div class="col-sm-10 col-md-10">
@@ -65,4 +65,3 @@
         </div>
     </form>
 </@layout.mainLayout>
-

keycloak/themes/dsfr/account/resources/css/account.css

@@ -274,4 +274,4 @@ hr + .form-horizontal {
     font-size: 16px;
     padding: 10px;
     margin: 50px 0;
-}
+}

keycloak/themes/dsfr/admin/resources/css/styles.css

@@ -477,4 +477,3 @@ table.credential-data-table td.key {
     text-align: right;
     font-weight: bold;
 }
-