.editorconfig

@@ -1,5 +1,5 @@
 # EditorConfig configuration for aurweb
-# https://editorconfig.org
+# https://EditorConfig.org
 
 # Top-most EditorConfig file
 root = true

.gitlab-ci.yml

@@ -19,9 +19,9 @@ variables:
 lint:
   stage: .pre
   before_script:
-    - pacman -Sy --noconfirm --noprogressbar
+    - pacman -Sy --noconfirm --noprogressbar --cachedir .pkg-cache
       archlinux-keyring
-    - pacman -Syu --noconfirm --noprogressbar
+    - pacman -Syu --noconfirm --noprogressbar --cachedir .pkg-cache
       git python python-pre-commit
   script:
     - export XDG_CACHE_HOME=.pre-commit
@@ -60,7 +60,7 @@ test:
         path: coverage.xml
 
 .init_tf: &init_tf
-  - pacman -Syu --needed --noconfirm terraform
+  - pacman -Syu --needed --noconfirm --cachedir .pkg-cache terraform
   - export TF_VAR_name="aurweb-${CI_COMMIT_REF_SLUG}"
   - TF_ADDRESS="${CI_API_V4_URL}/projects/${TF_STATE_PROJECT}/terraform/state/${CI_COMMIT_REF_SLUG}"
   - cd ci/tf
@@ -97,7 +97,7 @@ provision_review:
     - deploy_review
   script:
     - *init_tf
-    - pacman -Syu --noconfirm --needed ansible git openssh jq
+    - pacman -Syu --noconfirm --needed --cachedir .pkg-cache ansible git openssh jq
       # Get ssh key from terraform state file
     - mkdir -p ~/.ssh
     - chmod 700 ~/.ssh

aurweb/asgi.py

@@ -14,12 +14,6 @@ from fastapi import FastAPI, HTTPException, Request, Response
 from fastapi.responses import RedirectResponse
 from fastapi.staticfiles import StaticFiles
 from jinja2 import TemplateNotFound
-from opentelemetry import trace
-from opentelemetry.exporter.otlp.proto.http.trace_exporter import OTLPSpanExporter
-from opentelemetry.instrumentation.fastapi import FastAPIInstrumentor
-from opentelemetry.sdk.resources import Resource
-from opentelemetry.sdk.trace import TracerProvider
-from opentelemetry.sdk.trace.export import BatchSpanProcessor
 from sqlalchemy import and_
 from starlette.exceptions import HTTPException as StarletteHTTPException
 from starlette.middleware.authentication import AuthenticationMiddleware
@@ -28,6 +22,7 @@ from starlette.middleware.sessions import SessionMiddleware
 import aurweb.captcha  # noqa: F401
 import aurweb.config
 import aurweb.filters  # noqa: F401
+import aurweb.pkgbase.util as pkgbaseutil
 from aurweb import aur_logging, prometheus, util
 from aurweb.aur_redis import redis_connection
 from aurweb.auth import BasicAuthBackend
@@ -59,17 +54,6 @@ instrumentator().add(prometheus.http_requests_total())
 instrumentator().instrument(app)
 
 
-# Instrument FastAPI for tracing
-FastAPIInstrumentor.instrument_app(app)
-
-resource = Resource(attributes={"service.name": "aurweb"})
-otlp_endpoint = aurweb.config.get("tracing", "otlp_endpoint")
-otlp_exporter = OTLPSpanExporter(endpoint=otlp_endpoint)
-span_processor = BatchSpanProcessor(otlp_exporter)
-trace.set_tracer_provider(TracerProvider(resource=resource))
-trace.get_tracer_provider().add_span_processor(span_processor)
-
-
 async def app_startup():
     # https://stackoverflow.com/questions/67054759/about-the-maximum-recursion-error-in-fastapi
     # Test failures have been observed by internal starlette code when
@@ -231,13 +215,7 @@ async def http_exception_handler(request: Request, exc: HTTPException) -> Respon
         if matches and len(tokens) == 2:
             try:
                 pkgbase = get_pkg_or_base(matches.group(1))
