removes unused imports and variable definitions with ruff

web/flaskr/__init__.py

@@ -10,7 +10,7 @@
 # ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
 # FOR A PARTICULAR PURPOSE.
 
-from flask import Flask, render_template, request, session
+from flask import Flask, request, session
 import os
 import logging
 
@@ -79,7 +79,7 @@ def global_processor():
         from .models import db
 
         db.init_app(app)
-        migrate = Migrate(app, db, compare_type=True)
+        Migrate(app, db, compare_type=True)
 
     # ensure the instance folder exists
     os.makedirs(app.instance_path, exist_ok=True)

web/flaskr/forms.py

@@ -7,7 +7,6 @@
     SelectField,
     StringField,
     TextAreaField,
-    MultipleFileField,
     BooleanField,
     HiddenField,
     validators,

web/flaskr/models.py

@@ -12,14 +12,9 @@
 
 from flaskr.tasks import background_upload
 
-import importlib.util
-import sys
-import signal
-
 
 from flask_sqlalchemy import SQLAlchemy
 from sqlalchemy_utils import EncryptedType
-from os import fork
 
 from flask import current_app, url_for, render_template
 
@@ -63,7 +58,7 @@ def get_user_nc_credentials(username):
         )
         data = response.json()
         return data
-    except requests.exceptions.RequestException as e:
+    except requests.exceptions.RequestException:
         print("Cannot contact NC, returning None values")
         return {"nctoken": None, "nclocator": None, "nclogin": None}
 
@@ -259,7 +254,6 @@ def insertDocsNoDefault(self):
             "%s/%s" % (BIGBLUEBUTTON_ENDPOINT, insertAction),
             params=params,
         )
-        headers = {"Content-Type": "application/xml"}
         pr = request.prepare()
         bigbluebutton_secret = BIGBLUEBUTTON_SECRET
         s = "%s%s" % (
@@ -268,7 +262,7 @@ def insertDocsNoDefault(self):
         )
         params["checksum"] = hashlib.sha1(s.encode("utf-8")).hexdigest()
 
