Skip to content

Packages upgrade - #2535

Merged
Didayolo merged 6 commits into
developfrom
packages_upgrades
Sep 15, 2026
Merged

Didayolo merged 6 commits into
developfrom
packages_upgrades

Conversation

@ObadaS

@ObadaS ObadaS commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

A brief description of the purpose of the changes contained in this PR.

  • Updated djangorestframework in pyproject and some other packages via uv lock --upgrade
  • Updated Gunicorn and Uvicorn to latest
  • Updated boto3 to latest (and removed s3transfer and botocore from pyproject, they are dependencies of boto3 and make upgrading boto3 more complicated for no reason)
  • Upgraded DOMPurify, marked and katex packages
  • Upgrading django-oauth-toolkit requires a migration. The migration is run automatically in the docker-compose.yml file, but make sure to backup your database before.

I think this fixes all the points from #2502

Checklist

  • Code review by me
  • Hand tested by me
  • I'm proud of my work
  • Code review by reviewer
  • Hand tested by reviewer
  • CircleCi tests are passing
  • Ready to merge

@Didayolo

Didayolo commented Sep 15, 2026

Copy link
Copy Markdown
Member

Trying to test this but it looks like I'm polluted by the recent #2499, #2533:

Running migrations:
  Applying competitions.0063_remove_competition_allow_robot_submissions...Traceback (most recent call last):
  File "/.venv/lib/python3.13/site-packages/django/db/backends/utils.py", line 103, in _execute
    return self.cursor.execute(sql)
           ~~~~~~~~~~~~~~~~~~~^^^^^
psycopg2.errors.UndefinedColumn: column "allow_robot_submissions" of relation "competitions_competition" does not exist


The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/app/./manage.py", line 15, in <module>
    execute_from_command_line(sys.argv)
    ~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^
  File "/.venv/lib/python3.13/site-packages/django/core/management/__init__.py", line 442, in execute_from_command_line
    utility.execute()
    ~~~~~~~~~~~~~~~^^
  File "/.venv/lib/python3.13/site-packages/django/core/management/__init__.py", line 436, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^
  File "/.venv/lib/python3.13/site-packages/django/core/management/base.py", line 420, in run_from_argv
    self.execute(*args, **cmd_options)
    ~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^
  File "/.venv/lib/python3.13/site-packages/django/core/management/base.py", line 464, in execute
    output = self.handle(*args, **options)
  File "/.venv/lib/python3.13/site-packages/django/core/management/base.py", line 111, in wrapper
    res = handle_func(*args, **kwargs)
  File "/.venv/lib/python3.13/site-packages/django/core/management/commands/migrate.py", line 353, in handle
    post_migrate_state = executor.migrate(
        targets,
    ...<3 lines>...
        fake_initial=fake_initial,
    )
  File "/.venv/lib/python3.13/site-packages/django/db/migrations/executor.py", line 135, in migrate
    state = self._migrate_all_forwards(
        state, plan, full_plan, fake=fake, fake_initial=fake_initial
    )
  File "/.venv/lib/python3.13/site-packages/django/db/migrations/executor.py", line 167, in _migrate_all_forwards
    state = self.apply_migration(
        state, migration, fake=fake, fake_initial=fake_initial
    )
  File "/.venv/lib/python3.13/site-packages/django/db/migrations/executor.py", line 255, in apply_migration
    state = migration.apply(state, schema_editor)
  File "/.venv/lib/python3.13/site-packages/django/db/migrations/migration.py", line 132, in apply
    operation.database_forwards(
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~^
        self.app_label, schema_editor, old_state, project_state
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    )
    ^
  File "/.venv/lib/python3.13/site-packages/django/db/migrations/operations/fields.py", line 174, in database_forwards
    schema_editor.remove_field(
    ~~~~~~~~~~~~~~~~~~~~~~~~~~^
        from_model, from_model._meta.get_field(self.name)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    )
    ^
  File "/.venv/lib/python3.13/site-packages/django/db/backends/base/schema.py", line 829, in remove_field
    self.execute(sql)
    ~~~~~~~~~~~~^^^^^
  File "/.venv/lib/python3.13/site-packages/django/db/backends/postgresql/schema.py", line 48, in execute
    return super().execute(sql, None)
           ~~~~~~~~~~~~~~~^^^^^^^^^^^
  File "/.venv/lib/python3.13/site-packages/django/db/backends/base/schema.py", line 204, in execute
    cursor.execute(sql, params)
    ~~~~~~~~~~~~~~^^^^^^^^^^^^^
  File "/.venv/lib/python3.13/site-packages/django/db/backends/utils.py", line 79, in execute
    return self._execute_with_wrappers(
           ~~~~~~~~~~~~~~~~~~~~~~~~~~~^
        sql, params, many=False, executor=self._execute
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    )
    ^
  File "/.venv/lib/python3.13/site-packages/django/db/backends/utils.py", line 92, in _execute_with_wrappers
    return executor(sql, params, many, context)
  File "/.venv/lib/python3.13/site-packages/django/db/backends/utils.py", line 100, in _execute
    with self.db.wrap_database_errors:
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/.venv/lib/python3.13/site-packages/django/db/utils.py", line 91, in __exit__
    raise dj_exc_value.with_traceback(traceback) from exc_value
  File "/.venv/lib/python3.13/site-packages/django/db/backends/utils.py", line 103, in _execute
    return self.cursor.execute(sql)
           ~~~~~~~~~~~~~~~~~~~^^^^^
django.db.utils.ProgrammingError: column "allow_robot_submissions" of relation "competitions_competition" does not exist

Putting it here to try to debug.

@Didayolo

Copy link
Copy Markdown
Member

I think it is only my local instance that has problem. Going to fix it.

@ObadaS

ObadaS commented Sep 15, 2026

Copy link
Copy Markdown
Collaborator Author

This does look like missing migrations or something, not related to this PR

@Didayolo

Didayolo commented Sep 15, 2026

Copy link
Copy Markdown
Member

This does look like missing migrations or something, not related to this PR

Yeah it's because I fixed migrations with #2533 but my local history is unhealthy. I'll just re-install my instance from scratch.

Testing the develop upgrade is important though.

@Didayolo
Didayolo marked this pull request as ready for review September 15, 2026 14:23
@Didayolo
Didayolo merged commit bc930d6 into develop Sep 15, 2026
1 check passed
@Didayolo
Didayolo deleted the packages_upgrades branch September 15, 2026 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Stored XSS in competition pages / terms / phase descriptions

2 participants