-                context["pkgbase"] = pkgbase
+                context = pkgbaseutil.make_context(request, pkgbase)
-                context["git_clone_uri_anon"] = aurweb.config.get(
-                    "options", "git_clone_uri_anon"
-                )
-                context["git_clone_uri_priv"] = aurweb.config.get(
-                    "options", "git_clone_uri_priv"
-                )
             except HTTPException:
                 pass
 

aurweb/aur_redis.py

@@ -1,5 +1,4 @@
 import fakeredis
-from opentelemetry.instrumentation.redis import RedisInstrumentor
 from redis import ConnectionPool, Redis
 
 import aurweb.config
@@ -8,8 +7,6 @@ from aurweb import aur_logging
 logger = aur_logging.get_logger(__name__)
 pool = None
 
-RedisInstrumentor().instrument()
-
 
 class FakeConnectionPool:
     """A fake ConnectionPool class which holds an internal reference

aurweb/captcha.py

@@ -3,7 +3,6 @@
 import hashlib
 
 from jinja2 import pass_context
-from sqlalchemy import func
 
 from aurweb.db import query
 from aurweb.models import User
@@ -12,8 +11,7 @@ from aurweb.templates import register_filter
 
 def get_captcha_salts():
     """Produce salts based on the current user count."""
-    count = query(func.count(User.ID)).scalar()
+    count = query(User).count()
-
     salts = []
     for i in range(0, 6):
         salts.append(f"aurweb-{count - i}")

aurweb/db.py

@@ -298,12 +298,9 @@ def get_engine(dbname: str = None, echo: bool = False):
             connect_args["check_same_thread"] = False
 
         kwargs = {"echo": echo, "connect_args": connect_args}
-        from opentelemetry.instrumentation.sqlalchemy import SQLAlchemyInstrumentor
         from sqlalchemy import create_engine
 
-        engine = create_engine(get_sqlalchemy_url(), **kwargs)
+        _engines[dbname] = create_engine(get_sqlalchemy_url(), **kwargs)
-        SQLAlchemyInstrumentor().instrument(engine=engine)
-        _engines[dbname] = engine
 
         if is_sqlite:  # pragma: no cover
             setup_sqlite(_engines.get(dbname))

aurweb/pkgbase/util.py

@@ -2,7 +2,6 @@ from typing import Any
 
 from fastapi import Request
 from sqlalchemy import and_
-from sqlalchemy.orm import joinedload
 
 from aurweb import config, db, defaults, l10n, time, util
 from aurweb.models import PackageBase, User
@@ -27,8 +26,6 @@ def make_context(
     if not context:
         context = _make_context(request, pkgbase.Name)
 
-    is_authenticated = request.user.is_authenticated()
-
     # Per page and offset.
     offset, per_page = util.sanitize_params(
         request.query_params.get("O", defaults.O),
@@ -41,15 +38,12 @@ def make_context(
     context["pkgbase"] = pkgbase
     context["comaintainers"] = [
         c.User
-        for c in pkgbase.comaintainers.options(joinedload(PackageComaintainer.User))
+        for c in pkgbase.comaintainers.order_by(
-        .order_by(PackageComaintainer.Priority.asc())
+            PackageComaintainer.Priority.asc()
-        .all()
+        ).all()
     ]
-    if is_authenticated:
+    context["unflaggers"] = context["comaintainers"].copy()
-        context["unflaggers"] = context["comaintainers"].copy()
+    context["unflaggers"].extend([pkgbase.Maintainer, pkgbase.Flagger])
-        context["unflaggers"].extend([pkgbase.Maintainer, pkgbase.Flagger])
-    else:
-        context["unflaggers"] = []
 
     context["packages_count"] = pkgbase.packages.count()
     context["keywords"] = pkgbase.keywords
@@ -66,28 +60,17 @@ def make_context(
     ).order_by(PackageComment.CommentTS.desc())
 
     context["is_maintainer"] = bool(request.user == pkgbase.Maintainer)
-    if is_authenticated:
+    context["notified"] = request.user.notified(pkgbase)
-        context["notified"] = request.user.notified(pkgbase)
-    else:
-        context["notified"] = False
 
     context["out_of_date"] = bool(pkgbase.OutOfDateTS)
 
-    if is_authenticated:
+    context["voted"] = request.user.package_votes.filter(
-        context["voted"] = db.query(
+        PackageVote.PackageBaseID == pkgbase.ID
-            request.user.package_votes.filter(
+    ).scalar()
-                PackageVote.PackageBaseID == pkgbase.ID
-            ).exists()
-        ).scalar()
-    else:
-        context["voted"] = False
 
-    if is_authenticated:
+    context["requests"] = pkgbase.requests.filter(
-        context["requests"] = pkgbase.requests.filter(
+        and_(PackageRequest.Status == PENDING_ID, PackageRequest.ClosedTS.is_(None))
-            and_(PackageRequest.Status == PENDING_ID, PackageRequest.ClosedTS.is_(None))
+    ).count()
-        ).count()
-    else:
-        context["requests"] = []
 
     context["popularity"] = popularity(pkgbase, time.utcnow())
 

aurweb/routers/packages.py

@@ -190,17 +190,6 @@ async def package(
     if not all_deps:
         deps = deps.limit(max_listing)
     context["dependencies"] = deps.all()
-    # Existing dependencies to avoid multiple lookups
-    context["dependencies_names_from_aur"] = [
-        item.Name
-        for item in db.query(models.Package)
-        .filter(
-            models.Package.Name.in_(
-                pkg.package_dependencies.with_entities(models.PackageDependency.DepName)
-            )
-        )
-        .all()
-    ]
 
     # Package requirements (other packages depend on this one).
     reqs = pkgutil.pkg_required(pkg.Name, [p.RelName for p in rels_data.get("p", [])])

conf/config.defaults

@@ -177,6 +177,3 @@ expiry_time_search = 600
 expiry_time_statistics = 300
 ; number of seconds after a cache entry for rss queries expires, default is 5 minutes
 expiry_time_rss = 300
-
-[tracing]
-otlp_endpoint = http://localhost:4318/v1/traces

conf/config.dev

@@ -73,6 +73,3 @@ pkgnames-repo = pkgnames.git
 
 [aurblup]
 db-path = YOUR_AUR_ROOT/aurblup/
-
-[tracing]
-otlp_endpoint = http://tempo:4318/v1/traces

docker-compose.aur-dev.yml

@@ -1,4 +1,5 @@
----
+version: "3.8"
+
 services:
   ca:
     volumes:

docker-compose.override.yml

@@ -1,10 +1,16 @@
----
+version: "3.8"
+
 services:
   ca:
     volumes:
       - ./data:/data
       - step:/root/.step
 
+  mariadb_init:
+    depends_on:
+      mariadb:
+        condition: service_healthy
+
   git:
     volumes:
       - git_data:/aurweb/aur.git
@@ -15,6 +21,9 @@ services:
       - git_data:/aurweb/aur.git
       - ./data:/data
       - smartgit_run:/var/run/smartgit
+    depends_on:
+      mariadb:
+        condition: service_healthy
 
   fastapi:
     volumes:

docker-compose.yml

@@ -1,4 +1,3 @@
----
 #
 # Docker service definitions for the aurweb project.
 #
@@ -17,6 +16,8 @@
 #
 # Copyright (C) 2021 aurweb Development
 # All Rights Reserved.
+version: "3.8"
+
 services:
   aurweb-image:
     build: .
@@ -48,7 +49,7 @@ services:
     image: aurweb:latest
     init: true
     entrypoint: /docker/mariadb-entrypoint.sh
-    command: /usr/bin/mariadbd-safe --datadir=/var/lib/mysql
+    command: /usr/bin/mysqld_safe --datadir=/var/lib/mysql
     ports:
       # This will expose mariadbd on 127.0.0.1:13306 in the host.
       # Ex: `mysql -uaur -paur -h 127.0.0.1 -P 13306 aurweb`
@@ -80,7 +81,7 @@ services:
     environment:
       - MARIADB_PRIVILEGED=1
     entrypoint: /docker/mariadb-entrypoint.sh
-    command: /usr/bin/mariadbd-safe --datadir=/var/lib/mysql
+    command: /usr/bin/mysqld_safe --datadir=/var/lib/mysql
     ports:
       # This will expose mariadbd on 127.0.0.1:13307 in the host.
       # Ex: `mysql -uaur -paur -h 127.0.0.1 -P 13306 aurweb`
@@ -106,10 +107,8 @@ services:
       test: "bash /docker/health/sshd.sh"
       interval: 3s
     depends_on:
-      mariadb:
-        condition: service_healthy
       mariadb_init:
-        condition: service_completed_successfully
+        condition: service_started
     volumes:
       - mariadb_run:/var/run/mysqld
 
@@ -123,9 +122,6 @@ services:
     healthcheck:
       test: "bash /docker/health/smartgit.sh"
       interval: 3s
-    depends_on:
-      mariadb:
-        condition: service_healthy
 
   cgit-fastapi:
     image: aurweb:latest
@@ -156,10 +152,8 @@ services:
     entrypoint: /docker/cron-entrypoint.sh
     command: /docker/scripts/run-cron.sh
     depends_on:
-      mariadb:
-        condition: service_healthy
       mariadb_init:
-        condition: service_completed_successfully
+        condition: service_started
     volumes:
       - ./aurweb:/aurweb/aurweb
       - mariadb_run:/var/run/mysqld
@@ -188,12 +182,6 @@ services:
         condition: service_healthy
       cron:
         condition: service_started
-      mariadb:
-        condition: service_healthy
-      mariadb_init:
-        condition: service_completed_successfully
-      tempo:
-        condition: service_healthy
     volumes:
       - archives:/var/lib/aurweb/archives
       - mariadb_run:/var/run/mysqld
@@ -293,56 +281,6 @@ services:
       - ./test:/aurweb/test
       - ./templates:/aurweb/templates
 
-  grafana:
-    # TODO: check if we need init: true
-    image: grafana/grafana:11.1.3
-    environment:
-      - GF_AUTH_ANONYMOUS_ENABLED=true
-      - GF_AUTH_ANONYMOUS_ORG_ROLE=Admin
-      - GF_AUTH_DISABLE_LOGIN_FORM=true
-      - GF_LOG_LEVEL=warn
-    # check if depends ar ecorrect, does stopping or restarting a child exit grafana?
-    depends_on:
-      prometheus:
-        condition: service_healthy
-      tempo:
-        condition: service_healthy
-    ports:
-      - "127.0.0.1:3000:3000"
-    volumes:
-      - ./docker/config/grafana/datasources:/etc/grafana/provisioning/datasources
-
-  prometheus:
-    image: prom/prometheus:latest
-    command:
-      - --config.file=/etc/prometheus/prometheus.yml
-      - --web.enable-remote-write-receiver
-      - --web.listen-address=prometheus:9090
-    healthcheck:
-      # TODO: check if there is a status route
-      test: "sh /docker/health/prometheus.sh"
-      interval: 3s
-    ports:
-      - "127.0.0.1:9090:9090"
-    volumes:
-      - ./docker/config/prometheus.yml:/etc/prometheus/prometheus.yml
-      - ./docker/health/prometheus.sh:/docker/health/prometheus.sh
-
-  tempo:
-    image: grafana/tempo:2.5.0
-    command:
-      - -config.file=/etc/tempo/config.yml
-    healthcheck:
-      # TODO: check if there is a status route
-      test: "sh /docker/health/tempo.sh"
-      interval: 3s
-    ports:
-      - "127.0.0.1:3200:3200"
-      - "127.0.0.1:4318:4318"
-    volumes:
-      - ./docker/config/tempo.yml:/etc/tempo/config.yml
-      - ./docker/health/tempo.sh:/docker/health/tempo.sh
-
 volumes:
   mariadb_test_run: {}
   mariadb_run: {} # Share /var/run/mysqld/mysqld.sock

docker/README.md

@@ -47,7 +47,7 @@ Luckily such data can be generated.
 docker compose exec fastapi /bin/bash
 pacman -S words fortune-mod
 ./schema/gendummydata.py dummy.sql
-mariadb aurweb < dummy.sql
+mysql aurweb < dummy.sql
 ```
 
 The generation script may prompt you to install other Arch packages before it

docker/config/grafana/datasources/datasource.yml

@@ -1,42 +0,0 @@
----
-apiVersion: 1
-
-deleteDatasources:
-  - name: Prometheus
-  - name: Tempo
-
-datasources:
-  - name: Prometheus
-    type: prometheus
-    uid: prometheus
-    access: proxy
-    url: http://prometheus:9090
-    orgId: 1
-    editable: false
-    jsonData:
-      timeInterval: 1m
-  - name: Tempo
-    type: tempo
-    uid: tempo
-    access: proxy
-    url: http://tempo:3200
-    orgId: 1
-    editable: false
-    jsonData:
-      tracesToMetrics:
-        datasourceUid: 'prometheus'
-        spanStartTimeShift: '1h'
-        spanEndTimeShift: '-1h'
-      serviceMap:
-        datasourceUid: 'prometheus'
-      nodeGraph:
-        enabled: true
-      search:
-        hide: false
-      traceQuery:
-        timeShiftEnabled: true
-        spanStartTimeShift: '1h'
-        spanEndTimeShift: '-1h'
-      spanBar:
-        type: 'Tag'
-        tag: 'http.path'

docker/config/prometheus.yml

@@ -1,15 +0,0 @@
----
-global:
-  scrape_interval: 60s
-
-scrape_configs:
-  - job_name: tempo
-    static_configs:
-      - targets: ['tempo:3200']
-        labels:
-          instance: tempo
-  - job_name: aurweb
-    static_configs:
-      - targets: ['fastapi:8000']
-        labels:
-          instance: aurweb

docker/config/tempo.yml

@@ -1,54 +0,0 @@
----
-stream_over_http_enabled: true
-server:
-  http_listen_address: tempo
-  http_listen_port: 3200
-  log_level: info
-
-query_frontend:
-  search:
-    duration_slo: 5s
-    throughput_bytes_slo: 1.073741824e+09
-  trace_by_id:
-    duration_slo: 5s
-
-distributor:
-  receivers:
-    otlp:
-      protocols:
-        http:
-          endpoint: tempo:4318
-  log_received_spans:
-    enabled: false
-  metric_received_spans:
-    enabled: false
-
-ingester:
-  max_block_duration: 5m
-
-compactor:
-  compaction:
-    block_retention: 1h
-
-metrics_generator:
-  registry:
-    external_labels:
-      source: tempo
-  storage:
-    path: /tmp/tempo/generator/wal
-    remote_write:
-      - url: http://prometheus:9090/api/v1/write
-        send_exemplars: true
-  traces_storage:
-    path: /tmp/tempo/generator/traces
-
-storage:
-  trace:
-    backend: local
-    wal:
-      path: /tmp/tempo/wal
-    local:
-      path: /tmp/tempo/blocks
-
-overrides:
-  metrics_generator_processors: [service-graphs, span-metrics, local-blocks]

docker/health/mariadb.sh

@@ -1,2 +1,2 @@
 #!/bin/bash
-exec mariadb-admin ping --silent
+exec mysqladmin ping --silent

docker/health/prometheus.sh

@@ -1,2 +0,0 @@
-#!/bin/sh
-exec wget -q http://prometheus:9090/status -O /dev/null

docker/health/tempo.sh

@@ -1,2 +0,0 @@
-#!/bin/sh
-exec wget -q http://tempo:3200/status -O /dev/null

docker/mariadb-entrypoint.sh

@@ -6,8 +6,8 @@ MYSQL_DATA=/var/lib/mysql
 mariadb-install-db --user=mysql --basedir=/usr --datadir=$MYSQL_DATA
 
 # Start it up.
-mariadbd-safe --datadir=$MYSQL_DATA --skip-networking &
+mysqld_safe --datadir=$MYSQL_DATA --skip-networking &
-while ! mariadb-admin ping 2>/dev/null; do
+while ! mysqladmin ping 2>/dev/null; do
     sleep 1s
 done
 
@@ -15,17 +15,17 @@ done
 DATABASE="aurweb" # Persistent database for fastapi.
 
 echo "Taking care of primary database '${DATABASE}'..."
-mariadb -u root -e "CREATE USER IF NOT EXISTS 'aur'@'localhost' IDENTIFIED BY 'aur';"
+mysql -u root -e "CREATE USER IF NOT EXISTS 'aur'@'localhost' IDENTIFIED BY 'aur';"
-mariadb -u root -e "CREATE USER IF NOT EXISTS 'aur'@'%' IDENTIFIED BY 'aur';"
+mysql -u root -e "CREATE USER IF NOT EXISTS 'aur'@'%' IDENTIFIED BY 'aur';"
-mariadb -u root -e "CREATE DATABASE IF NOT EXISTS $DATABASE;"
+mysql -u root -e "CREATE DATABASE IF NOT EXISTS $DATABASE;"
 
-mariadb -u root -e "CREATE USER IF NOT EXISTS 'aur'@'%' IDENTIFIED BY 'aur';"
+mysql -u root -e "CREATE USER IF NOT EXISTS 'aur'@'%' IDENTIFIED BY 'aur';"
-mariadb -u root -e "GRANT ALL ON aurweb.* TO 'aur'@'localhost';"
+mysql -u root -e "GRANT ALL ON aurweb.* TO 'aur'@'localhost';"
-mariadb -u root -e "GRANT ALL ON aurweb.* TO 'aur'@'%';"
+mysql -u root -e "GRANT ALL ON aurweb.* TO 'aur'@'%';"
 
-mariadb -u root -e "CREATE USER IF NOT EXISTS 'root'@'%' IDENTIFIED BY 'aur';"
+mysql -u root -e "CREATE USER IF NOT EXISTS 'root'@'%' IDENTIFIED BY 'aur';"
-mariadb -u root -e "GRANT ALL ON *.* TO 'root'@'%' WITH GRANT OPTION;"
+mysql -u root -e "GRANT ALL ON *.* TO 'root'@'%' WITH GRANT OPTION;"
 
-mariadb-admin -uroot shutdown
+mysqladmin -uroot shutdown
 
 exec "$@"

docker/scripts/install-deps.sh

@@ -13,7 +13,7 @@ pacman -Sy --noconfirm --noprogressbar archlinux-keyring
 
 # Install other OS dependencies.
 pacman -Syu --noconfirm --noprogressbar \
-    git gpgme nginx redis openssh \
+    --cachedir .pkg-cache git gpgme nginx redis openssh \
     mariadb mariadb-libs cgit-aurweb uwsgi uwsgi-plugin-cgi \
     python-pip pyalpm python-srcinfo curl libeatmydata cronie \
     python-poetry python-poetry-core step-cli step-ca asciidoc \

pyproject.toml

@@ -16,7 +16,7 @@ combine_as_imports = true
 #
 [tool.poetry]
 name = "aurweb"
-version = "v6.2.16"
+version = "v6.2.12"
 license = "GPL-2.0-only"
 description = "Source code for the Arch User Repository's website"
 homepage = "https://aur.archlinux.org"
@@ -52,13 +52,13 @@ build-backend = "poetry.masonry.api"
 "Request Mailing List" = "https://lists.archlinux.org/listinfo/aur-requests"
 
 [tool.poetry.dependencies]
-python = ">=3.10,<3.14"
+python = ">=3.9,<3.13"
 
 # poetry-dynamic-versioning is used to produce tool.poetry.version
 # based on git tags.
 
 # General
-aiofiles = "^24.0.0"
+aiofiles = "^23.2.1"
 asgiref = "^3.8.1"
 bcrypt = "^4.1.2"
 bleach = "^6.1.0"
@@ -69,42 +69,34 @@ httpx = "^0.27.0"
 itsdangerous = "^2.1.2"
 lxml = "^5.2.1"
 orjson = "^3.10.0"
-pygit2 = "^1.17.0"
+protobuf = "^5.26.1"
-python-multipart = "0.0.19"
+pygit2 = "^1.14.1"
+python-multipart = "^0.0.9"
 redis = "^5.0.3"
 requests = "^2.31.0"
 paginate = "^0.5.6"
 
 # SQL
 alembic = "^1.13.1"
-mysqlclient = "^2.2.3"
+mysqlclient = "^2.2.4"
 Authlib = "^1.3.0"
 Jinja2 = "^3.1.3"
 Markdown = "^3.6"
 Werkzeug = "^3.0.2"
 SQLAlchemy = "^1.4.52"
-greenlet = "3.1.1" # Explicitly add greenlet (dependency of SQLAlchemy) for python 3.13 support
 
 # ASGI
-uvicorn = "^0.30.0"
+uvicorn = "^0.29.0"
-gunicorn = "^22.0.0"
+gunicorn = "^21.2.0"
-Hypercorn = "^0.17.0"
+Hypercorn = "^0.16.0"
+prometheus-fastapi-instrumentator = "^7.0.0"
 pytest-xdist = "^3.5.0"
 filelock = "^3.13.3"
 posix-ipc = "^1.1.1"
 pyalpm = "^0.10.6"
-fastapi = "^0.112.0"
+fastapi = "^0.110.0"
 srcinfo = "^0.1.2"
-tomlkit = "^0.13.0"
+tomlkit = "^0.12.0"
-
-# Tracing
-prometheus-fastapi-instrumentator = "^7.0.0"
-opentelemetry-api = "^1.26.0"
-opentelemetry-sdk = "^1.26.0"
-opentelemetry-exporter-otlp-proto-http = "^1.26.0"
-opentelemetry-instrumentation-fastapi = "^0.47b0"
-opentelemetry-instrumentation-redis = "^0.47b0"
-opentelemetry-instrumentation-sqlalchemy = "^0.47b0"
 
 [tool.poetry.dev-dependencies]
 coverage = "^7.4.4"
@@ -112,7 +104,7 @@ pytest = "^8.1.1"
 pytest-asyncio = "^0.23.0"
 pytest-cov = "^5.0.0"
 pytest-tap = "^3.4"
-watchfiles = "^1.0.4"
+watchfiles = "^0.21.0"
 
 [tool.poetry.scripts]
 aurweb-git-auth = "aurweb.git.auth:main"

templates/partials/packages/actions.html

@@ -24,73 +24,71 @@
                     {{ "Search wiki" | tr }}
                 </a>
             </li>
-            {% if request.user.is_authenticated() %}
+            {% if not out_of_date %}
-				{% if not out_of_date %}
-					<li>
-						<a href="/pkgbase/{{ pkgbase.Name }}/flag">
-							{{ "Flag package out-of-date" | tr }}
-						</a>
-					</li>
-				{% else %}
-					<li>
-						<span class="flagged">
-							<a href="/pkgbase/{{ pkgbase.Name }}/flag-comment">
-								{{
-								"Flagged out-of-date (%s)"
-								| tr | format(date_strftime(pkgbase.OutOfDateTS, "%Y-%m-%d"))
-								}}
-							</a>
-						</span>
-					</li>
-					{% if request.user.has_credential(creds.PKGBASE_UNFLAG, approved=unflaggers) %}
-					<li>
-						<form action="/pkgbase/{{ pkgbase.Name }}/unflag" method="post">
-							<input class="button text-button"
-								   type="submit"
-								   name="do_UnFlag"
-								   value="{{ 'Unflag package' | tr }}"
-							/>
-						</form>
-					</li>
-					{% endif %}
-				{% endif %}
                 <li>
-                    {% if not voted %}
+                    <a href="/pkgbase/{{ pkgbase.Name }}/flag">
-                    <form action="/pkgbase/{{ pkgbase.Name }}/vote" method="post">
+                        {{ "Flag package out-of-date" | tr }}
+                    </a>
+                </li>
+            {% else %}
+                <li>
+                    <span class="flagged">
+                        <a href="/pkgbase/{{ pkgbase.Name }}/flag-comment">
+                            {{
+                            "Flagged out-of-date (%s)"
+                            | tr | format(date_strftime(pkgbase.OutOfDateTS, "%Y-%m-%d"))
+                            }}
+                        </a>
+                    </span>
+                </li>
+                {% if request.user.has_credential(creds.PKGBASE_UNFLAG, approved=unflaggers) %}
+                <li>
+                    <form action="/pkgbase/{{ pkgbase.Name }}/unflag" method="post">
+                        <input class="button text-button"
+                               type="submit"
+                               name="do_UnFlag"
+                               value="{{ 'Unflag package' | tr }}"
+                        />
+                    </form>
+                </li>
+                {% endif %}
+            {% endif %}
+            <li>
+                {% if not voted %}
+                <form action="/pkgbase/{{ pkgbase.Name }}/vote" method="post">
+                    <input type="submit"
+                           class="button text-button"
+                           name="do_Vote"
+                           value="{{ 'Vote for this package' | tr }}" />
+                </form>
+                {% else %}
+                <form action="/pkgbase/{{ pkgbase.Name }}/unvote" method="post">
+                    <input type="submit"
+                           class="button text-button"
+                           name="do_UnVote"
+                           value="{{ 'Remove vote' | tr }}" />
+                </form>
+                {% endif %}
+            </li>
+            <li>
+                {% if notified %}
+                    <form action="/pkgbase/{{ pkgbase.Name }}/unnotify" method="post">
                         <input type="submit"
                                class="button text-button"
-                               name="do_Vote"
+                               name="do_UnNotify"
-                               value="{{ 'Vote for this package' | tr }}" />
+                               value="{{ 'Disable notifications' | tr }}"
+                        />
                     </form>
-                    {% else %}
+                {% else %}
-                    <form action="/pkgbase/{{ pkgbase.Name }}/unvote" method="post">
+                    <form action="/pkgbase/{{ pkgbase.Name }}/notify" method="post">
-                       <input type="submit"
+                        <input type="submit"
-                              class="button text-button"
+                               class="button text-button"
-                              name="do_UnVote"
+                               name="do_Notify"
-                              value="{{ 'Remove vote' | tr }}" />
+                               value="{{ 'Enable notifications' | tr }}"
+                        />
                     </form>
-                    {% endif %}
+                {% endif %}
-                </li>
+            </li>
-                <li>
-                    {% if notified %}
-                        <form action="/pkgbase/{{ pkgbase.Name }}/unnotify" method="post">
-                            <input type="submit"
-                                   class="button text-button"
-                                   name="do_UnNotify"
-                                   value="{{ 'Disable notifications' | tr }}"
-                            />
-                        </form>
-                    {% else %}
-                        <form action="/pkgbase/{{ pkgbase.Name }}/notify" method="post">
-                            <input type="submit"
-                                   class="button text-button"
-                                   name="do_Notify"
-                                   value="{{ 'Enable notifications' | tr }}"
-                            />
-                        </form>
-                    {% endif %}
-                </li>
-            {% endif %}
             {% if request.user.has_credential(creds.PKGBASE_EDIT_COMAINTAINERS, approved=[pkgbase.Maintainer]) %}
                 <li>
                     <a href="/pkgbase/{{ pkgbase.Name }}/comaintainers">
@@ -113,13 +111,11 @@
                     </li>
                 {% endif %}
             {% endif %}
-            {% if request.user.is_authenticated() %}
+            <li>
-                <li>
+                <a href="/pkgbase/{{ pkgbase.Name }}/request?{{ {'next': '/pkgbase/%s' | format(pkgbase.Name)} | urlencode }}">
-                    <a href="/pkgbase/{{ pkgbase.Name }}/request?{{ {'next': '/pkgbase/%s' | format(pkgbase.Name)} | urlencode }}">
+                    {{ "Submit Request" | tr }}
-                        {{ "Submit Request" | tr }}
+                </a>
-                    </a>
+            </li>
-                </li>
-            {% endif %}
             {% if request.user.has_credential(creds.PKGBASE_DELETE) %}
                 <li>
                     <a href="/pkgbase/{{ pkgbase.Name }}/delete?next=/packages">

templates/partials/packages/package_metadata.html

@@ -20,7 +20,7 @@
                     {% endif %}
                 {% else -%}
                     </a>
-                    {%- if dep.DepName in dependencies_names_from_aur -%}
+                    {%- if dep.is_aur_package() -%}
                         <sup><small>AUR</small></sup>
                     {% endif %}
                 {% endif %}

test/test_packages_routes.py

@@ -507,9 +507,7 @@ def test_package_requests_display(
     client: TestClient, user: User, package: Package, pkgreq: PackageRequest
 ):
     # Test that a single request displays "1 pending request".
-    cookies = {"AURSID": user.login(Request(), "testPassword")}
     with client as request:
-        request.cookies = cookies
         resp = request.get(package_endpoint(package))
     assert resp.status_code == int(HTTPStatus.OK)
 
@@ -532,7 +530,6 @@ def test_package_requests_display(
 
     # Test that a two requests display "2 pending requests".
     with client as request:
-        request.cookies = cookies
         resp = request.get(package_endpoint(package))
     assert resp.status_code == int(HTTPStatus.OK)