-        r = requests.post(
+        requests.post(
             f"{BIGBLUEBUTTON_ENDPOINT}/{insertAction}",
             headers={"Content-Type": "application/xml"},
             data=xml,
@@ -415,15 +409,14 @@ def create(self):
                 "%s/%s" % (BIGBLUEBUTTON_ENDPOINT, insertAction),
                 params=params,
             )
-            headers = {"Content-Type": "application/xml"}
             pr = request.prepare()
             bigbluebutton_secret = BIGBLUEBUTTON_SECRET
             s = "%s%s" % (
                 pr.url.replace("?", "").replace(BIGBLUEBUTTON_ENDPOINT + "/", ""),
                 bigbluebutton_secret,
             )
             params["checksum"] = hashlib.sha1(s.encode("utf-8")).hexdigest()
-            task = background_upload.delay(
+            background_upload.delay(
                 f"{BIGBLUEBUTTON_ENDPOINT}/{insertAction}", xml, params
             )
 

web/flaskr/routes.py

@@ -13,7 +13,7 @@
 from werkzeug.utils import secure_filename
 
 from webdav3.client import Client as webdavClient
-from webdav3.exceptions import MethodNotSupported, WebDavException
+from webdav3.exceptions import WebDavException
 
 from pathlib import Path
 
@@ -27,38 +27,34 @@
     render_template,
     send_file,
     make_response,
-    Response,
     request,
     redirect,
     flash,
     session,
     jsonify,
     url_for,
 )
-from xml.etree import ElementTree
 from flask_babel import lazy_gettext
 from flask_pyoidc import OIDCAuthentication
 from flask_pyoidc.provider_configuration import (
     ProviderConfiguration,
     ClientMetadata,
 )
 from flask_pyoidc.user_session import UserSession
-from sqlalchemy import or_, exc
+from sqlalchemy import exc
 import random
 import hashlib
 import os
 import re
 import smtplib
 import string
 import requests
-import shutil
 import secrets
 import uuid
 
 from datetime import datetime, date
 from email.mime.text import MIMEText
 from email.message import EmailMessage
-from urllib.parse import quote
 from netaddr import IPNetwork, IPAddress
 
 from flaskr.forms import (
@@ -230,7 +226,7 @@ def get_meetings_stats():
         stats_array = [row.split(",") for row in stats_array]
         participantCount = int(stats_array[current_app.config["STATS_INDEX"]][1])
         runningCount = int(stats_array[current_app.config["STATS_INDEX"]][2])
-    except Exception as e:
+    except Exception:
         return None
 
     result = {"participantCount": participantCount, "runningCount": runningCount}
@@ -241,7 +237,7 @@ def get_meetings_stats():
 @auth.token_auth(provider_name="default")
 def api_meetings():
     if auth.current_token_identity:
-        current_identity = auth.current_token_identity
+        pass
     else:
         return redirect("/")
     info = {
@@ -250,10 +246,9 @@ def api_meetings():
         "email": auth.current_token_identity["email"],
     }
     user = get_or_create_user(info)
-    fullname = user.fullname
-    stats = get_meetings_stats()
+    get_meetings_stats()
     if user is not None:
-        logged_in = True
+        pass
     return {
         "meetings": [
             {
@@ -272,7 +267,7 @@ def api_meetings():
 def insertDocuments(meeting_id):
     from flask import request
 
-    user = get_current_user()
+    get_current_user()
     meeting = Meeting.query.get(meeting_id)
     files_title = request.get_json()
     secret_key = current_app.config["SECRET_KEY"]
@@ -296,7 +291,6 @@ def insertDocuments(meeting_id):
         "%s/%s" % (current_app.config["BIGBLUEBUTTON_ENDPOINT"], "insertDocument"),
         params=params,
     )
-    headers = {"Content-Type": "application/xml"}
     pr = request.prepare()
     bigbluebutton_secret = current_app.config["BIGBLUEBUTTON_SECRET"]
     s = "%s%s" % (
@@ -306,7 +300,7 @@ def insertDocuments(meeting_id):
         bigbluebutton_secret,
     )
     params["checksum"] = hashlib.sha1(s.encode("utf-8")).hexdigest()
-    r = requests.post(
+    requests.post(
         f"{bbb_endpoint}/insertDocument",
         headers={"Content-Type": "application/xml"},
         data=xml,
@@ -546,7 +540,7 @@ def quick_mail_meeting():
         id=email
     )  # this user can probably be removed if we created adock function
     m = get_quick_meeting_from_user_and_random_string(user)
-    signinurl = _send_mail(m, email)
+    _send_mail(m, email)
     flash(
         lazy_gettext("Vous avez reçu un courriel pour vous connecter"), "success_login"
     )
@@ -851,7 +845,7 @@ def set_meeting_default_file(meeting_id):
     data = request.get_json()
 
     if user is not None:
-        logged_in = True
+        pass
     meeting = Meeting.query.get(meeting_id)
     if meeting.user_id == user.id:
         actual_default_file = meeting.default_file
@@ -929,7 +923,7 @@ def add_meeting_file_dropzone(title, meeting_id, is_default):
             meetingFile.is_default = False
 
         meetingFile.save()
-        secret_key = current_app.config["SECRET_KEY"]
+        current_app.config["SECRET_KEY"]
         meetingFile.update()
         # file has been associated AND uploaded to nextcloud, we can safely remove it from visio-agent tmp directory
         removeDropzoneFile(dropzonePath)
@@ -949,7 +943,7 @@ def add_meeting_file_dropzone(title, meeting_id, is_default):
 
 
 def add_meeting_file_URL(url, meeting_id, is_default):
-    user = get_current_user()
+    get_current_user()
     title = url.rsplit("/", 1)[-1]
 
     # test MAX_SIZE_UPLOAD for 20Mo
@@ -978,7 +972,7 @@ def add_meeting_file_URL(url, meeting_id, is_default):
     meetingFile.url = url
     meetingFile.is_default = is_default
 
-    getFile = requests.get(url)
+    requests.get(url)
 
     try:
         meetingFile.save()
@@ -1009,7 +1003,7 @@ def add_meeting_file_nextcloud(path, meeting_id, is_default):
     try:
         client = webdavClient(options)
         metadata = client.info(path)
-    except WebDavException as exception:
+    except WebDavException:
         user.disable_nextcloud()
         return jsonify(
             status=500,
@@ -1030,7 +1024,7 @@ def add_meeting_file_nextcloud(path, meeting_id, is_default):
     meetingFile.meeting_id = meeting_id
     meetingFile.nc_path = path
     meetingFile.is_default = is_default
-    secret_key = current_app.config["SECRET_KEY"]
+    current_app.config["SECRET_KEY"]
 
     try:
         meetingFile.save()
@@ -1050,7 +1044,7 @@ def add_meeting_file_nextcloud(path, meeting_id, is_default):
 
 
 def add_external_meeting_file_nextcloud(path, meeting_id):
-    user = get_current_user()
+    get_current_user()
 
     externalMeetingFile = MeetingFilesExternal()
 
@@ -1069,7 +1063,7 @@ def add_meeting_files(meeting_id):
     meeting = Meeting.query.get(meeting_id)
 
     if user is not None:
-        logged_in = True
+        pass
 
     data = request.get_json()
     is_default = False
@@ -1152,7 +1146,7 @@ def upload(user, meeting_id, file):
 def delete_meeting_file():
     user = get_current_user()
     if user is not None:
-        logged_in = True
+        pass
 
     data = request.get_json()
     meeting_file_id = data["id"]
@@ -1230,7 +1224,7 @@ def save_meeting():
     )
 
     if meeting.is_meeting_running():
-        end_meeting_form = EndMeetingForm()
+        EndMeetingForm()
         EndMeetingForm.meeting_id.data = meeting.id
         return render_template(
             "meeting/end.html",
@@ -1309,7 +1303,7 @@ def insertDoc(token):
     # xml now use
     xml = f"<?xml version='1.0' encoding='UTF-8'?> <modules>  <module name='presentation'><document url='{current_app.config['SERVER_FQDN']}/ncdownload/{m.id}/{m.download_hash}' filename='m.title' /> </module></modules>"
 
-    r = requests.post(
+    requests.post(
         f"{current_app.config['BIGBLUEBUTTON_ENDPOINT']}/insertDocument",
         data=xml,
         headers=headers,
@@ -1382,7 +1376,7 @@ def ncdownload(isexternal, mfid, mftoken):
             "local_path": tmpName,
         }
         client.download_sync(**kwargs)
-    except WebDavException as exception:
+    except WebDavException:
         meeting_file.meeting.user.disable_nextcloud()
         return jsonify(status=500, msg="La connexion avec Nextcloud semble rompue")
     # send the downloaded file to the BBB:
@@ -1539,7 +1533,7 @@ def join_mail_meeting():
         return redirect("/")
     fullname = form["fullname"].data
     meeting_fake_id = form["meeting_fake_id"].data
-    user_id = form["user_id"].data
+    form["user_id"].data
     expiration = form["expiration"].data
     h = form["h"].data
 
@@ -1632,7 +1626,7 @@ def delete_meeting():
                 flash(
                     "Nous n'avons pas pu supprimer les vidéos de cette "
                     + current_app.config["WORDINGS"]["meeting_label"]
-                    + " : {message}".format(code=return_code, message=message),
+                    + " : {message}".format(message=message),
                     "error",
                 )
             else:

web/flaskr/tasks.py

@@ -1,5 +1,4 @@
 import os
-import time
 import requests
 
 from celery import Celery
@@ -13,7 +12,7 @@
 
 @celery.task(name="background_upload")
 def background_upload(endpoint, xml, params):
-    r = requests.post(
+    requests.post(
         endpoint,
         headers={"Content-Type": "application/xml"},
         data=xml,

web/migrations/alembic_helpers.py

@@ -1,7 +1,7 @@
 # Code based on https://github.com/talkpython/data-driven-web-apps-with-flask
 
 from alembic import op
-from sqlalchemy import engine_from_config, inspect
+from sqlalchemy import engine_from_config
 
 from sqlalchemy import MetaData
 
@@ -36,7 +36,7 @@ def table_does_not_exist(table):
 def table_has_column(table, column):
     has_column = False
     print(table, flush=True)
-    if not table in schema:
+    if table not in schema:
         return
     for s_column in schema[table]:
         print("  %s" % s_column, flush=True)

web/migrations/versions/1094e771bd3f_create_meeting_files_table.py

@@ -7,7 +7,6 @@
 """
 from alembic import op
 import sqlalchemy as sa
-from sqlalchemy.dialects import postgresql
 
 import os
 import sys

web/migrations/versions/65acbe9b0646_add_user_nextcloud_connection_info.py

@@ -7,7 +7,6 @@
 """
 from alembic import op
 import sqlalchemy as sa
-from sqlalchemy.dialects import postgresql
 
 import os
 import sys

web/migrations/versions/7d80b9223a1e_guestpolicy_migration.py

@@ -7,7 +7,6 @@
 """
 from alembic import op
 import sqlalchemy as sa
-from sqlalchemy.dialects import postgresql
 
 import os
 import sys

web/migrations/versions/8fe077ecfb10_add_last_connection_utc_datetime.py

@@ -7,7 +7,6 @@
 """
 from alembic import op
 import sqlalchemy as sa
-from sqlalchemy.dialects import postgresql
 
 import os
 import sys