Add missing migration

AUTHORS

@@ -1,9 +0,0 @@
-Thomas Sileo <t@a4.io>
-Kevin Wallace <doof@doof.net>
-Miguel Jacq <mig@mig5.net>
-Alexey Shpakovsky <alexey@shpakovsky.ru>
-Josh Washburne <josh@jodh.us>
-Sam <samr1.dev@pm.me>
-Ash McAllan <acegiak@gmail.com>
-Cassio Zen <cassio@hey.com>
-Cocoa <momijizukamori@gmail.com>

Makefile

@@ -12,32 +12,32 @@ config:
 
 .PHONY: update
 update:
-	-docker run --rm --volume `pwd`/data:/app/data --volume `pwd`/app/static:/app/app/static microblogpub/microblogpub inv update --no-update-deps
+	-docker run --volume `pwd`/data:/app/data --volume `pwd`/app/static:/app/app/static microblogpub/microblogpub inv update --no-update-deps
 
 .PHONY: prune-old-data
 prune-old-data:
-	-docker run --rm --volume `pwd`/data:/app/data --volume `pwd`/app/static:/app/app/static microblogpub/microblogpub inv prune-old-data
+	-docker run --volume `pwd`/data:/app/data --volume `pwd`/app/static:/app/app/static microblogpub/microblogpub inv prune-old-data
 
 .PHONY: webfinger
 webfinger:
-	-docker run --rm --volume `pwd`/data:/app/data --volume `pwd`/app/static:/app/app/static microblogpub/microblogpub inv webfinger $(account)
+	-docker run --volume `pwd`/data:/app/data --volume `pwd`/app/static:/app/app/static microblogpub/microblogpub inv webfinger $(account)
 
 .PHONY: move-to
 move-to:
-	-docker run --rm --volume `pwd`/data:/app/data --volume `pwd`/app/static:/app/app/static microblogpub/microblogpub inv move-to $(account)
+	-docker run --volume `pwd`/data:/app/data --volume `pwd`/app/static:/app/app/static microblogpub/microblogpub inv move-to $(account)
 
 .PHONY: self-destruct
 self-destruct:
-	-docker run --rm --volume `pwd`/data:/app/data --volume `pwd`/app/static:/app/app/static microblogpub/microblogpub inv self-destruct
+	-docker run --volume `pwd`/data:/app/data --volume `pwd`/app/static:/app/app/static microblogpub/microblogpub inv self-destruct
 
 .PHONY: reset-password
 reset-password:
-	-docker run --rm -it --volume `pwd`/data:/app/data --volume `pwd`/app/static:/app/app/static microblogpub/microblogpub inv reset-password
+	-docker run --volume `pwd`/data:/app/data --volume `pwd`/app/static:/app/app/static microblogpub/microblogpub inv reset-password
 
 .PHONY: check-config
 check-config:
-	-docker run --rm --volume `pwd`/data:/app/data --volume `pwd`/app/static:/app/app/static microblogpub/microblogpub inv check-config
+	-docker run --volume `pwd`/data:/app/data --volume `pwd`/app/static:/app/app/static microblogpub/microblogpub inv check-config
 
 .PHONY: compile-scss
 compile-scss:
-	-docker run --rm --volume `pwd`/data:/app/data --volume `pwd`/app/static:/app/app/static microblogpub/microblogpub inv compile-scss
+	-docker run --volume `pwd`/data:/app/data --volume `pwd`/app/static:/app/app/static microblogpub/microblogpub inv compile-scss

README.md

@@ -22,7 +22,7 @@ There are still some rough edges, but the server is mostly functional.
     - Author notes in Markdown, with code highlighting support
     - Dedicated section for articles/blog posts (enabled when the first article is posted)
  - Lightweight
-    - Uses SQLite, and Python 3.10+
+    - Uses SQLite, and no external dependencies except Python 3.10+
     - Can be deployed on small VPS
  - Privacy-aware
     - EXIF metadata (like GPS location) are stripped before storage
@@ -58,7 +58,7 @@ All the development takes place on [sourcehut](https://sr.ht/~tsileo/microblog.p
  - [Issue tracker](https://todo.sr.ht/~tsileo/microblog.pub)
  - [Mailing list](https://sr.ht/~tsileo/microblog.pub/lists)
 
-Contributions are welcomed, check out the [contributing section of the documentation](https://docs.microblog.pub/developer_guide.html#contributing) for more details.
+Contributions are welcomed, check out the [documentation](https://docs.microblog.pub) for more details.
 
 
 ## License

alembic/versions/2022_09_21_0708-c3027d0e18dc_add_support_for_quote_url.py

@@ -0,0 +1,34 @@
+"""Add support for quote URL
+
+Revision ID: c3027d0e18dc
+Revises: 604d125ea2fb
+Create Date: 2022-09-21 07:08:24.568124+00:00
+
+"""
+import sqlalchemy as sa
+
+from alembic import op
+
+# revision identifiers, used by Alembic.
+revision = 'c3027d0e18dc'
+down_revision = '604d125ea2fb'
+branch_labels = None
+depends_on = None
+
+
+def upgrade() -> None:
+    # ### commands auto generated by Alembic - please adjust! ###
+    with op.batch_alter_table('inbox', schema=None) as batch_op:
+        batch_op.add_column(sa.Column('quoted_inbox_object_id', sa.Integer(), nullable=True))
+        batch_op.create_foreign_key('fk_quoted_inbox_object_id', 'inbox', ['quoted_inbox_object_id'], ['id'])
+
+    # ### end Alembic commands ###
+
+
+def downgrade() -> None:
+    # ### commands auto generated by Alembic - please adjust! ###
+    with op.batch_alter_table('inbox', schema=None) as batch_op:
+        batch_op.drop_constraint('fk_quoted_inbox_object_id', type_='foreignkey')
+        batch_op.drop_column('quoted_inbox_object_id')
+
+    # ### end Alembic commands ###

alembic/versions/2022_10_30_1409-b28c0551c236_add_a_slug_field_for_outbox_objects.py

@@ -1,48 +0,0 @@
-"""Add a slug field for outbox objects
-
-Revision ID: b28c0551c236
-Revises: 604d125ea2fb
-Create Date: 2022-10-30 14:09:14.540461+00:00
-
-"""
-import sqlalchemy as sa
-from sqlalchemy import select
-from sqlalchemy.orm.session import Session
-
-from alembic import op
-
-# revision identifiers, used by Alembic.
-revision = 'b28c0551c236'
-down_revision = '604d125ea2fb'
-branch_labels = None
-depends_on = None
-
-
-def upgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
-    with op.batch_alter_table('outbox', schema=None) as batch_op:
-        batch_op.add_column(sa.Column('slug', sa.String(), nullable=True))
-        batch_op.create_index(batch_op.f('ix_outbox_slug'), ['slug'], unique=False)
-
-    # ### end Alembic commands ###
-
-    # Backfill the slug for existing articles
-    from app.models import OutboxObject
-    from app.utils.text import slugify
-    sess = Session(op.get_bind())
-    articles = sess.execute(select(OutboxObject).where(
-        OutboxObject.ap_type == "Article")
-    ).scalars()
-    for article in articles:
-        title = article.ap_object["name"]
-        article.slug = slugify(title)
-    sess.commit()
-
-
-def downgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
-    with op.batch_alter_table('outbox', schema=None) as batch_op:
-        batch_op.drop_index(batch_op.f('ix_outbox_slug'))
-        batch_op.drop_column('slug')
-
-    # ### end Alembic commands ###

alembic/versions/2022_11_16_1942-fadfd359ce78_add_webmention_webmention_type.py

@@ -1,32 +0,0 @@
-"""Add Webmention.webmention_type
-
-Revision ID: fadfd359ce78
-Revises: b28c0551c236
-Create Date: 2022-11-16 19:42:56.925512+00:00
-
-"""
-import sqlalchemy as sa
-
-from alembic import op
-
-# revision identifiers, used by Alembic.
-revision = 'fadfd359ce78'
-down_revision = 'b28c0551c236'
-branch_labels = None
-depends_on = None
-
-
-def upgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
-    with op.batch_alter_table('webmention', schema=None) as batch_op:
-        batch_op.add_column(sa.Column('webmention_type', sa.Enum('UNKNOWN', 'LIKE', 'REPLY', 'REPOST', name='webmentiontype'), nullable=True))
-
-    # ### end Alembic commands ###
-
-
-def downgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
-    with op.batch_alter_table('webmention', schema=None) as batch_op:
-        batch_op.drop_column('webmention_type')
-
-    # ### end Alembic commands ###

app/activitypub.py

@@ -6,6 +6,7 @@ from typing import Any
 
 import httpx
 from loguru import logger
+from markdown import markdown
 
 from app import config
 from app.config import ALSO_KNOWN_AS
@@ -13,7 +14,6 @@ from app.config import AP_CONTENT_TYPE  # noqa: F401
 from app.config import MOVED_TO
 from app.httpsig import auth
 from app.key import get_pubkey_as_pem
-from app.source import dedup_tags
 from app.source import hashtagify
 from app.utils.url import check_url
 
@@ -53,26 +53,15 @@ AS_EXTENDED_CTX = [
 ]
 
 
-class FetchError(Exception):
-    def __init__(self, url: str, resp: httpx.Response | None = None) -> None:
-        resp_part = ""
-        if resp:
-            resp_part = f", got HTTP {resp.status_code}: {resp.text}"
-        message = f"Failed to fetch {url}{resp_part}"
-        super().__init__(message)
-        self.resp = resp
-        self.url = url
-
-
-class ObjectIsGoneError(FetchError):
+class ObjectIsGoneError(Exception):
     pass
 
 
-class ObjectNotFoundError(FetchError):
+class ObjectNotFoundError(Exception):
     pass
 
 
-class ObjectUnavailableError(FetchError):
+class ObjectUnavailableError(Exception):
     pass
 
 
@@ -101,19 +90,6 @@ class VisibilityEnum(str, enum.Enum):
 
 
 _LOCAL_ACTOR_SUMMARY, _LOCAL_ACTOR_TAGS = hashtagify(config.CONFIG.summary)
-_LOCAL_ACTOR_METADATA = []
-if config.CONFIG.metadata:
-    for kv in config.CONFIG.metadata:
-        kv_value, kv_tags = hashtagify(kv.value)
-        _LOCAL_ACTOR_METADATA.append(
-            {
-                "name": kv.key,
-                "type": "PropertyValue",
-                "value": kv_value,
-            }
-        )
-        _LOCAL_ACTOR_TAGS.extend(kv_tags)
-
 
 ME = {
     "@context": AS_EXTENDED_CTX,
@@ -126,7 +102,7 @@ ME = {
     "outbox": config.BASE_URL + "/outbox",
     "preferredUsername": config.USERNAME,
     "name": config.CONFIG.name,
-    "summary": _LOCAL_ACTOR_SUMMARY,
+    "summary": markdown(_LOCAL_ACTOR_SUMMARY, extensions=["mdx_linkify"]),
     "endpoints": {
         # For compat with servers expecting a sharedInbox...
         "sharedInbox": config.BASE_URL
@@ -134,35 +110,35 @@ ME = {
     },
     "url": config.ID + "/",  # XXX: the path is important for Mastodon compat
     "manuallyApprovesFollowers": config.CONFIG.manually_approves_followers,
-    "attachment": _LOCAL_ACTOR_METADATA,
+    "attachment": [
+        {
+            "name": kv.key,
+            "type": "PropertyValue",
+            "value": markdown(kv.value, extensions=["mdx_linkify", "fenced_code"]),
+        }
+        for kv in config.CONFIG.metadata
+    ]
+    if config.CONFIG.metadata
+    else [],
+    "icon": {
+        "mediaType": mimetypes.guess_type(config.CONFIG.icon_url)[0],
+        "type": "Image",
+        "url": config.CONFIG.icon_url,
+    },
     "publicKey": {
         "id": f"{config.ID}#main-key",
         "owner": config.ID,
         "publicKeyPem": get_pubkey_as_pem(config.KEY_PATH),
     },
-    "tag": dedup_tags(_LOCAL_ACTOR_TAGS),
+    "tag": _LOCAL_ACTOR_TAGS,
 }
 
-if config.CONFIG.icon_url:
-    ME["icon"] = {
-        "mediaType": mimetypes.guess_type(config.CONFIG.icon_url)[0],
-        "type": "Image",
-        "url": config.CONFIG.icon_url,
-    }
-
 if ALSO_KNOWN_AS:
     ME["alsoKnownAs"] = [ALSO_KNOWN_AS]
 
 if MOVED_TO:
     ME["movedTo"] = MOVED_TO
 
-if config.CONFIG.image_url:
-    ME["image"] = {
-        "mediaType": mimetypes.guess_type(config.CONFIG.image_url)[0],
-        "type": "Image",
-        "url": config.CONFIG.image_url,
-    }
-
 
 class NotAnObjectError(Exception):
     def __init__(self, url: str, resp: httpx.Response | None = None) -> None:
@@ -194,17 +170,13 @@ async def fetch(
 
     # Special handling for deleted object
     if resp.status_code == 410:
-        raise ObjectIsGoneError(url, resp)
+        raise ObjectIsGoneError(f"{url} is gone")
     elif resp.status_code in [401, 403]:
-        raise ObjectUnavailableError(url, resp)
+        raise ObjectUnavailableError(f"not allowed to fetch {url}")
     elif resp.status_code == 404:
-        raise ObjectNotFoundError(url, resp)
-
-    try:
-        resp.raise_for_status()
-    except httpx.HTTPError as http_error:
-        raise FetchError(url, resp) from http_error
+        raise ObjectNotFoundError(f"{url} not found")
 
+    resp.raise_for_status()
     try:
         return resp.json()
     except json.JSONDecodeError:

app/actor.py

@@ -1,7 +1,6 @@
 import hashlib
 import typing
 from dataclasses import dataclass
-from datetime import timedelta
 from functools import cached_property
 from typing import Union
 from urllib.parse import urlparse
@@ -12,10 +11,7 @@ from sqlalchemy.orm import joinedload
 
 from app import activitypub as ap
 from app import media
-from app.config import BASE_URL
 from app.database import AsyncSession
-from app.utils.datetime import as_utc
-from app.utils.datetime import now
 
 if typing.TYPE_CHECKING:
     from app.models import Actor as ActorModel
@@ -83,21 +79,11 @@ class Actor:
 
     @property
     def icon_url(self) -> str | None:
-        if icon := self.ap_actor.get("icon"):
-            return icon.get("url")
-        return None
+        return self.ap_actor.get("icon", {}).get("url")
 
     @property
     def icon_media_type(self) -> str | None:
-        if icon := self.ap_actor.get("icon"):
-            return icon.get("mediaType")
-        return None
-
-    @property
-    def image_url(self) -> str | None:
-        if image := self.ap_actor.get("image"):
-            return image.get("url")
-        return None
+        return self.ap_actor.get("icon", {}).get("mediaType")
 
     @property
     def public_key_as_pem(self) -> str:
@@ -112,14 +98,14 @@ class Actor:
         if self.icon_url:
             return media.proxied_media_url(self.icon_url)
         else:
-            return BASE_URL + "/static/nopic.png"
+            return "/static/nopic.png"
 
     @property
     def resized_icon_url(self) -> str:
         if self.icon_url:
             return media.resized_media_url(self.icon_url, 50)
         else:
-            return BASE_URL + "/static/nopic.png"
+            return "/static/nopic.png"
 
     @property
     def tags(self) -> list[ap.RawObject]:
@@ -203,64 +189,26 @@ async def fetch_actor(
     if existing_actor:
         if existing_actor.is_deleted:
             raise ap.ObjectNotFoundError(f"{actor_id} was deleted")
-
-        if now() - as_utc(existing_actor.updated_at) > timedelta(hours=24):
-            logger.info(
-                f"Refreshing {actor_id=} last updated {existing_actor.updated_at}"
-            )
-            try:
-                ap_actor = await ap.fetch(actor_id)
-                await update_actor_if_needed(
-                    db_session,
-                    existing_actor,
-                    RemoteActor(ap_actor),
-                )
-                return existing_actor
-            except Exception:
-                logger.exception(f"Failed to refresh {actor_id}")
-                # If we fail to refresh the actor, return the cached one
-                return existing_actor
-        else:
-            return existing_actor
-
-    if save_if_not_found:
-        ap_actor = await ap.fetch(actor_id)
-        # Some softwares uses URL when we expect ID or uses a different casing
-        # (like Birdsite LIVE) , which mean we may already have it in DB
-        existing_actor_by_url = (
-            await db_session.scalars(
-                select(models.Actor).where(
-                    models.Actor.ap_id == ap.get_id(ap_actor),
-                )
-            )
-        ).one_or_none()
-        if existing_actor_by_url:
-            # Update the actor as we had to fetch it anyway
-            await update_actor_if_needed(
-                db_session,
-                existing_actor_by_url,
-                RemoteActor(ap_actor),
-            )
-            return existing_actor_by_url
-
-        return await save_actor(db_session, ap_actor)
+        return existing_actor
     else:
-        raise ap.ObjectNotFoundError(actor_id)
+        if save_if_not_found:
+            ap_actor = await ap.fetch(actor_id)
+            # Some softwares uses URL when we expect ID
+            if actor_id == ap_actor.get("url"):
+                # Which mean we may already have it in DB
+                existing_actor_by_url = (
+                    await db_session.scalars(
+                        select(models.Actor).where(
+                            models.Actor.ap_id == ap.get_id(ap_actor),
+                        )
+                    )
+                ).one_or_none()
+                if existing_actor_by_url:
+                    return existing_actor_by_url
 
-
-async def update_actor_if_needed(
-    db_session: AsyncSession,
-    actor_in_db: "ActorModel",
-    ra: RemoteActor,
-) -> None:
-    # Check if we actually need to udpte the actor in DB
-    if _actor_hash(ra) != _actor_hash(actor_in_db):
-        actor_in_db.ap_actor = ra.ap_actor
-        actor_in_db.handle = ra.handle
-        actor_in_db.ap_type = ra.ap_type
-
-    actor_in_db.updated_at = now()
-    await db_session.flush()
+            return await save_actor(db_session, ap_actor)
+        else:
+            raise ap.ObjectNotFoundError
 
 
 @dataclass
@@ -269,11 +217,9 @@ class ActorMetadata:
     is_following: bool
     is_follower: bool
     is_follow_request_sent: bool
-    is_follow_request_rejected: bool
     outbox_follow_ap_id: str | None
     inbox_follow_ap_id: str | None
     moved_to: typing.Optional["ActorModel"]
-    has_blocked_local_actor: bool
 
 
 ActorsMetadata = dict[str, ActorMetadata]
@@ -316,26 +262,6 @@ async def get_actors_metadata(
             )
         )
     }
-    rejected_follow_requests = {
-        reject.activity_object_ap_id
-        for reject in await db_session.execute(
-            select(models.InboxObject.activity_object_ap_id).where(
-                models.InboxObject.ap_type == "Reject",
-                models.InboxObject.ap_actor_id.in_(ap_actor_ids),
-            )
-        )
-    }
-    blocks = {
-        block.ap_actor_id
-        for block in await db_session.execute(
-            select(models.InboxObject.ap_actor_id).where(
-                models.InboxObject.ap_type == "Block",
-                models.InboxObject.undone_by_inbox_object_id.is_(None),
-                models.InboxObject.ap_actor_id.in_(ap_actor_ids),
-            )
-        )
-    }
-
     idx: ActorsMetadata = {}
     for actor in actors:
         if not actor.ap_id:
@@ -358,15 +284,9 @@ async def get_actors_metadata(
             is_following=actor.ap_id in following,
             is_follower=actor.ap_id in followers,
             is_follow_request_sent=actor.ap_id in sent_follow_requests,
-            is_follow_request_rejected=bool(
-                sent_follow_requests[actor.ap_id] in rejected_follow_requests
-            )
-            if actor.ap_id in sent_follow_requests
-            else False,
             outbox_follow_ap_id=sent_follow_requests.get(actor.ap_id),
             inbox_follow_ap_id=followers.get(actor.ap_id),
             moved_to=moved_to,
-            has_blocked_local_actor=actor.ap_id in blocks,
         )
     return idx
 
@@ -391,9 +311,6 @@ def _actor_hash(actor: Actor) -> bytes:
     if actor.icon_url:
         h.update(actor.icon_url.encode())
 
-    if actor.image_url:
-        h.update(actor.image_url.encode())
-
     if actor.attachments:
         for a in actor.attachments:
             if a.get("type") != "PropertyValue":

app/admin.py

@@ -11,7 +11,6 @@ from fastapi.exceptions import HTTPException
 from fastapi.responses import RedirectResponse
 from loguru import logger
 from sqlalchemy import and_
-from sqlalchemy import delete
 from sqlalchemy import func
 from sqlalchemy import or_
 from sqlalchemy import select
@@ -26,11 +25,8 @@ from app.actor import fetch_actor
 from app.actor import get_actors_metadata
 from app.boxes import get_inbox_object_by_ap_id
 from app.boxes import get_outbox_object_by_ap_id
-from app.boxes import send_block
 from app.boxes import send_follow
-from app.boxes import send_unblock
 from app.config import EMOJIS
-from app.config import SESSION_TIMEOUT
 from app.config import generate_csrf_token
 from app.config import session_serializer
 from app.config import verify_csrf_token
@@ -63,17 +59,14 @@ async def user_session_or_redirect(
     )
 
     if not session:
-        logger.info("No existing admin session")
         raise _RedirectToLoginPage
 
     try:
-        loaded_session = session_serializer.loads(session, max_age=SESSION_TIMEOUT)
+        loaded_session = session_serializer.loads(session, max_age=3600 * 12)
     except Exception:
-        logger.exception("Failed to validate admin session")
         raise _RedirectToLoginPage
 
     if not loaded_session.get("is_logged_in"):
-        logger.info(f"Admin session invalidated: {loaded_session}")
         raise _RedirectToLoginPage
 
     return None
@@ -347,7 +340,6 @@ async def admin_inbox(
                 "Update",
                 "Undo",
                 "Read",
-                "Reject",
                 "Add",
                 "Remove",
                 "EmojiReact",
@@ -444,7 +436,6 @@ async def admin_direct_messages(
                     models.InboxObject.ap_context.is_not(None),
                     # Skip transient object like poll relies
                     models.InboxObject.is_transient.is_(False),
-                    models.InboxObject.is_deleted.is_(False),
                 )
                 .group_by(models.InboxObject.ap_context, models.InboxObject.actor_id)
             )
@@ -467,7 +458,6 @@ async def admin_direct_messages(
                     models.OutboxObject.ap_context.is_not(None),
                     # Skip transient object like poll relies
                     models.OutboxObject.is_transient.is_(False),
-                    models.OutboxObject.is_deleted.is_(False),
                 )
                 .group_by(models.OutboxObject.ap_context)
             )
@@ -723,9 +713,13 @@ async def get_notifications(
     actors_metadata = await get_actors_metadata(
         db_session, [notif.actor for notif in notifications if notif.actor]
     )
+
+    for notif in notifications:
+        notif.is_new = False
+    await db_session.commit()
+
     more_unread_count = 0
     next_cursor = None
-
     if notifications and remaining_count > page_size:
         decoded_next_cursor = notifications[-1].created_at
         next_cursor = pagination.encode_cursor(decoded_next_cursor)
@@ -739,8 +733,7 @@ async def get_notifications(
             )
         )
 
-    # Render the template before we change the new flag on notifications
-    tpl_resp = await templates.render_template(
+    return await templates.render_template(
         db_session,
         request,
         "notifications.html",
@@ -752,13 +745,6 @@ async def get_notifications(
         },
     )
 
-    if len({notif.id for notif in notifications if notif.is_new}):
-        for notif in notifications:
-            notif.is_new = False
-        await db_session.commit()
-
-    return tpl_resp
-
 
 @router.get("/object")
 async def admin_object(
@@ -861,66 +847,6 @@ async def admin_profile(
     )
 
 
-@router.post("/actions/force_delete")
-async def admin_actions_force_delete(
-    request: Request,
-    ap_object_id: str = Form(),
-    redirect_url: str = Form(),
-    csrf_check: None = Depends(verify_csrf_token),
-    db_session: AsyncSession = Depends(get_db_session),
-) -> RedirectResponse:
-    ap_object_to_delete = await get_inbox_object_by_ap_id(db_session, ap_object_id)
-    if not ap_object_to_delete:
-        raise ValueError(f"Cannot find {ap_object_id}")
-
-    logger.info(f"Deleting {ap_object_to_delete.ap_type}/{ap_object_to_delete.ap_id}")
-    await boxes._revert_side_effect_for_deleted_object(
-        db_session,
-        None,
-        ap_object_to_delete,
-        None,
-    )
-    ap_object_to_delete.is_deleted = True
-    await db_session.commit()
-    return RedirectResponse(redirect_url, status_code=302)
-
-
-@router.post("/actions/force_delete_webmention")
-async def admin_actions_force_delete_webmention(
-    request: Request,
-    webmention_id: int = Form(),
-    redirect_url: str = Form(),
-    csrf_check: None = Depends(verify_csrf_token),
-    db_session: AsyncSession = Depends(get_db_session),
-) -> RedirectResponse:
-    webmention = await boxes.get_webmention_by_id(db_session, webmention_id)
-    if not webmention:
-        raise ValueError(f"Cannot find {webmention_id}")
-    if not webmention.outbox_object:
-        raise ValueError(f"Missing related outbox object for {webmention_id}")
-
-    # TODO: move this
-    logger.info(f"Deleting {webmention_id}")
-    webmention.is_deleted = True
-    await db_session.flush()
-    from app.webmentions import _handle_webmention_side_effects
-
-    await _handle_webmention_side_effects(
-        db_session, webmention, webmention.outbox_object
-    )
-    # Delete related notifications
-    notif_deletion_result = await db_session.execute(
-        delete(models.Notification)
-        .where(models.Notification.webmention_id == webmention.id)
-        .execution_options(synchronize_session=False)
-    )
-    logger.info(
-        f"Deleted {notif_deletion_result.rowcount} notifications"  # type: ignore
-    )
-    await db_session.commit()
-    return RedirectResponse(redirect_url, status_code=302)
-
-
 @router.post("/actions/follow")
 async def admin_actions_follow(
     request: Request,
@@ -942,7 +868,10 @@ async def admin_actions_block(
     csrf_check: None = Depends(verify_csrf_token),
     db_session: AsyncSession = Depends(get_db_session),
 ) -> RedirectResponse:
-    await send_block(db_session, ap_actor_id)
+    logger.info(f"Blocking {ap_actor_id}")
+    actor = await fetch_actor(db_session, ap_actor_id)
+    actor.is_blocked = True
+    await db_session.commit()
     return RedirectResponse(redirect_url, status_code=302)
 
 
@@ -955,7 +884,9 @@ async def admin_actions_unblock(
     db_session: AsyncSession = Depends(get_db_session),
 ) -> RedirectResponse:
     logger.info(f"Unblocking {ap_actor_id}")
-    await send_unblock(db_session, ap_actor_id)
+    actor = await fetch_actor(db_session, ap_actor_id)
+    actor.is_blocked = False
+    await db_session.commit()
     return RedirectResponse(redirect_url, status_code=302)
 
 

app/ap_object.py

@@ -1,18 +1,18 @@
 import hashlib
-import mimetypes
 from datetime import datetime
 from functools import cached_property
 from typing import Any
+from typing import Optional
 
 import pydantic
 from bs4 import BeautifulSoup  # type: ignore
-from mistletoe import markdown  # type: ignore
+from loguru import logger
+from markdown import markdown
 
 from app import activitypub as ap
 from app.actor import LOCAL_ACTOR
 from app.actor import Actor
 from app.actor import RemoteActor
-from app.config import ID
 from app.media import proxied_media_url
 from app.utils.datetime import now
 from app.utils.datetime import parse_isoformat
@@ -76,6 +76,10 @@ class Object:
     def tags(self) -> list[ap.RawObject]:
         return ap.as_list(self.ap_object.get("tag", []))
 
+    @property
+    def quote_url(self) -> str | None:
+        return self.ap_object.get("quoteUrl")
+
     @cached_property
     def inlined_images(self) -> set[str]:
         image_urls: set[str] = set()
@@ -97,9 +101,6 @@ class Object:
     def attachments(self) -> list["Attachment"]:
         attachments = []
         for obj in ap.as_list(self.ap_object.get("attachment", [])):
-            if obj.get("type") == "PropertyValue":
-                continue
-
             if obj.get("type") == "Link":
                 attachments.append(
                     Attachment.parse_obj(
@@ -160,7 +161,7 @@ class Object:
     @cached_property
     def url(self) -> str | None:
         obj_url = self.ap_object.get("url")
-        if isinstance(obj_url, str) and obj_url:
+        if isinstance(obj_url, str):
             return obj_url
         elif obj_url:
             for u in ap.as_list(obj_url):
@@ -180,7 +181,7 @@ class Object:
 
         # PeerTube returns the content as markdown
         if self.ap_object.get("mediaType") == "text/markdown":
-            content = markdown(content)
+            content = markdown(content, extensions=["mdx_linkify"])
 
         return content
 
@@ -213,15 +214,6 @@ class Object:
     def in_reply_to(self) -> str | None:
         return self.ap_object.get("inReplyTo")
 
-    @property
-    def is_local_reply(self) -> bool:
-        if not self.in_reply_to:
-            return False
-
-        return bool(
-            self.in_reply_to.startswith(ID) and self.content  # Hide votes from Question
-        )
-
     @property
     def is_in_reply_to_from_inbox(self) -> bool | None:
         if not self.in_reply_to:
@@ -290,25 +282,17 @@ class Attachment(BaseModel):
     proxied_url: str | None = None
     resized_url: str | None = None
 
-    width: int | None = None
-    height: int | None = None
-
-    @property
-    def mimetype(self) -> str:
-        mimetype = self.media_type
-        if not mimetype:
-            mimetype, _ = mimetypes.guess_type(self.url)
-
-        if not mimetype:
-            return "unknown"
-
-        return mimetype.split("/")[-1]
-
 
 class RemoteObject(Object):
-    def __init__(self, raw_object: ap.RawObject, actor: Actor):
+    def __init__(
+        self,
+        raw_object: ap.RawObject,
+        actor: Actor,
+        quoted_object: Object | None = None,
+    ):
         self._raw_object = raw_object
         self._actor = actor
+        self._quoted_object = quoted_object
 
         if self._actor.ap_id != ap.get_actor_id(self._raw_object):
             raise ValueError(f"Invalid actor {self._actor.ap_id}")
@@ -318,6 +302,7 @@ class RemoteObject(Object):
         cls,
         raw_object: ap.RawObject,
         actor: Actor | None = None,
+        fetch_quoted_url: bool = True,
     ):
         # Pre-fetch the actor
         actor_id = ap.get_actor_id(raw_object)
@@ -334,7 +319,17 @@ class RemoteObject(Object):
                 ap_actor=await ap.fetch(ap.get_actor_id(raw_object)),
             )
 
-        return cls(raw_object, _actor)
+        quoted_object: Object | None = None
+        if quote_url := raw_object.get("quoteUrl"):
+            try:
+                quoted_object = await RemoteObject.from_raw_object(
+                    await ap.fetch(quote_url),
+                    fetch_quoted_url=fetch_quoted_url,
+                )
+            except Exception:
+                logger.exception(f"Failed to fetch {quote_url=}")
+
+        return cls(raw_object, _actor, quoted_object=quoted_object)
 
     @property
     def og_meta(self) -> list[dict[str, Any]] | None:
@@ -347,3 +342,9 @@ class RemoteObject(Object):
     @property
     def actor(self) -> Actor:
         return self._actor
+
+    @property
+    def quoted_object(self) -> Optional["RemoteObject"]:
+        if self._quoted_object:
+            return self._quoted_object
+        return None

app/config.py

@@ -1,5 +1,4 @@
 import hashlib
-import hmac
 import os
 import secrets
 from pathlib import Path
@@ -13,11 +12,8 @@ from fastapi import HTTPException
 from fastapi import Request
 from itsdangerous import URLSafeTimedSerializer
 from loguru import logger
-from mistletoe import markdown  # type: ignore
+from markdown import markdown
 
-from app.customization import _CUSTOM_ROUTES
-from app.customization import _StreamVisibilityCallback
-from app.customization import default_stream_visibility_callback
 from app.utils.emoji import _load_emojis
 from app.utils.version import get_version_commit
 
@@ -44,14 +40,11 @@ except FileNotFoundError:
 JS_HASH = "none"
 try:
     # To keep things simple, we keep a single hash for the 2 files
-    dat = b""
-    for j in [
-        ROOT_DIR / "app" / "static" / "common.js",
-        ROOT_DIR / "app" / "static" / "common-admin.js",
-        ROOT_DIR / "app" / "static" / "new.js",
-    ]:
-        dat += j.read_bytes()
-    JS_HASH = hashlib.md5(dat, usedforsecurity=False).hexdigest()
+    js_data_common = (ROOT_DIR / "app" / "static" / "common-admin.js").read_bytes()
+    js_data_new = (ROOT_DIR / "app" / "static" / "new.js").read_bytes()
+    JS_HASH = hashlib.md5(
+        js_data_common + js_data_new, usedforsecurity=False
+    ).hexdigest()
 except FileNotFoundError:
     pass
 
@@ -96,8 +89,7 @@ class Config(pydantic.BaseModel):
     name: str
     summary: str
     https: bool
-    icon_url: str | None = None
-    image_url: str | None = None
+    icon_url: str
     secret: str
     debug: bool = False
     trusted_hosts: list[str] = ["127.0.0.1"]
@@ -115,19 +107,10 @@ class Config(pydantic.BaseModel):
 
     inbox_retention_days: int = 15
 
-    custom_content_security_policy: str | None = None
-
     # Config items to make tests easier
     sqlalchemy_database: str | None = None
     key_path: str | None = None
 
-    session_timeout: int = 3600 * 24 * 3  # in seconds, 3 days by default
-
-    disabled_notifications: list[str] = []
-
-    # Only set when the app is served on a non-root path
-    id: str | None = None
-
 
 def load_config() -> Config:
     try:
@@ -162,11 +145,6 @@ CONFIG = load_config()
 DOMAIN = CONFIG.domain
 _SCHEME = "https" if CONFIG.https else "http"
 ID = f"{_SCHEME}://{DOMAIN}"
-
-# When running the app on a path, the ID maybe set by the config, but in this
-# case, a valid webfinger must be served on the root domain
-if CONFIG.id:
-    ID = CONFIG.id
 USERNAME = CONFIG.username
 MANUALLY_APPROVES_FOLLOWERS = CONFIG.manually_approves_followers
 HIDES_FOLLOWERS = CONFIG.hides_followers
@@ -177,12 +155,12 @@ if CONFIG.privacy_replace:
 
 BLOCKED_SERVERS = {blocked_server.hostname for blocked_server in CONFIG.blocked_servers}
 ALSO_KNOWN_AS = CONFIG.also_known_as
-CUSTOM_CONTENT_SECURITY_POLICY = CONFIG.custom_content_security_policy
 
 INBOX_RETENTION_DAYS = CONFIG.inbox_retention_days
-SESSION_TIMEOUT = CONFIG.session_timeout
 CUSTOM_FOOTER = (
-    markdown(CONFIG.custom_footer.replace("{version}", VERSION))
+    markdown(
+        CONFIG.custom_footer.replace("{version}", VERSION), extensions=["mdx_linkify"]
+    )
     if CONFIG.custom_footer
     else None
 )
@@ -199,9 +177,7 @@ if CONFIG.emoji:
     EMOJIS = CONFIG.emoji
 
 # Emoji template for the FE
-EMOJI_TPL = (
-    '<img src="{base_url}/static/twemoji/{filename}.svg" alt="{raw}" class="emoji">'
-)
+EMOJI_TPL = '<img src="/static/twemoji/{filename}.svg" alt="{raw}" class="emoji">'
 
 _load_emojis(ROOT_DIR, BASE_URL)
 
@@ -210,31 +186,6 @@ CODE_HIGHLIGHTING_THEME = CONFIG.code_highlighting_theme
 MOVED_TO = _get_moved_to()
 
 
-_NavBarItem = tuple[str, str]
-
-
-class NavBarItems:
-    EXTRA_NAVBAR_ITEMS: list[_NavBarItem] = []
-    INDEX_NAVBAR_ITEM: _NavBarItem | None = None
-    NOTES_PATH = "/"
-
-
-def load_custom_routes() -> None:
-    try:
-        from data import custom_routes  # type: ignore  # noqa: F401
-    except ImportError:
-        pass
-
-    for path, custom_handler in _CUSTOM_ROUTES.items():
-        # If a handler wants to replace the root, move the index to /notes
-        if path == "/":
-            NavBarItems.NOTES_PATH = "/notes"
-            NavBarItems.INDEX_NAVBAR_ITEM = (path, custom_handler.title)
-        else:
-            if custom_handler.show_in_navbar:
-                NavBarItems.EXTRA_NAVBAR_ITEMS.append((path, custom_handler.title))
-
-
 session_serializer = URLSafeTimedSerializer(
     CONFIG.secret,
     salt=f"{ID}.session",
@@ -265,18 +216,3 @@ def verify_csrf_token(
             detail=f"The security token has expired, {please_try_again}",
         )
     return None
-
-
-def hmac_sha256() -> hmac.HMAC:
-    return hmac.new(CONFIG.secret.encode(), digestmod=hashlib.sha256)
-
-
-stream_visibility_callback: _StreamVisibilityCallback
-try:
-    from data.stream import (  # type: ignore  # noqa: F401, E501
-        custom_stream_visibility_callback,
-    )
-
-    stream_visibility_callback = custom_stream_visibility_callback
-except ImportError:
-    stream_visibility_callback = default_stream_visibility_callback

app/customization.py

@@ -1,155 +0,0 @@
-from dataclasses import dataclass
-from pathlib import Path
-from typing import TYPE_CHECKING
-from typing import Any
-from typing import Callable
-
-from fastapi import APIRouter
-from fastapi import Depends
-from fastapi import Request
-from loguru import logger
-from starlette.responses import JSONResponse
-
-if TYPE_CHECKING:
-    from app.ap_object import RemoteObject
-
-
-_DATA_DIR = Path().parent.resolve() / "data"
-_Handler = Callable[..., Any]
-
-
-class HTMLPage:
-    def __init__(
-        self,
-        title: str,
-        html_file: str,
-        show_in_navbar: bool,
-    ) -> None:
-        self.title = title
-        self.html_file = _DATA_DIR / html_file
-        self.show_in_navbar = show_in_navbar
-
-
-class RawHandler:
-    def __init__(
-        self,
-        title: str,
-        handler: Any,
-        show_in_navbar: bool,
-    ) -> None:
-        self.title = title
-        self.handler = handler
-        self.show_in_navbar = show_in_navbar
-
-
-_CUSTOM_ROUTES: dict[str, HTMLPage | RawHandler] = {}
-
-
-def register_html_page(
-    path: str,
-    *,
-    title: str,
-    html_file: str,
-    show_in_navbar: bool = True,
-) -> None:
-    if path in _CUSTOM_ROUTES:
-        raise ValueError(f"{path} is already registered")
-
-    _CUSTOM_ROUTES[path] = HTMLPage(title, html_file, show_in_navbar)
-
-
-def register_raw_handler(
-    path: str,
-    *,
-    title: str,
-    handler: _Handler,
-    show_in_navbar: bool = True,
-) -> None:
-    if path in _CUSTOM_ROUTES:
-        raise ValueError(f"{path} is already registered")
-
-    _CUSTOM_ROUTES[path] = RawHandler(title, handler, show_in_navbar)
-
-
-class ActivityPubResponse(JSONResponse):
-    media_type = "application/activity+json"
-
-
-def _custom_page_handler(path: str, html_page: HTMLPage) -> Any:
-    from app import templates
-    from app.actor import LOCAL_ACTOR
-    from app.config import is_activitypub_requested
-    from app.database import AsyncSession
-    from app.database import get_db_session
-
-    async def _handler(
-        request: Request,
-        db_session: AsyncSession = Depends(get_db_session),
-    ) -> templates.TemplateResponse | ActivityPubResponse:
-        if path == "/" and is_activitypub_requested(request):
-            return ActivityPubResponse(LOCAL_ACTOR.ap_actor)
-
-        return await templates.render_template(
-            db_session,
-            request,
-            "custom_page.html",
-            {
-                "page_content": html_page.html_file.read_text(),
-                "title": html_page.title,
-            },
-        )
-
-    return _handler
-
-
-def get_custom_router() -> APIRouter | None:
-    if not _CUSTOM_ROUTES:
-        return None
-
-    router = APIRouter()
-
-    for path, handler in _CUSTOM_ROUTES.items():
-        if isinstance(handler, HTMLPage):
-            router.add_api_route(
-                path, _custom_page_handler(path, handler), methods=["GET"]
-            )
-        else:
-            router.add_api_route(path, handler.handler)
-
-    return router
-
-
-@dataclass
-class ObjectInfo:
-    # Is it a reply?
-    is_reply: bool
-
-    # Is it a reply to an outbox object
-    is_local_reply: bool
-
-    # Is the object mentioning the local actor
-    is_mention: bool
-
-    # Is it from someone the local actor is following
-    is_from_following: bool
-
-    # List of hashtags, e.g. #microblogpub
-    hashtags: list[str]
-
-    # @dev@microblog.pub
-    actor_handle: str
-
-    remote_object: "RemoteObject"
-
-
-_StreamVisibilityCallback = Callable[[ObjectInfo], bool]
-
-
-def default_stream_visibility_callback(object_info: ObjectInfo) -> bool:
-    result = (
-        (not object_info.is_reply and object_info.is_from_following)
-        or object_info.is_mention
-        or object_info.is_local_reply
-    )
-    logger.info(f"{object_info=}/{result=}")
-    return result

app/httpsig.py

@@ -1,6 +1,5 @@
 import base64
 import hashlib
-import json
 import typing
 from dataclasses import dataclass
 from datetime import datetime
@@ -23,12 +22,12 @@ from sqlalchemy import select
 
 from app import activitypub as ap
 from app import config
+from app.config import BLOCKED_SERVERS
 from app.config import KEY_PATH
 from app.database import AsyncSession
 from app.database import get_db_session
 from app.key import Key
 from app.utils.datetime import now
-from app.utils.url import is_hostname_blocked
 
 _KEY_CACHE: MutableMapping[str, Key] = LFUCache(256)
 
@@ -89,12 +88,8 @@ def _body_digest(body: bytes) -> str:
     return "SHA-256=" + base64.b64encode(h.digest()).decode("utf-8")
 
 
-async def _get_public_key(
-    db_session: AsyncSession,
-    key_id: str,
-    should_skip_cache: bool = False,
-) -> Key:
-    if not should_skip_cache and (cached_key := _KEY_CACHE.get(key_id)):
+async def _get_public_key(db_session: AsyncSession, key_id: str) -> Key:
+    if cached_key := _KEY_CACHE.get(key_id):
         logger.info(f"Key {key_id} found in cache")
         return cached_key
 
@@ -106,18 +101,15 @@ async def _get_public_key(
             select(models.Actor).where(models.Actor.ap_id == key_id.split("#")[0])
         )
     ).one_or_none()
-    if not should_skip_cache:
-        if existing_actor and existing_actor.public_key_id == key_id:
-            k = Key(existing_actor.ap_id, key_id)
-            k.load_pub(existing_actor.public_key_as_pem)
-            logger.info(f"Found {key_id} on an existing actor")
-            _KEY_CACHE[key_id] = k
-            return k
+    if existing_actor and existing_actor.public_key_id == key_id:
+        k = Key(existing_actor.ap_id, key_id)
+        k.load_pub(existing_actor.public_key_as_pem)
+        logger.info(f"Found {key_id} on an existing actor")
+        _KEY_CACHE[key_id] = k
+        return k
 
     # Fetch it
     from app import activitypub as ap
-    from app.actor import RemoteActor
-    from app.actor import update_actor_if_needed
 
     # Without signing the request as if it's the first contact, the 2 servers
     # might race to fetch each other key
@@ -141,12 +133,6 @@ async def _get_public_key(
             f"failed to fetch requested key {key_id}: got {actor['publicKey']}"
         )
 
-    if should_skip_cache and actor["type"] != "Key" and existing_actor:
-        # We had to skip the cache, which means the actor key probably changed
-        # and we want to update our cached version
-        await update_actor_if_needed(db_session, existing_actor, RemoteActor(actor))
-        await db_session.commit()
-
     _KEY_CACHE[key_id] = k
     return k
 
@@ -184,7 +170,7 @@ async def httpsig_checker(
         )
 
     server = urlparse(key_id).hostname
-    if is_hostname_blocked(server):
+    if server in BLOCKED_SERVERS:
         return HTTPSigInfo(
             has_valid_signature=False,
             server=server,
@@ -199,32 +185,6 @@ async def httpsig_checker(
             server=server,
         )
 
-    # Try to drop Delete activity spams early on, this prevent making an extra
-    # HTTP requests trying to fetch an unavailable actor to verify the HTTP sig
-    try:
-        if request.method == "POST" and request.url.path.endswith("/inbox"):
-            from app import models  # TODO: solve this circular import
-
-            activity = json.loads(body)
-            actor_id = ap.get_id(activity["actor"])
-            if (
-                ap.as_list(activity["type"])[0] == "Delete"
-                and actor_id == ap.get_id(activity["object"])
-                and not (
-                    await db_session.scalars(
-                        select(models.Actor).where(
-                            models.Actor.ap_id == actor_id,
-                        )
-                    )
-                ).one_or_none()
-            ):
-                logger.info(f"Dropping Delete activity early for {body=}")
-                raise fastapi.HTTPException(status_code=202)
-    except fastapi.HTTPException as http_exc:
-        raise http_exc
-    except Exception:
-        logger.exception("Failed to check for Delete spam")
-
     # logger.debug(f"hsig={hsig}")
     signed_string, signature_date = _build_signed_string(
         hsig["headers"],
@@ -256,17 +216,7 @@ async def httpsig_checker(
     has_valid_signature = _verify_h(
         signed_string, base64.b64decode(hsig["signature"]), k.pubkey
     )
-
-    # If the signature is not valid, we may have to update the cached actor
-    if not has_valid_signature:
-        logger.info("Invalid signature, trying to refresh actor")
-        try:
-            k = await _get_public_key(db_session, hsig["keyId"], should_skip_cache=True)
-            has_valid_signature = _verify_h(
-                signed_string, base64.b64decode(hsig["signature"]), k.pubkey
-            )
-        except Exception:
-            logger.exception("Failed to refresh actor")
+    # FIXME: fetch/update the user if the signature is wrong
 
     httpsig_info = HTTPSigInfo(
         has_valid_signature=has_valid_signature,

app/incoming_activities.py

@@ -3,6 +3,7 @@ import traceback
 from datetime import datetime
 from datetime import timedelta
 
+import httpx
 from loguru import logger
 from sqlalchemy import func
 from sqlalchemy import select
@@ -107,7 +108,6 @@ async def process_next_incoming_activity(
 
     next_activity.tries = next_activity.tries + 1
     next_activity.last_try = now()
-    await db_session.commit()
 
     if next_activity.ap_object and next_activity.sent_by_ap_actor_id:
         try:
@@ -120,16 +120,13 @@ async def process_next_incoming_activity(
                     ),
                     timeout=60,
                 )
-        except asyncio.exceptions.TimeoutError:
-            logger.error("Activity took too long to process")
-            await db_session.rollback()
-            await db_session.refresh(next_activity)
+        except httpx.TimeoutException as exc:
+            url = exc._request.url if exc._request else None
+            logger.error(f"Failed, HTTP timeout when fetching {url}")
             next_activity.error = traceback.format_exc()
             _set_next_try(next_activity)
         except Exception:
             logger.exception("Failed")
-            await db_session.rollback()
-            await db_session.refresh(next_activity)
             next_activity.error = traceback.format_exc()
             _set_next_try(next_activity)
         else:

app/main.py

@@ -48,7 +48,6 @@ from app import boxes
 from app import config
 from app import httpsig
 from app import indieauth
-from app import media
 from app import micropub
 from app import models
 from app import templates
@@ -64,7 +63,6 @@ from app.config import USER_AGENT
 from app.config import USERNAME
 from app.config import is_activitypub_requested
 from app.config import verify_csrf_token
-from app.customization import get_custom_router
 from app.database import AsyncSession
 from app.database import async_session
 from app.database import get_db_session
@@ -73,9 +71,6 @@ from app.templates import is_current_user_admin
 from app.uploads import UPLOAD_DIR
 from app.utils import pagination
 from app.utils.emoji import EMOJIS_BY_NAME
-from app.utils.facepile import Face
-from app.utils.facepile import WebmentionReply
-from app.utils.facepile import merge_faces
 from app.utils.highlight import HIGHLIGHT_CSS_HASH
 from app.utils.url import check_url
 from app.webfinger import get_remote_follow_template
@@ -140,15 +135,9 @@ class CustomMiddleware:
                 headers["x-frame-options"] = "DENY"
                 headers["permissions-policy"] = "interest-cohort=()"
                 headers["content-security-policy"] = (
-                    (
-                        f"default-src 'self'; "
-                        f"style-src 'self' 'sha256-{HIGHLIGHT_CSS_HASH}'; "
-                        f"frame-ancestors 'none'; base-uri 'self'; form-action 'self';"
-                    )
-                    if not config.CUSTOM_CONTENT_SECURITY_POLICY
-                    else config.CUSTOM_CONTENT_SECURITY_POLICY.format(
-                        HIGHLIGHT_CSS_HASH=HIGHLIGHT_CSS_HASH
-                    )
+                    f"default-src 'self'; "
+                    f"style-src 'self' 'sha256-{HIGHLIGHT_CSS_HASH}'; "
+                    f"frame-ancestors 'none'; base-uri 'self'; form-action 'self';"
                 )
                 if not DEBUG:
                     headers["strict-transport-security"] = "max-age=63072000;"
@@ -203,9 +192,6 @@ app.include_router(admin.unauthenticated_router, prefix="/admin")
 app.include_router(indieauth.router)
 app.include_router(micropub.router)
 app.include_router(webmentions.router)
-config.load_custom_routes()
-if custom_router := get_custom_router():
-    app.include_router(custom_router)
 
 # XXX: order matters, the proxy middleware needs to be last
 app.add_middleware(CustomMiddleware)
@@ -257,34 +243,11 @@ class ActivityPubResponse(JSONResponse):
     media_type = "application/activity+json"
 
 
-async def redirect_to_remote_instance(
-    request: Request,
-    db_session: AsyncSession,
-    url: str,
-) -> templates.TemplateResponse:
-    """
-    Similar to RedirectResponse, but uses a 200 response with HTML.
-
-    Needed for remote redirects on form submission endpoints,
-    since our CSP policy disallows remote form submission.
-    https://github.com/w3c/webappsec-csp/issues/8#issuecomment-810108984
-    """
-    return await templates.render_template(
-        db_session,
-        request,
-        "redirect_to_remote_instance.html",
-        {
-            "request": request,
-            "url": url,
-        },
-        headers={"Refresh": "0;url=" + url},
-    )
-
-
-@app.get(config.NavBarItems.NOTES_PATH)
+@app.get("/")
 async def index(
     request: Request,
     db_session: AsyncSession = Depends(get_db_session),
+    _: httpsig.HTTPSigInfo = Depends(httpsig.httpsig_checker),
     page: int | None = None,
 ) -> templates.TemplateResponse | ActivityPubResponse:
     if is_activitypub_requested(request):
@@ -296,7 +259,7 @@ async def index(
         models.OutboxObject.visibility == ap.VisibilityEnum.PUBLIC,
         models.OutboxObject.is_deleted.is_(False),
         models.OutboxObject.is_hidden_from_homepage.is_(False),
-        models.OutboxObject.ap_type.in_(["Announce", "Note", "Video", "Question"]),
+        models.OutboxObject.ap_type != "Article",
     )
     q = select(models.OutboxObject).where(*where)
     total_count = await db_session.scalar(
@@ -669,75 +632,13 @@ async def _check_outbox_object_acl(
     raise HTTPException(status_code=404)
 
 
-async def _fetch_likes(
-    db_session: AsyncSession,
-    outbox_object: models.OutboxObject,
-) -> list[models.InboxObject]:
-    return (
-        (
-            await db_session.scalars(
-                select(models.InboxObject)
-                .where(
-                    models.InboxObject.ap_type == "Like",
-                    models.InboxObject.activity_object_ap_id == outbox_object.ap_id,
-                    models.InboxObject.is_deleted.is_(False),
-                )
-                .options(joinedload(models.InboxObject.actor))
-                .order_by(models.InboxObject.ap_published_at.desc())
-                .limit(10)
-            )
-        )
-        .unique()
-        .all()
-    )
-
-
-async def _fetch_shares(
-    db_session: AsyncSession,
-    outbox_object: models.OutboxObject,
-) -> list[models.InboxObject]:
-    return (
-        (
-            await db_session.scalars(
-                select(models.InboxObject)
-                .filter(
-                    models.InboxObject.ap_type == "Announce",
-                    models.InboxObject.activity_object_ap_id == outbox_object.ap_id,
-                    models.InboxObject.is_deleted.is_(False),
-                )
-                .options(joinedload(models.InboxObject.actor))
-                .order_by(models.InboxObject.ap_published_at.desc())
-                .limit(10)
-            )
-        )
-        .unique()
-        .all()
-    )
-
-
-async def _fetch_webmentions(
-    db_session: AsyncSession,
-    outbox_object: models.OutboxObject,
-) -> list[models.Webmention]:
-    return (
-        await db_session.scalars(
-            select(models.Webmention)
-            .filter(
-                models.Webmention.outbox_object_id == outbox_object.id,
-                models.Webmention.is_deleted.is_(False),
-            )
-            .limit(50)
-        )
-    ).all()
-
-
 @app.get("/o/{public_id}")
 async def outbox_by_public_id(
     public_id: str,
     request: Request,
     db_session: AsyncSession = Depends(get_db_session),
     httpsig_info: httpsig.HTTPSigInfo = Depends(httpsig.httpsig_checker),
-) -> ActivityPubResponse | templates.TemplateResponse | RedirectResponse:
+) -> ActivityPubResponse | templates.TemplateResponse:
     maybe_object = (
         (
             await db_session.execute(
@@ -764,148 +665,69 @@ async def outbox_by_public_id(
     if is_activitypub_requested(request):
         return ActivityPubResponse(maybe_object.ap_object)
 
-    if maybe_object.ap_type == "Article":
-        return RedirectResponse(
-            f"{BASE_URL}/articles/{public_id[:7]}/{maybe_object.slug}",
-            status_code=301,
-        )
-
     replies_tree = await boxes.get_replies_tree(
         db_session,
         maybe_object,
         is_current_user_admin=is_current_user_admin(request),
     )
 
-    webmentions = await _fetch_webmentions(db_session, maybe_object)
-    likes = await _fetch_likes(db_session, maybe_object)
-    shares = await _fetch_shares(db_session, maybe_object)
-    return await templates.render_template(
-        db_session,
-        request,
-        "object.html",
-        {
-            "replies_tree": _merge_replies(replies_tree, webmentions),
-            "outbox_object": maybe_object,
-            "likes": _merge_faces_from_inbox_object_and_webmentions(
-                likes,
-                webmentions,
-                models.WebmentionType.LIKE,
-            ),
-            "shares": _merge_faces_from_inbox_object_and_webmentions(
-                shares,
-                webmentions,
-                models.WebmentionType.REPOST,
-            ),
-            "webmentions": _filter_webmentions(webmentions),
-        },
-    )
-
-
-def _filter_webmentions(
-    webmentions: list[models.Webmention],
-) -> list[models.Webmention]:
-    return [
-        wm
-        for wm in webmentions
-        if wm.webmention_type
-        not in [
-            models.WebmentionType.LIKE,
-            models.WebmentionType.REPOST,
-            models.WebmentionType.REPLY,
-        ]
-    ]
-
-
-def _merge_faces_from_inbox_object_and_webmentions(
-    inbox_objects: list[models.InboxObject],
-    webmentions: list[models.Webmention],
-    webmention_type: models.WebmentionType,
-) -> list[Face]:
-    wm_faces = []
-    for wm in webmentions:
-        if wm.webmention_type != webmention_type:
-            continue
-        if face := Face.from_webmention(wm):
-            wm_faces.append(face)
-
-    return merge_faces(
-        [Face.from_inbox_object(obj) for obj in inbox_objects] + wm_faces
-    )
-
-
-def _merge_replies(
-    reply_tree_node: boxes.ReplyTreeNode,
-    webmentions: list[models.Webmention],
-) -> boxes.ReplyTreeNode:
-    # TODO: return None as we update the object in place
-    webmention_replies = []
-    for wm in [
-        wm for wm in webmentions if wm.webmention_type == models.WebmentionType.REPLY
-    ]:
-        if rep := WebmentionReply.from_webmention(wm):
-            webmention_replies.append(
-                boxes.ReplyTreeNode(
-                    ap_object=None,
-                    wm_reply=rep,
-                    is_requested=False,
-                    children=[],
+    likes = (
+        (
+            await db_session.scalars(
+                select(models.InboxObject)
+                .where(
+                    models.InboxObject.ap_type == "Like",
+                    models.InboxObject.activity_object_ap_id == maybe_object.ap_id,
+                    models.InboxObject.is_deleted.is_(False),
                 )
+                .options(joinedload(models.InboxObject.actor))
+                .order_by(models.InboxObject.ap_published_at.desc())
+                .limit(10)
             )
-
-    reply_tree_node.children = sorted(
-        reply_tree_node.children + webmention_replies,
-        key=lambda node: node.published_at,
-        reverse=True,
-    )
-    return reply_tree_node
-
-
-@app.get("/articles/{short_id}/{slug}")
-async def article_by_slug(
-    short_id: str,
-    slug: str,
-    request: Request,
-    db_session: AsyncSession = Depends(get_db_session),
-    httpsig_info: httpsig.HTTPSigInfo = Depends(httpsig.httpsig_checker),
-) -> ActivityPubResponse | templates.TemplateResponse | RedirectResponse:
-    maybe_object = await boxes.get_outbox_object_by_slug_and_short_id(
-        db_session, slug, short_id
-    )
-    if not maybe_object:
-        raise HTTPException(status_code=404)
-
-    await _check_outbox_object_acl(request, db_session, maybe_object, httpsig_info)
-
-    if is_activitypub_requested(request):
-        return ActivityPubResponse(maybe_object.ap_object)
-
-    replies_tree = await boxes.get_replies_tree(
-        db_session,
-        maybe_object,
-        is_current_user_admin=is_current_user_admin(request),
+        )
+        .unique()
+        .all()
     )
 
-    likes = await _fetch_likes(db_session, maybe_object)
-    shares = await _fetch_shares(db_session, maybe_object)
-    webmentions = await _fetch_webmentions(db_session, maybe_object)
+    shares = (
+        (
+            await db_session.scalars(
+                select(models.InboxObject)
+                .filter(
+                    models.InboxObject.ap_type == "Announce",
+                    models.InboxObject.activity_object_ap_id == maybe_object.ap_id,
+                    models.InboxObject.is_deleted.is_(False),
+                )
+                .options(joinedload(models.InboxObject.actor))
+                .order_by(models.InboxObject.ap_published_at.desc())
+                .limit(10)
+            )
+        )
+        .unique()
+        .all()
+    )
+
+    webmentions = (
+        await db_session.scalars(
+            select(models.Webmention)
+            .filter(
+                models.Webmention.outbox_object_id == maybe_object.id,
+                models.Webmention.is_deleted.is_(False),
+            )
+            .limit(10)
+        )
+    ).all()
+
     return await templates.render_template(
         db_session,
         request,
         "object.html",
         {
-            "replies_tree": _merge_replies(replies_tree, webmentions),
+            "replies_tree": replies_tree,
             "outbox_object": maybe_object,
-            "likes": _merge_faces_from_inbox_object_and_webmentions(
-                likes,
-                webmentions,
-                models.WebmentionType.LIKE,
-            ),
-            "shares": _merge_faces_from_inbox_object_and_webmentions(
-                shares,
-                webmentions,
-                models.WebmentionType.REPOST,
-            ),
-            "webmentions": _filter_webmentions(webmentions),
+            "likes": likes,
+            "shares": shares,
+            "webmentions": webmentions,
         },
     )
 
@@ -941,7 +763,7 @@ async def tag_by_name(
     _: httpsig.HTTPSigInfo = Depends(httpsig.httpsig_checker),
 ) -> ActivityPubResponse | templates.TemplateResponse:
     where = [
-        models.TaggedOutboxObject.tag == tag.lower(),
+        models.TaggedOutboxObject.tag == tag,
         models.OutboxObject.visibility == ap.VisibilityEnum.PUBLIC,
         models.OutboxObject.is_deleted.is_(False),
     ]
@@ -967,7 +789,7 @@ async def tag_by_name(
         return ActivityPubResponse(
             {
                 "@context": ap.AS_CTX,
-                "id": BASE_URL + f"/t/{tag.lower()}",
+                "id": BASE_URL + f"/t/{tag}",
                 "type": "OrderedCollection",
                 "totalItems": tagged_count,
                 "orderedItems": [
@@ -1044,10 +866,9 @@ async def get_remote_follow(
 @app.post("/remote_follow")
 async def post_remote_follow(
     request: Request,
-    db_session: AsyncSession = Depends(get_db_session),
     csrf_check: None = Depends(verify_csrf_token),
     profile: str = Form(),
-) -> templates.TemplateResponse:
+) -> RedirectResponse:
     if not profile.startswith("@"):
         profile = f"@{profile}"
 
@@ -1056,54 +877,9 @@ async def post_remote_follow(
         # TODO(ts): error message to user
         raise HTTPException(status_code=404)
 
-    return await redirect_to_remote_instance(
-        request,
-        db_session,
-        remote_follow_template.format(uri=ID),
-    )
-
-
-@app.get("/remote_interaction")
-async def remote_interaction(
-    request: Request,
-    ap_id: str,
-    db_session: AsyncSession = Depends(get_db_session),
-) -> templates.TemplateResponse:
-    outbox_object = await boxes.get_outbox_object_by_ap_id(
-        db_session,
-        ap_id,
-    )
-    if not outbox_object:
-        raise HTTPException(status_code=404)
-
-    return await templates.render_template(
-        db_session,
-        request,
-        "remote_interact.html",
-        {"outbox_object": outbox_object},
-    )
-
-
-@app.post("/remote_interaction")
-async def post_remote_interaction(
-    request: Request,
-    db_session: AsyncSession = Depends(get_db_session),
-    csrf_check: None = Depends(verify_csrf_token),
-    profile: str = Form(),
-    ap_id: str = Form(),
-) -> templates.TemplateResponse:
-    if not profile.startswith("@"):
-        profile = f"@{profile}"
-
-    remote_follow_template = await get_remote_follow_template(profile)
-    if not remote_follow_template:
-        # TODO(ts): error message to user
-        raise HTTPException(status_code=404)
-
-    return await redirect_to_remote_instance(
-        request,
-        db_session,
+    return RedirectResponse(
         remote_follow_template.format(uri=ID),
+        status_code=302,
     )
 
 
@@ -1179,11 +955,11 @@ async def nodeinfo(
     )
 
 
+proxy_client = httpx.AsyncClient(follow_redirects=True, http2=True)
+
+
 async def _proxy_get(
-    proxy_client: httpx.AsyncClient,
-    request: starlette.requests.Request,
-    url: str,
-    stream: bool,
+    request: starlette.requests.Request, url: str, stream: bool
 ) -> httpx.Response:
     # Request the URL (and filter request headers)
     proxy_req = proxy_client.build_request(
@@ -1224,35 +1000,21 @@ def _add_cache_control(headers: dict[str, str]) -> dict[str, str]:
     return {**headers, "Cache-Control": "max-age=31536000"}
 
 
-@app.get("/proxy/media/{exp}/{sig}/{encoded_url}")
+@app.get("/proxy/media/{encoded_url}")
 async def serve_proxy_media(
     request: Request,
-    exp: int,
-    sig: str,
     encoded_url: str,
-    background_tasks: fastapi.BackgroundTasks,
 ) -> StreamingResponse | PlainTextResponse:
     # Decode the base64-encoded URL
     url = base64.urlsafe_b64decode(encoded_url).decode()
     check_url(url)
-    media.verify_proxied_media_sig(exp, url, sig)
 
-    proxy_client = httpx.AsyncClient(
-        follow_redirects=True,
-        timeout=httpx.Timeout(timeout=10.0),
-        transport=httpx.AsyncHTTPTransport(retries=1),
-    )
-
-    async def _close_proxy_client():
-        await proxy_client.aclose()
-
-    background_tasks.add_task(_close_proxy_client)
-    proxy_resp = await _proxy_get(proxy_client, request, url, stream=True)
+    proxy_resp = await _proxy_get(request, url, stream=True)
 
     if proxy_resp.status_code >= 300:
         logger.info(f"failed to proxy {url}, got {proxy_resp.status_code}")
-        await proxy_resp.aclose()
         return PlainTextResponse(
+            "proxy error",
             status_code=proxy_resp.status_code,
         )
 
@@ -1278,14 +1040,11 @@ async def serve_proxy_media(
     )
 
 
-@app.get("/proxy/media/{exp}/{sig}/{encoded_url}/{size}")
+@app.get("/proxy/media/{encoded_url}/{size}")
 async def serve_proxy_media_resized(
     request: Request,
-    exp: int,
-    sig: str,
     encoded_url: str,
     size: int,
-    background_tasks: fastapi.BackgroundTasks,
 ) -> PlainTextResponse:
     if size not in {50, 740}:
         raise ValueError("Unsupported size")
@@ -1293,7 +1052,6 @@ async def serve_proxy_media_resized(
     # Decode the base64-encoded URL
     url = base64.urlsafe_b64decode(encoded_url).decode()
     check_url(url)
-    media.verify_proxied_media_sig(exp, url, sig)
 
     if cached_resp := _RESIZED_CACHE.get((url, size)):
         resized_content, resized_mimetype, resp_headers = cached_resp
@@ -1303,21 +1061,11 @@ async def serve_proxy_media_resized(
             headers=resp_headers,
         )
 
-    proxy_client = httpx.AsyncClient(
-        follow_redirects=True,
-        timeout=httpx.Timeout(timeout=10.0),
-        transport=httpx.AsyncHTTPTransport(retries=1),
-    )
-
-    async def _close_proxy_client():
-        await proxy_client.aclose()
-
-    background_tasks.add_task(_close_proxy_client)
-    proxy_resp = await _proxy_get(proxy_client, request, url, stream=False)
+    proxy_resp = await _proxy_get(request, url, stream=False)
     if proxy_resp.status_code >= 300:
         logger.info(f"failed to proxy {url}, got {proxy_resp.status_code}")
-        await proxy_resp.aclose()
         return PlainTextResponse(
+            "proxy error",
             status_code=proxy_resp.status_code,
         )
 
@@ -1431,7 +1179,6 @@ async def robots_file():
 Disallow: /followers
 Disallow: /following
 Disallow: /admin
-Disallow: /remote_interaction
 Disallow: /remote_follow"""
 
 
@@ -1481,7 +1228,7 @@ async def json_feed(
                 ],
             }
         )
-    result = {
+    return {
         "version": "https://jsonfeed.org/version/1",
         "title": f"{LOCAL_ACTOR.display_name}'s microblog'",
         "home_page_url": LOCAL_ACTOR.url,
@@ -1489,12 +1236,10 @@ async def json_feed(
         "author": {
             "name": LOCAL_ACTOR.display_name,
             "url": LOCAL_ACTOR.url,
+            "avatar": LOCAL_ACTOR.icon_url,
         },
         "items": data,
     }
-    if LOCAL_ACTOR.icon_url:
-        result["author"]["avatar"] = LOCAL_ACTOR.icon_url  # type: ignore
-    return result
 
 
 async def _gen_rss_feed(
@@ -1506,8 +1251,7 @@ async def _gen_rss_feed(
     fg.description(f"{LOCAL_ACTOR.display_name}'s microblog")
     fg.author({"name": LOCAL_ACTOR.display_name})
     fg.link(href=LOCAL_ACTOR.url, rel="alternate")
-    if LOCAL_ACTOR.icon_url:
-        fg.logo(LOCAL_ACTOR.icon_url)
+    fg.logo(LOCAL_ACTOR.icon_url)
     fg.language("en")
 
     outbox_objects = await _get_outbox_for_feed(db_session)

app/media.py

@@ -1,44 +1,15 @@
 import base64
-import time
 
 from app.config import BASE_URL
-from app.config import hmac_sha256
 
 SUPPORTED_RESIZE = [50, 740]
-EXPIRY_PERIOD = 86400
-EXPIRY_LENGTH = 7
-
-
-class InvalidProxySignatureError(Exception):
-    pass
-
-
-def proxied_media_sig(expires: int, url: str) -> str:
-    hm = hmac_sha256()
-    hm.update(f"{expires}".encode())
-    hm.update(b"|")
-    hm.update(url.encode())
-    return base64.urlsafe_b64encode(hm.digest()).decode()
-
-
-def verify_proxied_media_sig(expires: int, url: str, sig: str) -> None:
-    now = int(time.time() / EXPIRY_PERIOD)
-    expected = proxied_media_sig(expires, url)
-    if now > expires or sig != expected:
-        raise InvalidProxySignatureError("invalid or expired media")
 
 
 def proxied_media_url(url: str) -> str:
     if url.startswith(BASE_URL):
         return url
-    expires = int(time.time() / EXPIRY_PERIOD) + EXPIRY_LENGTH
-    sig = proxied_media_sig(expires, url)
 
-    return (
-        BASE_URL
-        + f"/proxy/media/{expires}/{sig}/"
-        + base64.urlsafe_b64encode(url.encode()).decode()
-    )
+    return "/proxy/media/" + base64.urlsafe_b64encode(url.encode()).decode()
 
 
 def resized_media_url(url: str, size: int) -> str:

app/models.py

@@ -113,6 +113,18 @@ class InboxObject(Base, BaseObject):
         uselist=False,
     )
 
+    quoted_inbox_object_id = Column(
+        Integer,
+        ForeignKey("inbox.id", name="fk_quoted_inbox_object_id"),
+        nullable=True,
+    )
+    quoted_inbox_object: Mapped[Optional["InboxObject"]] = relationship(
+        "InboxObject",
+        foreign_keys=quoted_inbox_object_id,
+        remote_side=id,
+        uselist=False,
+    )
+
     undone_by_inbox_object_id = Column(Integer, ForeignKey("inbox.id"), nullable=True)
 
     # Link the oubox AP ID to allow undo without any extra query
@@ -147,6 +159,12 @@ class InboxObject(Base, BaseObject):
     def is_from_inbox(self) -> bool:
         return True
 
+    @property
+    def quoted_object(self) -> Optional["InboxObject"]:
+        if self.quoted_inbox_object_id:
+            return self.quoted_inbox_object
+        return None
+
 
 class OutboxObject(Base, BaseObject):
     __tablename__ = "outbox"
@@ -158,7 +176,6 @@ class OutboxObject(Base, BaseObject):
     is_hidden_from_homepage = Column(Boolean, nullable=False, default=False)
 
     public_id = Column(String, nullable=False, index=True)
-    slug = Column(String, nullable=True, index=True)
 
     ap_type = Column(String, nullable=False, index=True)
     ap_id: Mapped[str] = Column(String, nullable=False, unique=True, index=True)
@@ -251,8 +268,6 @@ class OutboxObject(Base, BaseObject):
                         "mediaType": attachment.upload.content_type,
                         "name": attachment.alt or attachment.filename,
                         "url": url,
-                        "width": attachment.upload.width,
-                        "height": attachment.upload.height,
                         "proxiedUrl": url,
                         "resizedUrl": BASE_URL
                         + (
@@ -285,11 +300,8 @@ class OutboxObject(Base, BaseObject):
         return True
 
     @property
-    def url(self) -> str | None:
-        # XXX: rewrite old URL here for compat
-        if self.ap_type == "Article" and self.slug and self.public_id:
-            return f"{BASE_URL}/articles/{self.public_id[:7]}/{self.slug}"
-        return super().url
+    def quoted_object(self) -> Optional["InboxObject"]:
+        return None
 
 
 class Follower(Base):
@@ -468,14 +480,6 @@ class IndieAuthAccessToken(Base):
     is_revoked = Column(Boolean, nullable=False, default=False)
 
 
-@enum.unique
-class WebmentionType(str, enum.Enum):
-    UNKNOWN = "unknown"
-    LIKE = "like"
-    REPLY = "reply"
-    REPOST = "repost"
-
-
 class Webmention(Base):
     __tablename__ = "webmention"
     __table_args__ = (UniqueConstraint("source", "target", name="uix_source_target"),)
@@ -492,8 +496,6 @@ class Webmention(Base):
     outbox_object_id = Column(Integer, ForeignKey("outbox.id"), nullable=False)
     outbox_object = relationship(OutboxObject, uselist=False)
 
-    webmention_type = Column(Enum(WebmentionType), nullable=True)
-
     @property
     def as_facepile_item(self) -> webmentions.Webmention | None:
         if not self.source_microformats:
@@ -503,7 +505,6 @@ class Webmention(Base):
                 self.source_microformats["items"], self.source
             )
         except Exception:
-            # TODO: return a facepile with the unknown image
             logger.warning(
                 f"Failed to generate facefile item for Webmention id={self.id}"
             )
@@ -572,14 +573,6 @@ class NotificationType(str, enum.Enum):
     UPDATED_WEBMENTION = "updated_webmention"
     DELETED_WEBMENTION = "deleted_webmention"
 
-    # incoming
-    BLOCKED = "blocked"
-    UNBLOCKED = "unblocked"
-
-    # outgoing
-    BLOCK = "block"
-    UNBLOCK = "unblock"
-
 
 class Notification(Base):
     __tablename__ = "notifications"

app/scss/main.scss

@@ -51,20 +51,17 @@ $code-highlight-background: #f0f0f0;
   .p-summary {
     display: inline-block;
   }
-  .show-more-btn {
+  label {
     margin-left: 5px;
   }
-  summary {
-    display: inline-block;
+  .show-more-state {
+    display: none;
   }
-  summary::-webkit-details-marker {
-    display: none
+  .show-more-state ~ .obj-content {
+    margin-top: 0;
   }
-  &:not([open]) .show-more-btn::after {
-    content: 'show more';
-  }
-  &[open] .show-more-btn::after {
-    content: 'show less';
+  .show-more-state:checked ~ .obj-content {
+    display: none;
   }
 }
 .sensitive-attachment {
@@ -391,7 +388,7 @@ nav.flexbox {
             margin-right: 0px;
         }
     }
-    a:not(.label-btn) {
+    a {
         color: $primary-color;
         text-decoration: none;
         &:hover, &:active {
@@ -399,29 +396,23 @@ nav.flexbox {
             text-decoration: underline;
         }
     }
-    a.active:not(.label-btn) {
+    a.active {
         color: $secondary-color;
         font-weight: bold;
     }
 }
 
-// after nav.flexbox to override default behavior
-a.label-btn {
-  color: $form-text-color;
-  &:hover {
-    text-decoration: none;
-    color: $form-text-color;
-  }
-}
-
 .ap-object {
     margin: 15px 0;
     padding: 20px;
-    nav {
-        color: $muted-color;
-    }
     .in-reply-to {
-        display: inline;
+        color: $muted-color;
+        &:hover {
+            color: $secondary-color;
+            text-decoration: underline;
+        }
+    }
+    nav {
         color: $muted-color;
     }
     .e-content, .activity-og-meta {
@@ -470,9 +461,6 @@ a.label-btn {
   span {
     color: $muted-color;
   }
-  span.new {
-    color: $secondary-color;
-  }
 }
 .actor-metadata {
     color: $muted-color;
@@ -537,36 +525,3 @@ a.label-btn {
     text-decoration: underline;
   }
 }
-
-.ap-place {
-  h3 {
-    display: inline;
-    font-weight: normal;
-  }
-  h3::after {
-    content: ': ';
-  }
-}
-
-.margin-top-20 {
-  margin-top: 20px;
-}
-
-.video-wrapper {
-    position: relative;
-}
-
-.video-gif-overlay {
-    display: none;
-}
-
-.video-gif-mode + .video-gif-overlay {
-    display: block;
-    position: absolute;
-    top: 5px;
-    left: 5px;
-    padding: 0 3px;
-    font-size: 0.8em;
-    background: rgba(0,0,0,.5);
-    color: #fff;
-}

app/source.py

@@ -1,172 +1,74 @@
 import re
 import typing
 
-from loguru import logger
-from mistletoe import Document  # type: ignore
-from mistletoe.html_renderer import HTMLRenderer  # type: ignore
-from mistletoe.span_token import SpanToken  # type: ignore
-from pygments import highlight  # type: ignore
-from pygments.formatters import HtmlFormatter  # type: ignore
-from pygments.lexers import get_lexer_by_name as get_lexer  # type: ignore
-from pygments.lexers import guess_lexer  # type: ignore
+from markdown import markdown
 from sqlalchemy import select
 
 from app import webfinger
 from app.config import BASE_URL
-from app.config import CODE_HIGHLIGHTING_THEME
 from app.database import AsyncSession
 from app.utils import emoji
 
 if typing.TYPE_CHECKING:
     from app.actor import Actor
 
-_FORMATTER = HtmlFormatter(style=CODE_HIGHLIGHTING_THEME)
+
+def _set_a_attrs(attrs, new=False):
+    attrs[(None, "target")] = "_blank"
+    attrs[(None, "class")] = "external"
+    attrs[(None, "rel")] = "noopener"
+    attrs[(None, "title")] = attrs[(None, "href")]
+    return attrs
+
+
 _HASHTAG_REGEX = re.compile(r"(#[\d\w]+)")
-_MENTION_REGEX = re.compile(r"(@[\d\w_.+-]+@[\d\w-]+\.[\d\w\-.]+)")
-_URL_REGEX = re.compile(
-    "(https?:\\/\\/(?:www\\.)?[-a-zA-Z0-9@:%._\\+~#=]{1,256}\\.[a-zA-Z0-9()]{1,6}\\b(?:[-a-zA-Z0-9()@:%_\\+.~#?&\\/=]*))"  # noqa: E501
-)
+_MENTION_REGEX = re.compile(r"@[\d\w_.+-]+@[\d\w-]+\.[\d\w\-.]+")
 
 
-class AutoLink(SpanToken):
-    parse_inner = False
-    precedence = 1
-    pattern = _URL_REGEX
-
-    def __init__(self, match_obj: re.Match) -> None:
-        self.target = match_obj.group()
+def hashtagify(content: str) -> tuple[str, list[dict[str, str]]]:
+    tags = []
+    hashtags = re.findall(_HASHTAG_REGEX, content)
+    hashtags = sorted(set(hashtags), reverse=True)  # unique tags, longest first
+    for hashtag in hashtags:
+        tag = hashtag[1:]
+        link = f'<a href="{BASE_URL}/t/{tag}" class="mention hashtag" rel="tag">#<span>{tag}</span></a>'  # noqa: E501
+        tags.append(dict(href=f"{BASE_URL}/t/{tag}", name=hashtag, type="Hashtag"))
+        content = content.replace(hashtag, link)
+    return content, tags
 
 
-class Mention(SpanToken):
-    parse_inner = False
-    precedence = 10
-    pattern = _MENTION_REGEX
-
-    def __init__(self, match_obj: re.Match) -> None:
-        self.target = match_obj.group()
-
-
-class Hashtag(SpanToken):
-    parse_inner = False
-    precedence = 10
-    pattern = _HASHTAG_REGEX
-
-    def __init__(self, match_obj: re.Match) -> None:
-        self.target = match_obj.group()
-
-
-class CustomRenderer(HTMLRenderer):
-    def __init__(
-        self,
-        mentioned_actors: dict[str, "Actor"] = {},
-        enable_mentionify: bool = True,
-        enable_hashtagify: bool = True,
-    ) -> None:
-        extra_tokens = []
-        if enable_mentionify:
-            extra_tokens.append(Mention)
-        if enable_hashtagify:
-            extra_tokens.append(Hashtag)
-        super().__init__(AutoLink, *extra_tokens)
-
-        self.tags: list[dict[str, str]] = []
-        self.mentioned_actors = mentioned_actors
-
-    def render_auto_link(self, token: AutoLink) -> str:
-        template = '<a href="{target}" rel="noopener">{inner}</a>'
-        target = self.escape_url(token.target)
-        return template.format(target=target, inner=target)
-
-    def render_mention(self, token: Mention) -> str:
-        mention = token.target
-        suffix = ""
-        if mention.endswith("."):
-            mention = mention[:-1]
-            suffix = "."
-        actor = self.mentioned_actors.get(mention)
-        if not actor:
-            return mention
-
-        self.tags.append(dict(type="Mention", href=actor.ap_id, name=mention))
-
-        link = f'<span class="h-card"><a href="{actor.url}" class="u-url mention">{actor.handle}</a></span>{suffix}'  # noqa: E501
-        return link
-
-    def render_hashtag(self, token: Hashtag) -> str:
-        tag = token.target[1:]
-        link = f'<a href="{BASE_URL}/t/{tag.lower()}" class="mention hashtag" rel="tag">#<span>{tag}</span></a>'  # noqa: E501
-        self.tags.append(
-            dict(
-                href=f"{BASE_URL}/t/{tag.lower()}",
-                name=token.target.lower(),
-                type="Hashtag",
-            )
-        )
-        return link
-
-    def render_block_code(self, token: typing.Any) -> str:
-        code = token.children[0].content
-        lexer = get_lexer(token.language) if token.language else guess_lexer(code)
-        return highlight(code, lexer, _FORMATTER)
-
-
-async def _prefetch_mentioned_actors(
+async def _mentionify(
     db_session: AsyncSession,
     content: str,
-) -> dict[str, "Actor"]:
+) -> tuple[str, list[dict[str, str]], list["Actor"]]:
     from app import models
     from app.actor import fetch_actor
 
-    actors = {}
-
-    for mention in re.findall(_MENTION_REGEX, content):
-        if mention in actors:
-            continue
-
-        # XXX: the regex catches stuff like `@toto@example.com.`
-        if mention.endswith("."):
-            mention = mention[:-1]
-
-        try:
-            _, username, domain = mention.split("@")
-            actor = (
-                await db_session.execute(
-                    select(models.Actor).where(
-                        models.Actor.handle == mention,
-                        models.Actor.is_deleted.is_(False),
-                    )
-                )
-            ).scalar_one_or_none()
-            if not actor:
-                actor_url = await webfinger.get_actor_url(mention)
-                if not actor_url:
-                    # FIXME(ts): raise an error?
-                    continue
-                actor = await fetch_actor(db_session, actor_url)
-
-            actors[mention] = actor
-        except Exception:
-            logger.exception(f"Failed to prefetch {mention}")
-
-    return actors
-
-
-def hashtagify(
-    content: str,
-) -> tuple[str, list[dict[str, str]]]:
     tags = []
-    with CustomRenderer(
-        mentioned_actors={},
-        enable_mentionify=False,
-        enable_hashtagify=True,
-    ) as renderer:
-        rendered_content = renderer.render(Document(content))
-        tags.extend(renderer.tags)
+    mentioned_actors = []
+    for mention in re.findall(_MENTION_REGEX, content):
+        _, username, domain = mention.split("@")
+        actor = (
+            await db_session.execute(
+                select(models.Actor).where(
+                    models.Actor.handle == mention,
+                    models.Actor.is_deleted.is_(False),
+                )
+            )
+        ).scalar_one_or_none()
+        if not actor:
+            actor_url = await webfinger.get_actor_url(mention)
+            if not actor_url:
+                # FIXME(ts): raise an error?
+                continue
+            actor = await fetch_actor(db_session, actor_url)
 
-    # Handle custom emoji
-    tags.extend(emoji.tags(content))
+        mentioned_actors.append(actor)
+        tags.append(dict(type="Mention", href=actor.ap_id, name=mention))
 
-    return rendered_content, tags
+        link = f'<span class="h-card"><a href="{actor.url}" class="u-url mention">{actor.handle}</a></span>'  # noqa: E501
+        content = content.replace(mention, link)
+    return content, tags, mentioned_actors
 
 
 async def markdownify(
@@ -180,33 +82,17 @@ async def markdownify(
 
     """
     tags = []
-    mentioned_actors: dict[str, "Actor"] = {}
+    mentioned_actors: list["Actor"] = []
+    if enable_hashtagify:
+        content, hashtag_tags = hashtagify(content)
+        tags.extend(hashtag_tags)
     if enable_mentionify:
-        mentioned_actors = await _prefetch_mentioned_actors(db_session, content)
-
-    with CustomRenderer(
-        mentioned_actors=mentioned_actors,
-        enable_mentionify=enable_mentionify,
-        enable_hashtagify=enable_hashtagify,
-    ) as renderer:
-        rendered_content = renderer.render(Document(content))
-        tags.extend(renderer.tags)
+        content, mention_tags, mentioned_actors = await _mentionify(db_session, content)
+        tags.extend(mention_tags)
 
     # Handle custom emoji
     tags.extend(emoji.tags(content))
 
-    return rendered_content, dedup_tags(tags), list(mentioned_actors.values())
+    content = markdown(content, extensions=["mdx_linkify", "fenced_code"])
 
-
-def dedup_tags(tags: list[dict[str, str]]) -> list[dict[str, str]]:
-    idx = set()
-    deduped_tags = []
-    for tag in tags:
-        tag_idx = (tag["type"], tag["name"])
-        if tag_idx in idx:
-            continue
-
-        idx.add(tag_idx)
-        deduped_tags.append(tag)
-
-    return deduped_tags
+    return content, tags, mentioned_actors

app/static/common.js

@@ -1,32 +0,0 @@
-function hasAudio (video) {
-    return video.mozHasAudio ||
-    Boolean(video.webkitAudioDecodedByteCount) ||
-    Boolean(video.audioTracks && video.audioTracks.length);
-}
-
-function setVideoInGIFMode(video) {
-    if (!hasAudio(video)) {
-        if (typeof video.loop == 'boolean' && video.duration <= 10.0) {
-            video.classList.add("video-gif-mode");
-            video.loop = true;
-            video.controls = false;
-            video.addEventListener("mouseover", () => {
-                video.play();
-            })
-            video.addEventListener("mouseleave", () => {
-                video.pause();
-            })
-        }
-    };
-}
-
-var items = document.getElementsByTagName("video")
-for (var i = 0; i < items.length; i++) {
-    if (items[i].duration) {
-        setVideoInGIFMode(items[i]);
-    } else {
-        items[i].addEventListener("loadeddata", function() {
-            setVideoInGIFMode(this);
-        });
-    }
-}

app/templates.py

@@ -1,3 +1,4 @@
+import base64
 from datetime import datetime
 from datetime import timezone
 from functools import lru_cache
@@ -27,7 +28,6 @@ from app.ap_object import Object
 from app.config import BASE_URL
 from app.config import CUSTOM_FOOTER
 from app.config import DEBUG
-from app.config import SESSION_TIMEOUT
 from app.config import VERSION
 from app.config import generate_csrf_token
 from app.config import session_serializer
@@ -39,7 +39,7 @@ from app.utils.highlight import HIGHLIGHT_CSS
 from app.utils.highlight import highlight
 
 _templates = Jinja2Templates(
-    directory=["data/templates", "app/templates"],  # type: ignore  # bad typing
+    directory="app/templates",
     trim_blocks=True,
     lstrip_blocks=True,
 )
@@ -59,8 +59,13 @@ def _filter_domain(text: str) -> str:
 
 def _media_proxy_url(url: str | None) -> str:
     if not url:
-        return BASE_URL + "/static/nopic.png"
-    return proxied_media_url(url)
+        return "/static/nopic.png"
+
+    if url.startswith(BASE_URL):
+        return url
+
+    encoded_url = base64.urlsafe_b64encode(url.encode()).decode()
+    return f"/proxy/media/{encoded_url}"
 
 
 def is_current_user_admin(request: Request) -> bool:
@@ -70,10 +75,10 @@ def is_current_user_admin(request: Request) -> bool:
         try:
             loaded_session = session_serializer.loads(
                 session_cookie,
-                max_age=SESSION_TIMEOUT,
+                max_age=3600 * 12,
             )
         except Exception:
-            logger.exception("Failed to validate session timeout")
+            pass
         else:
             is_admin = loaded_session.get("is_logged_in")
 
@@ -86,7 +91,6 @@ async def render_template(
     template: str,
     template_args: dict[str, Any] | None = None,
     status_code: int = 200,
-    headers: dict[str, str] | None = None,
 ) -> TemplateResponse:
     if template_args is None:
         template_args = {}
@@ -131,7 +135,6 @@ async def render_template(
             **template_args,
         },
         status_code=status_code,
-        headers=headers,
     )
 
 
@@ -288,10 +291,6 @@ ALLOWED_ATTRIBUTES: dict[str, list[str] | Callable[[str, str, str], bool]] = {
 }
 
 
-def _allow_all_attributes(tag: Any, name: Any, value: Any) -> bool:
-    return True
-
-
 @lru_cache(maxsize=256)
 def _update_inline_imgs(content):
     soup = BeautifulSoup(content, "html5lib")
@@ -321,11 +320,7 @@ def _clean_html(html: str, note: Object) -> str:
                         _update_inline_imgs(highlight(html))
                     ),
                     tags=ALLOWED_TAGS,
-                    attributes=(
-                        _allow_all_attributes
-                        if note.ap_id.startswith(config.ID)
-                        else ALLOWED_ATTRIBUTES
-                    ),
+                    attributes=ALLOWED_ATTRIBUTES,
                     strip=True,
                 ),
                 note,
@@ -336,14 +331,6 @@ def _clean_html(html: str, note: Object) -> str:
         raise
 
 
-def _clean_html_wm(html: str) -> str:
-    return bleach.clean(
-        html,
-        attributes=ALLOWED_ATTRIBUTES,
-        strip=True,
-    )
-
-
 def _timeago(original_dt: datetime) -> str:
     dt = original_dt
     if dt.tzinfo:
@@ -393,7 +380,7 @@ def _html2text(content: str) -> str:
 
 def _replace_emoji(u: str, _) -> str:
     filename = "-".join(hex(ord(c))[2:] for c in u)
-    return config.EMOJI_TPL.format(base_url=BASE_URL, filename=filename, raw=u)
+    return config.EMOJI_TPL.format(filename=filename, raw=u)
 
 
 def _emojify(text: str, is_local: bool) -> str:
@@ -420,7 +407,6 @@ def _poll_item_pct(item: ap.RawObject, voters_count: int) -> int:
 _templates.env.filters["domain"] = _filter_domain
 _templates.env.filters["media_proxy_url"] = _media_proxy_url
 _templates.env.filters["clean_html"] = _clean_html
-_templates.env.filters["clean_html_wm"] = _clean_html_wm
 _templates.env.filters["timeago"] = _timeago
 _templates.env.filters["format_date"] = _format_date
 _templates.env.filters["has_media_type"] = _has_media_type
@@ -435,5 +421,3 @@ _templates.env.globals["CSS_HASH"] = config.CSS_HASH
 _templates.env.globals["BASE_URL"] = config.BASE_URL
 _templates.env.globals["HIDES_FOLLOWERS"] = config.HIDES_FOLLOWERS
 _templates.env.globals["HIDES_FOLLOWING"] = config.HIDES_FOLLOWING
-_templates.env.globals["NAVBAR_ITEMS"] = config.NavBarItems
-_templates.env.globals["ICON_URL"] = config.CONFIG.icon_url

app/templates/admin_new.html

@@ -90,5 +90,5 @@
     </p>
 </form>
 </div>
-<script src="{{ BASE_URL }}/static/new.js?v={{ JS_HASH }}"></script>
+<script src="/static/new.js?v={{ JS_HASH }}"></script>
 {% endblock %}

app/templates/admin_outbox.html

@@ -12,16 +12,18 @@
 {% for outbox_object in outbox %}
 
     {% if outbox_object.ap_type == "Announce" %}
-        <div class="actor-action">You shared <span title="{{ outbox_object.ap_published_at.isoformat() }}">{{ outbox_object.ap_published_at | timeago }}</span></div>
+        <div class="actor-action">You shared</div>
         {{ utils.display_object(outbox_object.relates_to_anybox_object) }}
     {% elif outbox_object.ap_type == "Like" %}
-        <div class="actor-action">You liked <span title="{{ outbox_object.ap_published_at.isoformat() }}">{{ outbox_object.ap_published_at | timeago }}</span></div>
+        <div class="actor-action">You liked</div>
         {{ utils.display_object(outbox_object.relates_to_anybox_object) }}
     {% elif outbox_object.ap_type == "Follow" %}
-        <div class="actor-action">You followed <span title="{{ outbox_object.ap_published_at.isoformat() }}">{{ outbox_object.ap_published_at | timeago }}</span></div>
+        <div class="actor-action">You followed</div>
         {{ utils.display_actor(outbox_object.relates_to_actor, actors_metadata) }}
     {% elif outbox_object.ap_type in ["Article", "Note", "Video", "Question"] %}
         {{ utils.display_object(outbox_object) }}
+    {% else %}
+        Implement {{ outbox_object.ap_type }}
     {% endif %}
 
 {% endfor %}

app/templates/custom_page.html

@@ -1,30 +0,0 @@
-{%- import "utils.html" as utils with context -%}
-{% extends "layout.html" %}
-
-{% block head %}
-<title>{{ title }}</title>
-{% if request.url.path == "/" %}
-<link rel="indieauth-metadata" href="{{ url_for("well_known_authorization_server") }}">
-<link rel="authorization_endpoint" href="{{ url_for("indieauth_authorization_endpoint") }}">
-<link rel="token_endpoint" href="{{ url_for("indieauth_token_endpoint") }}">
-<link rel="micropub" href="{{ url_for("micropub_endpoint") }}">
-<link rel="alternate" href="{{ local_actor.url }}" title="ActivityPub profile"  type="application/activity+json">
-<meta content="profile" property="og:type" />
-<meta content="{{ local_actor.url }}" property="og:url" />
-<meta content="{{ local_actor.display_name }}'s microblog" property="og:site_name" />
-<meta content="Homepage" property="og:title" />
-<meta content="{{ local_actor.summary | html2text | trim }}" property="og:description" />
-<meta content="{{ ICON_URL }}" property="og:image" />
-<meta content="summary" property="twitter:card" />
-<meta content="{{ local_actor.handle }}" property="profile:username" />
-{% endif %}
-{% endblock %}
-
-{% block content %}
-{% include "header.html" %}
-
-<div class="box">
-    {{ page_content | safe }}
-</div>
-
-{% endblock %}

app/templates/followers.html

@@ -3,7 +3,6 @@
 
 {% block head %}
 <title>{{ local_actor.display_name }}'s followers</title>
-<meta name="robots" content="noindex, nofollow">
 {% endblock %}
 
 {% block content %}

app/templates/following.html

@@ -3,7 +3,6 @@
 
 {% block head %}
 <title>{{ local_actor.display_name }}'s follows</title>
-<meta name="robots" content="noindex, nofollow">
 {% endblock %}
 
 {% block content %}

app/templates/header.html

@@ -25,21 +25,13 @@
 </div>
 
 {%- macro header_link(url, text) -%}
-{% set url_for = BASE_URL + request.app.router.url_path_for(url) %}
-<a href="{{ url_for }}" {% if BASE_URL + request.url.path == url_for %}class="active"{% endif %}>{{ text }}</a>
-{% endmacro %}
-
-{%- macro navbar_item_link(navbar_item) -%}
-{% set url_for = BASE_URL + navbar_item[0] %}
-<a href="{{ navbar_item[0] }}" {% if BASE_URL + request.url.path == url_for %}class="active"{% endif %}>{{ navbar_item[1] }}</a>
+{% set url_for = request.app.router.url_path_for(url) %}
+<a href="{{ url_for }}" {% if request.url.path == url_for %}class="active"{% endif %}>{{ text }}</a>
 {% endmacro %}
 
 <div class="public-top-menu">
 <nav class="flexbox">
     <ul>
-        {% if NAVBAR_ITEMS.INDEX_NAVBAR_ITEM %}
-        <li>{{ navbar_item_link(NAVBAR_ITEMS.INDEX_NAVBAR_ITEM) }}</li>
-        {% endif %}
         <li>{{ header_link("index", "Notes") }}</li>
         {% if articles_count %}
             <li>{{ header_link("articles", "Articles") }}</li>
@@ -51,9 +43,6 @@
         <li>{{ header_link("following", "Following") }} <span class="counter">{{ following_count }}</span></li>
         {% endif %}
         <li>{{ header_link("get_remote_follow", "Remote follow") }}</li>
-        {% for navbar_item in NAVBAR_ITEMS.EXTRA_NAVBAR_ITEMS %}
-            {{ navbar_item_link(navbar_item) }}
-        {% endfor %}
     </ul>
 </nav>
 </div>

app/templates/index.html

@@ -13,7 +13,7 @@
 <meta content="{{ local_actor.display_name }}'s microblog" property="og:site_name" />
 <meta content="Homepage" property="og:title" />
 <meta content="{{ local_actor.summary | html2text | trim }}" property="og:description" />
-<meta content="{{ ICON_URL }}" property="og:image" />
+<meta content="{{ local_actor.url }}" property="og:image" />
 <meta content="summary" property="twitter:card" />
 <meta content="{{ local_actor.handle }}" property="profile:username" />
 {% endblock %}
@@ -26,30 +26,24 @@
     <div class="h-feed">
     <data class="p-name" value="{{ local_actor.display_name}}'s notes"></data>
     {% for outbox_object in objects %}
-    {% if outbox_object.ap_type in ["Note", "Video", "Question"] %}
+    {% if outbox_object.ap_type in ["Note", "Article", "Video", "Question"] %}
     {{ utils.display_object(outbox_object) }}
     {% elif outbox_object.ap_type == "Announce" %}
-    <div class="h-entry" id="{{ outbox_object.permalink_id }}">
-        <div class="shared-header"><strong><a class="p-author h-card" href="{{ local_actor.url }}">{{ utils.display_tiny_actor_icon(local_actor) }}  {{ local_actor.display_name | clean_html(local_actor) | safe }}</a></strong> shared <span title="{{ outbox_object.ap_published_at.isoformat() }}">{{ outbox_object.ap_published_at | timeago }}</span></div>
-        <div class="h-cite u-repost-of">
-            {{ utils.display_object(outbox_object.relates_to_anybox_object, is_h_entry=False) }}
-        </div>
-    </div>
+    <div class="shared-header"><strong>{{ utils.display_tiny_actor_icon(local_actor) }} {{ local_actor.display_name | clean_html(local_actor) | safe  }}</strong> shared <span title="{{ outbox_object.ap_published_at.isoformat() }}">{{ outbox_object.ap_published_at | timeago }}</span></div>
+    {{ utils.display_object(outbox_object.relates_to_anybox_object) }}
     {% endif %}
     {% endfor %}
     </div>
 
-    {% if has_previous_page or has_next_page %}
-        <div class="box">
-            {% if has_previous_page %}
-            <a href="{{ url_for("index") }}?page={{ current_page - 1 }}">Previous</a>
-            {% endif %}
+    <div class="box">
+        {% if has_previous_page %}
+        <a href="{{ url_for("index") }}?page={{ current_page - 1 }}">Previous</a>
+        {% endif %}
 
-            {% if has_next_page %}
-            <a href="{{ url_for("index") }}?page={{ current_page + 1 }}">Next</a>
-            {% endif %}
-        </div>
-    {% endif %}
+        {% if has_next_page %}
+        <a href="{{ url_for("index") }}?page={{ current_page + 1 }}">Next</a>
+        {% endif %}
+    </div>
 
 {% else %}
     <div class="empty-state">

app/templates/layout.html

@@ -4,11 +4,11 @@
 <meta charset="utf-8">
 <meta http-equiv="x-ua-compatible" content="ie=edge">
 <meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no">
-<link rel="stylesheet" href="{{ BASE_URL }}/static/css/main.css?v={{ CSS_HASH }}">
+<link rel="stylesheet" href="/static/css/main.css?v={{ CSS_HASH }}">
 <link rel="alternate" title="{{ local_actor.display_name}}'s microblog" type="application/json" href="{{ url_for("json_feed") }}" />
 <link rel="alternate" href="{{ url_for("rss_feed") }}" type="application/rss+xml" title="{{ local_actor.display_name}}'s microblog">
 <link rel="alternate" href="{{ url_for("atom_feed") }}" type="application/atom+xml" title="{{ local_actor.display_name}}'s microblog">
-<link rel="icon" type="image/x-icon" href="{{ BASE_URL }}/static/favicon.ico">
+<link rel="icon" type="image/x-icon" href="/static/favicon.ico">
 <style>{{ highlight_css }}</style>
 {% block head %}{% endblock %}
 </head>
@@ -18,8 +18,8 @@
 {% if is_admin %}
 <div id="admin">
 {% macro admin_link(url, text) %}
-{% set url_for = BASE_URL + request.app.router.url_path_for(url) %}
-<a href="{{ url_for }}" {% if BASE_URL + request.url.path == url_for %}class="active"{% endif %}>{{ text }}</a>
+{% set url_for = request.app.router.url_path_for(url) %}
+<a href="{{ url_for }}" {% if request.url.path == url_for %}class="active"{% endif %}>{{ text }}</a>
 {% endmacro %}
 <div class="admin-menu">
 <nav class="flexbox">
@@ -53,8 +53,7 @@
     </div>
 </footer>
 {% if is_admin %}
-<script src="{{ BASE_URL }}/static/common-admin.js?v={{ JS_HASH }}"></script>
+<script src="/static/common-admin.js?v={{ JS_HASH }}"></script>
 {% endif %}
-<script src="{{ BASE_URL }}/static/common.js?v={{ JS_HASH }}"></script>
 </body>
 </html>

app/templates/login.html

@@ -1,8 +1,5 @@
 {%- import "utils.html" as utils with context -%}
 {% extends "layout.html" %}
-{% block head %}
-<meta name="robots" content="noindex, nofollow">
-{% endblock %}
 {% block main_tag %} class="main-flex"{% endblock %}
 {% block content %}
 <div class="centered">
@@ -10,7 +7,7 @@
         {% if error %}
             <p class="primary-color">Invalid password.</p>
         {% endif %}
-        <form class="form" action="{{ BASE_URL }}/admin/login" method="POST">
+        <form class="form" action="/admin/login" method="POST">
             <input type="hidden" name="csrf_token" value="{{ csrf_token }}">
             <input type="hidden" name="redirect" value="{{ redirect }}">
             <input type="password" placeholder="password" name="password" autofocus>

app/templates/notifications.html

@@ -10,9 +10,6 @@
         <a href="{{ url_for("admin_profile") }}?actor_id={{ notif.actor.ap_id }}">
             {% if with_icon %}{{ utils.display_tiny_actor_icon(notif.actor) }}{% endif %} {{ notif.actor.display_name | clean_html(notif.actor) | safe }}</a> {{ text }}
         <span title="{{ notif.created_at.isoformat() }}">{{ notif.created_at | timeago }}</span>
-        {% if notif.is_new %}
-            <span class="new">new</span>
-        {% endif %}
     </div>
 {% endmacro %}
 
@@ -39,18 +36,6 @@
             {%- elif notif.notification_type.value == "follow_request_rejected" %}
                 {{ notif_actor_action(notif, "rejected your follow request") }}
                 {{ utils.display_actor(notif.actor, actors_metadata) }}
-            {% elif notif.notification_type.value == "blocked" %}
-                {{ notif_actor_action(notif, "blocked you") }}
-                {{ utils.display_actor(notif.actor, actors_metadata) }}
-            {% elif notif.notification_type.value == "unblocked" %}
-                {{ notif_actor_action(notif, "unblocked you") }}
-                {{ utils.display_actor(notif.actor, actors_metadata) }}
-            {% elif notif.notification_type.value == "block" %}
-                {{ notif_actor_action(notif, "was blocked") }}
-                {{ utils.display_actor(notif.actor, actors_metadata) }}
-            {% elif notif.notification_type.value == "unblock" %}
-                {{ notif_actor_action(notif, "was unblocked") }}
-                {{ utils.display_actor(notif.actor, actors_metadata) }}
             {%- elif notif.notification_type.value == "move" %}
                 {# for move notif, the actor is the target and the inbox object the Move activity #}
                 <div class="actor-action">
@@ -69,8 +54,8 @@
                 {{ notif_actor_action(notif, "shared a post", with_icon=True) }}
                 {{ utils.display_object(notif.outbox_object) }}
            {% elif notif.notification_type.value == "undo_announce" %}
-                {{ notif_actor_action(notif, "unshared a post", with_icon=True) }}
-                {{ utils.display_object(notif.outbox_object) }}
+                {{ notif_actor_action(notif, "unshared a post") }}
+                {{ utils.display_object(notif.outbox_object, with_icon=True) }}
             {% elif notif.notification_type.value == "mention" %}
                 {{ notif_actor_action(notif, "mentioned you") }}
                 {{ utils.display_object(notif.inbox_object) }}

app/templates/object.html

@@ -31,16 +31,9 @@
 {% macro display_replies_tree(replies_tree_node) %}
 
 {% if replies_tree_node.is_requested %}
-{{ utils.display_object(replies_tree_node.ap_object, likes=likes, shares=shares, webmentions=webmentions, expanded=not replies_tree_node.is_root, is_object_page=True, is_h_entry=False) }}
+{{ utils.display_object(replies_tree_node.ap_object, likes=likes, shares=shares, webmentions=webmentions, expanded=not replies_tree_node.is_root, is_object_page=True) }}
 {% else %}
-    {% if replies_tree_node.wm_reply %}
-        {# u-comment h-cite is displayed by default for webmention #}
-        {{ utils.display_webmention_reply(replies_tree_node.wm_reply) }}
-    {% else %}
-        <div class="u-comment h-cite">
-            {{ utils.display_object(replies_tree_node.ap_object, is_h_entry=False) }}
-        </div>
-    {% endif %}
+    {{ utils.display_object(replies_tree_node.ap_object) }}
 {% endif %}
 
 {% for child in replies_tree_node.children %}
@@ -49,8 +42,6 @@
 
 {% endmacro %}
 
-<div class="h-entry">
 {{ display_replies_tree(replies_tree) }}
-</div>
 
 {% endblock %}

app/templates/redirect_to_remote_instance.html

@@ -1,15 +0,0 @@
-{%- import "utils.html" as utils with context -%}
-{% extends "layout.html" %}
-
-{% block head %}
-<title>{{ local_actor.display_name }}'s microblog - Redirect</title>
-{% endblock %}
-
-{% block content %}
-{% include "header.html" %}
-
-<div class="box">
-    <p>You are being redirected to your instance: <a href="{{ url }}">{{ url }}</a></p>
-</div>
-
-{% endblock %}

app/templates/remote_follow.html

@@ -3,7 +3,6 @@
 
 {% block head %}
 <title>Remote follow {{ local_actor.display_name }}</title>
-<meta name="robots" content="noindex, nofollow">
 {% endblock %}
 
 {% block content %}

app/templates/remote_interact.html

@@ -1,27 +0,0 @@
-{%- import "utils.html" as utils with context -%}
-{% extends "layout.html" %}
-
-{% block head %}
-<title>Interact from your instance</title>
-<meta name="robots" content="noindex, nofollow">
-{% endblock %}
-
-{% block content %}
-{% include "header.html" %}
-
-<div class="box">
-    <h2>Interact with this object</h2>
-</div>
-
-{{ utils.display_object(outbox_object) }}
-
-<div class="box">
-    <form class="form" action="{{ url_for("post_remote_interaction") }}" method="POST">
-        {{ utils.embed_csrf_token() }}
-        <input type="text" name="profile" placeholder="you@instance.tld" autofocus>
-        <input type="hidden" name="ap_id" value="{{ outbox_object.ap_id }}">
-        <input type="submit" value="interact from your instance">
-    </form>
-</div>
-
-{% endblock %}

app/templates/utils.html

@@ -1,231 +1,168 @@
 {% macro embed_csrf_token() %}
-{% block embed_csrf_token scoped %}
 <input type="hidden" name="csrf_token" value="{{ csrf_token }}">
-{% endblock %}
 {% endmacro %}
 
 {% macro embed_redirect_url(permalink_id=None) %}
-{% block embed_redirect_url scoped %}
 <input type="hidden" name="redirect_url" value="{{ request.url }}{% if permalink_id %}#{{ permalink_id }}{% endif %}">
-{% endblock %}
 {% endmacro %}
 
 {% macro admin_block_button(actor) %}
-{% block admin_block_button scoped %}
 <form action="{{ request.url_for("admin_actions_block") }}" method="POST">
     {{ embed_csrf_token() }}
     {{ embed_redirect_url() }}
     <input type="hidden" name="ap_actor_id" value="{{ actor.ap_id }}">
     <input type="submit" value="block">
 </form>
-{% endblock %}
 {% endmacro %}
 
 {% macro admin_unblock_button(actor) %}
-{% block admin_unblock_button scoped %}
 <form action="{{ request.url_for("admin_actions_unblock") }}" method="POST">
     {{ embed_csrf_token() }}
     {{ embed_redirect_url() }}
     <input type="hidden" name="ap_actor_id" value="{{ actor.ap_id }}">
     <input type="submit" value="unblock">
 </form>
-{% endblock %}
 {% endmacro %}
 
 {% macro admin_follow_button(actor) %}
-{% block admin_follow_button scoped %}
 <form action="{{ request.url_for("admin_actions_follow") }}" method="POST">
     {{ embed_csrf_token() }}
     {{ embed_redirect_url() }}
     <input type="hidden" name="ap_actor_id" value="{{ actor.ap_id }}">
     <input type="submit" value="follow">
 </form>
-{% endblock %}
 {% endmacro %}
 
 {% macro admin_accept_incoming_follow_button(notif) %}
-{% block admin_accept_incoming_follow_button scoped %}
 <form action="{{ request.url_for("admin_actions_accept_incoming_follow") }}" method="POST">
     {{ embed_csrf_token() }}
     {{ embed_redirect_url() }}
     <input type="hidden" name="notification_id" value="{{ notif.id }}">
     <input type="submit" value="accept follow">
 </form>
-{% endblock %}
 {% endmacro %}
 
 {% macro admin_reject_incoming_follow_button(notif) %}
-{% block admin_reject_incoming_follow_button scoped %}
 <form action="{{ request.url_for("admin_actions_reject_incoming_follow") }}" method="POST">
     {{ embed_csrf_token() }}
     {{ embed_redirect_url() }}
     <input type="hidden" name="notification_id" value="{{ notif.id }}">
     <input type="submit" value="reject follow">
 </form>
-{% endblock %}
 {% endmacro %}
 
 {% macro admin_like_button(ap_object_id, permalink_id) %}
-{% block admin_like_button scoped %}
 <form action="{{ request.url_for("admin_actions_like") }}" method="POST">
     {{ embed_csrf_token() }}
     {{ embed_redirect_url(permalink_id) }}
     <input type="hidden" name="ap_object_id" value="{{ ap_object_id }}">
     <input type="submit" value="like">
 </form>
-{% endblock %}
 {% endmacro %}
 
 {% macro admin_bookmark_button(ap_object_id, permalink_id) %}
-{% block admin_bookmark_button scoped %}
 <form action="{{ request.url_for("admin_actions_bookmark") }}" method="POST">
     {{ embed_csrf_token() }}
     {{ embed_redirect_url(permalink_id) }}
     <input type="hidden" name="ap_object_id" value="{{ ap_object_id }}">
     <input type="submit" value="bookmark">
 </form>
-{% endblock %}
 {% endmacro %}
 
 {% macro admin_unbookmark_button(ap_object_id, permalink_id) %}
-{% block admin_unbookmark_button scoped %}
 <form action="{{ request.url_for("admin_actions_unbookmark") }}" method="POST">
     {{ embed_csrf_token() }}
     {{ embed_redirect_url(permalink_id) }}
     <input type="hidden" name="ap_object_id" value="{{ ap_object_id }}">
     <input type="submit" value="unbookmark">
 </form>
-{% endblock %}
 {% endmacro %}
 
 {% macro admin_pin_button(ap_object_id, permalink_id) %}
-{% block admin_pin_button scoped %}
 <form action="{{ request.url_for("admin_actions_pin") }}" method="POST">
     {{ embed_csrf_token() }}
     {{ embed_redirect_url(permalink_id) }}
     <input type="hidden" name="ap_object_id" value="{{ ap_object_id }}">
     <input type="submit" value="pin">
 </form>
-{% endblock %}
 {% endmacro %}
 
 {% macro admin_unpin_button(ap_object_id, permalink_id) %}
-{% block admin_unpin_button scoped %}
 <form action="{{ request.url_for("admin_actions_unpin") }}" method="POST">
     {{ embed_csrf_token() }}
     {{ embed_redirect_url(permalink_id) }}
     <input type="hidden" name="ap_object_id" value="{{ ap_object_id }}">
     <input type="submit" value="unpin">
 </form>
-{% endblock %}
 {% endmacro %}
 
 {% macro admin_delete_button(ap_object) %}
-{% block admin_delete_button scoped %}
 <form action="{{ request.url_for("admin_actions_delete") }}" class="object-delete-form" method="POST">
     {{ embed_csrf_token() }}
     <input type="hidden" name="redirect_url" value="{% if request.url.path.endswith("/" + ap_object.public_id) or (request.url.path == "/admin/object" and request.query_params.ap_id.endswith("/" + ap_object.public_id)) %}{{ request.base_url}}{% else %}{{ request.url }}{% endif %}">
     <input type="hidden" name="ap_object_id" value="{{ ap_object.ap_id }}">
     <input type="submit" value="delete">
 </form>
-{% endblock %}
-{% endmacro %}
-
-{% macro admin_force_delete_button(ap_object_id, permalink_id=None) %}
-{% block admin_force_delete_button scoped %}
-<form action="{{ request.url_for("admin_actions_force_delete") }}" class="object-delete-form" method="POST">
-    {{ embed_csrf_token() }}
-    {{ embed_redirect_url(permalink_id) }}
-    <input type="hidden" name="ap_object_id" value="{{ ap_object_id }}">
-    <input type="submit" value="local delete">
-</form>
-{% endblock %}
-{% endmacro %}
-
-{% macro admin_force_delete_webmention_button(webmention_id, permalink_id=None) %}
-{% block admin_force_delete_webmention_button scoped %}
-<form action="{{ request.url_for("admin_actions_force_delete_webmention") }}" class="object-delete-form" method="POST">
-    {{ embed_csrf_token() }}
-    {{ embed_redirect_url(permalink_id) }}
-    <input type="hidden" name="webmention_id" value="{{ webmention_id }}">
-    <input type="submit" value="local delete">
-</form>
-{% endblock %}
 {% endmacro %}
 
 {% macro admin_announce_button(ap_object_id, permalink_id=None) %}
-{% block admin_announce_button scoped %}
 <form action="{{ request.url_for("admin_actions_announce") }}" method="POST">
     {{ embed_csrf_token() }}
     {{ embed_redirect_url(permalink_id) }}
     <input type="hidden" name="ap_object_id" value="{{ ap_object_id }}">
     <input type="submit" value="share">
 </form>
-{% endblock %}
 {% endmacro %}
 
 {% macro admin_undo_button(ap_object_id, action="undo", permalink_id=None) %}
-{% block admin_undo_button scoped %}
 <form action="{{ request.url_for("admin_actions_undo") }}" method="POST">
     {{ embed_csrf_token() }}
     {{ embed_redirect_url(permalink_id) }}
     <input type="hidden" name="ap_object_id" value="{{ ap_object_id }}">
     <input type="submit" value="{{ action }}">
 </form>
-{% endblock %}
 {% endmacro %}
 
 {% macro admin_reply_button(ap_object_id) %}
-{% block admin_reply_button scoped %}
-<form action="{{ BASE_URL }}/admin/new"  method="GET">
+<form action="/admin/new"  method="GET">
 <input type="hidden" name="in_reply_to" value="{{ ap_object_id }}">
 <button type="submit">reply</button>
 </form> 
-{% endblock %}
 {% endmacro %}
 
 {% macro admin_dm_button(actor_handle) %}
-{% block admin_dm_button scoped %}
-<form action="{{ BASE_URL }}/admin/new"  method="GET">
+<form action="/admin/new"  method="GET">
 <input type="hidden" name="with_content" value="{{ actor_handle }}">
 <input type="hidden" name="with_visibility" value="DIRECT">
 <button type="submit">direct message</button>
 </form> 
-{% endblock %}
 {% endmacro %}
 
 {% macro admin_mention_button(actor_handle) %}
-{% block admin_mention_button scoped %}
-<form action="{{ BASE_URL }}/admin/new"  method="GET">
+<form action="/admin/new"  method="GET">
 <input type="hidden" name="with_content" value="{{ actor_handle }}">
 <button type="submit">mention</button>
 </form> 
-{% endblock %}
 {% endmacro %}
 
 
 
 {% macro admin_profile_button(ap_actor_id) %}
-{% block admin_profile_button scoped %}
 <form action="{{ url_for("admin_profile") }}"  method="GET">
 <input type="hidden" name="actor_id" value="{{ ap_actor_id }}">
 <button type="submit">profile</button>
 </form> 
-{% endblock %}
 {% endmacro %}
 
 {% macro admin_expand_button(ap_object) %}
-{% block admin_expand_button scoped %}
 {# TODO turn these into a regular link and append permalink ID if it's a reply #}
 <form action="{{ url_for("admin_object") }}"  method="GET">
 <input type="hidden" name="ap_id" value="{{ ap_object.ap_id }}">
 <button type="submit">expand</button>
 </form> 
-{% endblock %}
 {% endmacro %}
 
 {% macro display_box_filters(route) %}
-{% block display_box_filters scoped %}
 <nav class="flexbox box">
 <ul>
     <li>Filter by</li>
@@ -242,17 +179,13 @@
 {% endif %}
 </ul>
 </nav>
-{% endblock %}
 {% endmacro %}
 
 {% macro display_tiny_actor_icon(actor) %}
-{% block display_tiny_actor_icon scoped %}
-    <img class="tiny-actor-icon" src="{{ actor.resized_icon_url }}" alt="">
-{% endblock %}
+    <img class="tiny-actor-icon" src="{{ actor.resized_icon_url }}" alt="{{ actor.display_name }}'s avatar">
 {% endmacro %}
 
 {% macro actor_action(inbox_object, text, with_icon=False) %}
-{% block actor_action scoped %}
     <div class="actor-action">
         <a href="{{ url_for("admin_profile") }}?actor_id={{ inbox_object.actor.ap_id }}">
             {% if with_icon %}{{ display_tiny_actor_icon(inbox_object.actor) }}{% endif %} {{ inbox_object.actor.display_name | clean_html(inbox_object.actor) | safe }}
@@ -260,11 +193,9 @@
         <span title="{{ inbox_object.ap_published_at.isoformat() }}">{{ inbox_object.ap_published_at | timeago }}</span>
     </div>
 
-{% endblock %}
 {% endmacro %}
 
 {% macro display_actor(actor, actors_metadata={}, embedded=False, with_details=False, pending_incoming_follow_notif=None) %}
-{% block display_actor scoped %}
 {% set metadata = actors_metadata.get(actor.ap_id) %}
 
 {% if not embedded %}
@@ -285,25 +216,13 @@
 <div>
     <nav class="flexbox actor-metadata">
         <ul>
-            {% if metadata.has_blocked_local_actor %}
-                <li>blocked you</li>
-            {% endif %}
             {% if metadata.is_following %}
                 <li>already following</li>
                 <li>{{ admin_undo_button(metadata.outbox_follow_ap_id, "unfollow")}}</li>
-                {% if not with_details %}
-                    <li>{{ admin_profile_button(actor.ap_id) }}</li>
-                {% endif %}
+                <li>{{ admin_profile_button(actor.ap_id) }}</li>
             {% elif metadata.is_follow_request_sent %}
-                {% if metadata.is_follow_request_rejected %}
-                    <li>follow request rejected</li>
-                    {% if not metadata.has_blocked_local_actor %}
-                        <li>{{ admin_follow_button(actor) }}</li>
-                    {% endif %}
-                {% else %}
-                    <li>follow request sent</li>
-                    <li>{{ admin_undo_button(metadata.outbox_follow_ap_id, "undo follow") }}</li>
-                {% endif %}
+                <li>follow request sent</li>
+                <li>{{ admin_undo_button(metadata.outbox_follow_ap_id, "undo follow") }}</li>
             {% elif not actor.moved_to %}
                 <li>{{ admin_follow_button(actor) }}</li>
             {% endif %}
@@ -312,7 +231,7 @@
                 {% if not metadata.is_following and not with_details %}
                     <li>{{ admin_profile_button(actor.ap_id) }}</li>
                 {% endif %}
-            {% elif actor.is_from_db and not with_details and not metadata.is_following %}
+            {% elif actor.is_from_db and not with_details %}
                 <li>{{ admin_profile_button(actor.ap_id) }}</li>
             {% endif %}
             {% if actor.moved_to %}
@@ -342,9 +261,6 @@
                     <li>rejected</li>
                 {% endif %}
             {% endif %}
-            {% if with_details %}
-                <li><a href="{{ actor.url }}" class="label-btn">remote profile</a></li>
-            {% endif %}
         </ul>
     </nav>
 </div>
@@ -375,11 +291,9 @@
 </div>
 {% endif %}
 
-{% endblock %}
 {% endmacro %}
 
 {% macro display_og_meta(object) %}
-{% block display_og_meta scoped %}
 {% if object.og_meta %}
 {% for og_meta in object.og_meta[:1] %}
 <div class="activity-og-meta">
@@ -397,52 +311,38 @@
 </div>
 {% endfor %}
 {% endif %}
-{% endblock %}
 {% endmacro %}
 
 
 {% macro display_attachments(object) %}
-{% block display_attachments scoped %}
 
     {% for attachment in object.attachments %}
-    {% if attachment.type != "PropertyValue" %}
-    {% set orientation = "unknown" %}
-    {% if attachment.width %}
-        {% set orientation = "portrait" if attachment.width < attachment.height else "landscape" %}
-    {% endif %}
     {% if object.sensitive and (attachment.type == "Image" or (attachment | has_media_type("image")) or attachment.type == "Video" or (attachment | has_media_type("video"))) %}
     <div class="attachment-wrapper">
         <label for="{{attachment.proxied_url}}" class="label-btn show-hide-sensitive-btn">show/hide sensitive content</label>
         <div>
         <div class="sensitive-attachment">
         <input class="sensitive-attachment-state" type="checkbox" id="{{attachment.proxied_url}}" aria-hidden="true">
-        <div class="sensitive-attachment-box attachment-orientation-{{orientation}}">
+        <div class="sensitive-attachment-box">
             <div></div>
     {% else %}
-            <div class="attachment-item attachment-orientation-{{orientation}}">
+            <div class="attachment-item">
     {% endif %}
 
     {% if attachment.type == "Image" or (attachment | has_media_type("image")) %}
         {% if attachment.url not in object.inlined_images %}
-        <a class="media-link" href="{{ attachment.proxied_url }}" target="_blank">
-            <img src="{{ attachment.resized_url or attachment.proxied_url }}"{% if attachment.name %} title="{{ attachment.name }}" alt="{{ attachment.name }}"{% endif %} class="attachment u-photo">
-        </a>
+            <img src="{{ attachment.resized_url or attachment.proxied_url }}"{% if attachment.name %} title="{{ attachment.name }}" alt="{{ attachment.name }}"{% endif %} class="attachment">
         {% endif %}
     {% elif attachment.type == "Video" or (attachment | has_media_type("video")) %}
-    <div class="video-wrapper">
-    <video controls preload="metadata" src="{{ attachment.url | media_proxy_url }}"{% if attachment.name %} title="{{ attachment.name }}"{% endif %} class="u-video"></video>
-    <div class="video-gif-overlay">GIF</div>
-    </div>
+    <video controls preload="metadata"  src="{{ attachment.url | media_proxy_url }}"{% if attachment.name %} title="{{ attachment.name }}"{% endif %}></video>
     {% elif attachment.type == "Audio" or (attachment | has_media_type("audio")) %}
-    <audio controls preload="metadata"  src="{{ attachment.url | media_proxy_url }}"{% if attachment.name%} title="{{ attachment.name }}"{% endif %} class="attachment u-audio"></audio>
+    <audio controls preload="metadata"  src="{{ attachment.url | media_proxy_url }}"{% if attachment.name%} title="{{ attachment.name }}"{% endif %} class="attachment"></audio>
     {% elif attachment.type == "Link" %}
-    <a href="{{ attachment.url }}" class="attachment">{{ attachment.url | truncate(64, True) }}</a> ({{ attachment.mimetype}})
+        <a href="{{ attachment.url }}" class="attachment">{{ attachment.url }}</a>
     {% else %}
-    <a href="{{ attachment.url | media_proxy_url }}"{% if attachment.name %} title="{{ attachment.url }}"{% endif %} class="attachment">
-        {% if attachment.name %}{{ attachment.name }}{% else %}{{ attachment.url | truncate(64, True) }}{% endif %}
-    </a> ({{ attachment.mimetype }})
+    <a href="{{ attachment.url | media_proxy_url }}"{% if attachment.name %} title="{{ attachment.name }}"{% endif %} class="attachment">{{ attachment.url }}</a>
     {% endif %}
-    {% if object.sensitive and (attachment.type == "Image" or (attachment | has_media_type("image")) or attachment.type == "Video" or (attachment | has_media_type("video"))) %}
+    {% if object.sensitive %}
             </div>
         </div>
         </div>
@@ -450,60 +350,13 @@
     {% else %}
         </div>
     {% endif %}
-    {% endif %}
     {% endfor %}
-{% endblock %}
 {% endmacro %}
 
-{% macro display_webmention_reply(wm_reply) %}
-{% block display_webmention_reply scoped %}
-
-<div class="ap-object u-comment h-cite">
-<div class="actor-box h-card p-author">
-    <div class="icon-box">
-        <img src="{{ wm_reply.face.picture_url }}" alt="{{ wm_reply.face.name }}'s avatar" class="actor-icon u-photo">
-    </div>
-    <a href="{{ wm_reply.face.url }}" class="u-url">
-        <div><strong class="p-name">{{ wm_reply.face.name | clean_html_wm | safe }}</strong></div>
-        <div class="actor-handle">{{ wm_reply.face.url | truncate(64, True) }}</div>
-    </a>
-</div>
-
-    <p class="in-reply-to">in reply to <a href="{{ wm_reply.in_reply_to }}" title="{{ wm_reply.in_reply_to }}" rel="nofollow">
-        this object
-    </a></p>
-
-    <div class="obj-content margin-top-20">
-    <div class="e-content">
-        {{ wm_reply.content | clean_html_wm | safe }}
-    </div>
-    </div>
-
-    <nav class="flexbox activity-bar margin-top-20">
-    <ul>
-        <li>
-            <div><a href="{{ wm_reply.url }}" rel="nofollow" class="object-permalink u-url u-uid">permalink</a></div>
-        </li>
-        <li>
-            <time class="dt-published" datetime="{{ wm_reply.published_at.replace(microsecond=0).isoformat() }}" title="{{ wm_reply.published_at.replace(microsecond=0).isoformat() }}">{{ wm_reply.published_at | timeago }}</time>
-        </li>
-        {% if is_admin %}
-            <li>
-                {{ admin_force_delete_webmention_button(wm_reply.webmention_id) }}
-            </li>
-        {% endif %}
-    </ul>
-    </nav>
-</div>
-
-{% endblock %}
-{% endmacro %}
-
-{% macro display_object(object, likes=[], shares=[], webmentions=[], expanded=False, actors_metadata={}, is_object_page=False, is_h_entry=True) %}
-{% block display_object scoped %}
+{% macro display_object(object, likes=[], shares=[], webmentions=[], expanded=False, actors_metadata={}, is_object_page=False) %}
 {% set is_article_mode = object.is_from_outbox and object.ap_type == "Article" and is_object_page %}
-{% if object.ap_type in ["Note", "Article", "Video", "Page", "Question", "Event"] %}
-<div class="ap-object {% if expanded %}ap-object-expanded {% endif %}{% if is_h_entry %}h-entry{% endif %}" id="{{ object.permalink_id }}">
+{% if object.ap_type in ["Note", "Article", "Video", "Page", "Question"] %}
+<div class="ap-object {% if expanded %}ap-object-expanded {% endif %}h-entry" id="{{ object.permalink_id }}">
 
     {% if is_article_mode %}
     <data class="h-card">
@@ -516,55 +369,39 @@
     {% endif %}
 
     {% if object.in_reply_to %}
-    <p class="in-reply-to">in reply to <a href="{% if is_admin and object.is_in_reply_to_from_inbox %}{{ url_for("get_lookup") }}?query={% endif %}{{ object.in_reply_to }}" title="{{ object.in_reply_to }}" rel="nofollow">
-            this object
-        </a></p>
+    <a href="{% if is_admin and object.is_in_reply_to_from_inbox %}{{ url_for("get_lookup") }}?query={% endif %}{{ object.in_reply_to }}" title="{{ object.in_reply_to }}" class="in-reply-to" rel="nofollow">
+            in reply to {{ object.in_reply_to|truncate(64, True) }}
+        </a>
     {% endif %}
 
-    {% if object.ap_type in ["Article", "Event"] %}
+    {% if object.ap_type == "Article" %}
         <h2 class="p-name no-margin-top">{{ object.name }}</h2>
     {% endif %}
 
-    {% if object.ap_type == "Event" %}
-        {% if object.ap_object.get("endTime") and object.ap_object.get("startTime") %}
-            <p>On {{ object.ap_object.startTime | parse_datetime | format_date }}
-            (ends {{ object.ap_object.endTime | parse_datetime | format_date }})</p>
-        {% endif %}
-    {% endif %}
-
-    {% if object.ap_object.get("location") %}
-        {% set loc = object.ap_object.get("location") %}
-        {% if loc.type == "Place" and loc.latitude and loc.longitude %}
-        <div class="ap-place">
-            <h3>Location</h3>
-            {% if loc.name %}{{ loc.name }}{% endif %}
-            <span class="h-geo">
-                <data class="p-latitude" value="{{ loc.latitude}}"></data>
-                <data class="p-longitude" value="{{ loc.longitude }}"></data>
-                <a href="https://www.openstreetmap.org/?mlat={{ loc.latitude }}&mlon={{ loc.longitude }}#map=16/{{loc.latitude}}/{{loc.longitude}}">{{loc.latitude}},{{loc.longitude}}</a>
-            </span>
-        </div>
-        {% endif %}
-    {% endif %}
-
     {% if is_article_mode %}
                 <time class="dt-published muted" datetime="{{ object.ap_published_at.replace(microsecond=0).isoformat() }}" title="{{ object.ap_published_at.replace(microsecond=0).isoformat() }}">{{ object.ap_published_at.strftime("%b %d, %Y") }}</time>
     {% endif %}
 
     {% if object.summary %}
-    <details class="show-more-wrapper">
-        <summary>
-            <div class="p-summary">
-                <p>{{ object.summary | clean_html(object) | safe }}</p>
-            </div>
-            <span class="show-more-btn" aria-hidden="true"></span>
-        </summary>
+    <div class="show-more-wrapper">
+        <div class="p-summary">
+            <p>{{ object.summary | clean_html(object) | safe }}</p>
+        </div>
+        <label for="show-more-{{ object.permalink_id }}" class="show-more-btn">show/hide more</label>
+        <input class="show-more-state" type="checkbox" aria-hidden="true" id="show-more-{{ object.permalink_id }}" checked>
     {% endif %}
         <div class="obj-content">
         <div class="e-content">
             {{ object.content | clean_html(object) | safe }}
         </div>
 
+    {% if object.quoted_object %}
+        <div class="ap-object-expanded ap-quoted-object">
+        {{ display_object(object.quoted_object) }}
+        </div>
+    {% endif %}
+
+
     {% if object.ap_type == "Question" %}
         {% set can_vote = is_admin and object.is_from_inbox and not object.is_poll_ended and not object.voted_for_answers %}
         {% if can_vote %}
@@ -619,7 +456,7 @@
 
         </div>
     {% if object.summary %}
-    </details>
+    </div>
     {% endif %}
 
     <div class="activity-attachment">
@@ -631,16 +468,6 @@
         <li>
             <div><a href="{{ object.url }}"{% if object.is_from_inbox %} rel="nofollow"{% endif %} class="object-permalink u-url u-uid">permalink</a></div>
         </li>
-
-        {% if object.is_from_outbox and is_object_page and not is_admin and not request.url.path.startswith("/remote_interaction") %}
-            <li>
-                <a class="label-btn" href="{{ request.url_for("remote_interaction") }}?ap_id={{ object.ap_id }}">
-                    interact from your instance
-                </a>
-            </li>
-        {% endif %}
-
-
         {% if not is_article_mode %}
             <li>
                 <time class="dt-published" datetime="{{ object.ap_published_at.replace(microsecond=0).isoformat() }}" title="{{ object.ap_published_at.replace(microsecond=0).isoformat() }}">{{ object.ap_published_at | timeago }}</time>
@@ -736,7 +563,7 @@
             {% if object.visibility in [visibility_enum.PUBLIC, visibility_enum.UNLISTED] %}
             <li>
                 {% if object.announced_via_outbox_object_ap_id %}
-                    {{ admin_undo_button(object.announced_via_outbox_object_ap_id, "unshare") }}
+                    {{ admin_undo_button(object.liked_via_outbox_object_ap_id, "unshare") }}
                 {% else %}
                     {{ admin_announce_button(object.ap_id, permalink_id=object.permalink_id) }}
                 {% endif %}
@@ -754,11 +581,6 @@
                 {{ admin_expand_button(object) }}
             </li>
         {% endif %}
-        {% if object.is_from_inbox and not object.announced_via_outbox_object_ap_id and object.is_local_reply %}
-            <li>
-                {{ admin_force_delete_button(object.ap_id) }}
-            </li>
-        {% endif %}
     </ul>
     </nav>
     {% endif %} 
@@ -770,8 +592,8 @@
             <div class="interactions-block">Likes
                 <div class="facepile-wrapper">
                 {% for like in likes %}
-                    <a href="{% if is_admin and like.ap_actor_id %}{{ url_for("admin_profile") }}?actor_id={{ like.ap_actor_id }}{% else %}{{ like.url }}{% endif %}" title="{{ like.name }}" rel="noreferrer">
-                        <img src="{{ like.picture_url }}" alt="{{ like.name }}">
+                    <a href="{% if is_admin %}{{ url_for("admin_profile") }}?actor_id={{ like.actor.ap_id }}{% else %}{{ like.actor.url }}{% endif %}" title="{{ like.actor.handle }}" rel="noreferrer">
+                        <img src="{{ like.actor.resized_icon_url }}" alt="{{ like.actor.handle}}">
                     </a>
                 {% endfor %}
                 {% if object.likes_count > likes | length %}
@@ -787,8 +609,8 @@
             <div class="interactions-block">Shares
                 <div class="facepile-wrapper">
                 {% for share in shares %}
-                    <a href="{% if is_admin and share.ap_actor_id %}{{ url_for("admin_profile") }}?actor_id={{ share.ap_actor_id }}{% else %}{{ share.url }}{% endif %}" title="{{ share.name }}" rel="noreferrer">
-                        <img src="{{ share.picture_url }}" alt="{{ share.name }}">
+                    <a href="{% if is_admin %}{{ url_for("admin_profile") }}?actor_id={{ share.actor.ap_id }}{% else %}{{ share.actor.url }}{% endif %}" title="{{ share.actor.handle }}" rel="noreferrer">
+                        <img src="{{ share.actor.resized_icon_url }}" alt="{{ share.actor.handle}}">
                     </a>
                 {% endfor %}
                 {% if object.announces_count > shares | length %}
@@ -821,5 +643,4 @@
 
 </div>
 {% endif %}
-{% endblock %}
 {% endmacro %}

app/utils/custom_index_handler.py

@@ -1,32 +0,0 @@
-from typing import Any
-from typing import Awaitable
-from typing import Callable
-
-from fastapi import Depends
-from fastapi import Request
-from fastapi.responses import JSONResponse
-
-from app.actor import LOCAL_ACTOR
-from app.config import is_activitypub_requested
-from app.database import AsyncSession
-from app.database import get_db_session
-
-_Handler = Callable[[Request, AsyncSession], Awaitable[Any]]
-
-
-def build_custom_index_handler(handler: _Handler) -> _Handler:
-    async def custom_index(
-        request: Request,
-        db_session: AsyncSession = Depends(get_db_session),
-    ) -> Any:
-        # Serve the AP actor if requested
-        if is_activitypub_requested(request):
-            return JSONResponse(
-                LOCAL_ACTOR.ap_actor,
-                media_type="application/activity+json",
-            )
-
-        # Defer to the custom handler
-        return await handler(request, db_session)
-
-    return custom_index

app/utils/emoji.py

@@ -23,8 +23,6 @@ def _load_emojis(root_dir: Path, base_url: str) -> None:
             mt = mimetypes.guess_type(emoji.name)[0]
             if mt and mt.startswith("image/"):
                 name = emoji.name.split(".")[0]
-                if not re.match(EMOJI_REGEX, f":{name}:"):
-                    continue
                 ap_emoji: "RawObject" = {
                     "type": "Emoji",
                     "name": f":{name}:",

app/utils/facepile.py

@@ -1,159 +0,0 @@
-import datetime
-from dataclasses import dataclass
-from typing import Any
-from typing import Optional
-
-from loguru import logger
-
-from app import media
-from app.models import InboxObject
-from app.models import Webmention
-from app.utils.datetime import parse_isoformat
-from app.utils.url import make_abs
-
-
-@dataclass
-class Face:
-    ap_actor_id: str | None
-    url: str
-    name: str
-    picture_url: str
-    created_at: datetime.datetime
-
-    @classmethod
-    def from_inbox_object(cls, like: InboxObject) -> "Face":
-        return cls(
-            ap_actor_id=like.actor.ap_id,
-            url=like.actor.url,  # type: ignore
-            name=like.actor.handle,  # type: ignore
-            picture_url=like.actor.resized_icon_url,
-            created_at=like.created_at,  # type: ignore
-        )
-
-    @classmethod
-    def from_webmention(cls, webmention: Webmention) -> Optional["Face"]:
-        items = webmention.source_microformats.get("items", [])  # type: ignore
-        for item in items:
-            if item["type"][0] == "h-card":
-                try:
-                    return cls(
-                        ap_actor_id=None,
-                        url=(
-                            item["properties"]["url"][0]
-                            if item["properties"].get("url")
-                            else webmention.source
-                        ),
-                        name=item["properties"]["name"][0],
-                        picture_url=media.resized_media_url(
-                            make_abs(
-                                item["properties"]["photo"][0], webmention.source
-                            ),  # type: ignore
-                            50,
-                        ),
-                        created_at=webmention.created_at,  # type: ignore
-                    )
-                except Exception:
-                    logger.exception(
-                        f"Failed to build Face for webmention id={webmention.id}"
-                    )
-                    break
-            elif item["type"][0] == "h-entry":
-                author = item["properties"]["author"][0]
-                try:
-                    return cls(
-                        ap_actor_id=None,
-                        url=webmention.source,
-                        name=author["properties"]["name"][0],
-                        picture_url=media.resized_media_url(
-                            make_abs(
-                                author["properties"]["photo"][0], webmention.source
-                            ),  # type: ignore
-                            50,
-                        ),
-                        created_at=webmention.created_at,  # type: ignore
-                    )
-                except Exception:
-                    logger.exception(
-                        f"Failed to build Face for webmention id={webmention.id}"
-                    )
-                    break
-
-        return None
-
-
-def merge_faces(faces: list[Face]) -> list[Face]:
-    return sorted(
-        faces,
-        key=lambda f: f.created_at,
-        reverse=True,
-    )[:10]
-
-
-def _parse_face(webmention: Webmention, items: list[dict[str, Any]]) -> Face | None:
-    for item in items:
-        if item["type"][0] == "h-card":
-            try:
-                return Face(
-                    ap_actor_id=None,
-                    url=(
-                        item["properties"]["url"][0]
-                        if item["properties"].get("url")
-                        else webmention.source
-                    ),
-                    name=item["properties"]["name"][0],
-                    picture_url=media.resized_media_url(
-                        make_abs(
-                            item["properties"]["photo"][0], webmention.source
-                        ),  # type: ignore
-                        50,
-                    ),
-                    created_at=webmention.created_at,  # type: ignore
-                )
-            except Exception:
-                logger.exception(
-                    f"Failed to build Face for webmention id={webmention.id}"
-                )
-                break
-
-    return None
-
-
-@dataclass
-class WebmentionReply:
-    face: Face
-    content: str
-    url: str
-    published_at: datetime.datetime
-    in_reply_to: str
-    webmention_id: int
-
-    @classmethod
-    def from_webmention(cls, webmention: Webmention) -> Optional["WebmentionReply"]:
-        items = webmention.source_microformats.get("items", [])  # type: ignore
-        for item in items:
-            if item["type"][0] == "h-entry":
-                try:
-                    face = _parse_face(webmention, item["properties"].get("author", []))
-                    if not face:
-                        logger.info(
-                            "Failed to build WebmentionReply/Face for "
-                            f"webmention id={webmention.id}"
-                        )
-                        break
-                    return cls(
-                        face=face,
-                        content=item["properties"]["content"][0]["html"],
-                        url=item["properties"]["url"][0],
-                        published_at=parse_isoformat(
-                            item["properties"]["published"][0]
-                        ).replace(tzinfo=None),
-                        in_reply_to=webmention.target,  # type: ignore
-                        webmention_id=webmention.id,  # type: ignore
-                    )
-                except Exception:
-                    logger.exception(
-                        f"Failed to build Face for webmention id={webmention.id}"
-                    )
-                    break
-
-        return None

app/utils/opengraph.py

@@ -1,18 +1,14 @@
 import asyncio
 import mimetypes
 import re
-import signal
-from concurrent.futures import TimeoutError
 from typing import Any
 from urllib.parse import urlparse
 
 import httpx
 from bs4 import BeautifulSoup  # type: ignore
 from loguru import logger
-from pebble import concurrent  # type: ignore
 from pydantic import BaseModel
 
-from app import activitypub as ap
 from app import ap_object
 from app import config
 from app.actor import LOCAL_ACTOR
@@ -32,11 +28,7 @@ class OpenGraphMeta(BaseModel):
     site_name: str
 
 
-@concurrent.process(timeout=5)
 def _scrap_og_meta(url: str, html: str) -> OpenGraphMeta | None:
-    # Prevent SIGTERM to bubble up to the worker
-    signal.signal(signal.SIGTERM, signal.SIG_IGN)
-
     soup = BeautifulSoup(html, "html5lib")
     ogs = {
         og.attrs["property"]: og.attrs.get("content")
@@ -65,10 +57,6 @@ def _scrap_og_meta(url: str, html: str) -> OpenGraphMeta | None:
     return OpenGraphMeta.parse_obj(raw)
 
 
-def scrap_og_meta(url: str, html: str) -> OpenGraphMeta | None:
-    return _scrap_og_meta(url, html).result()
-
-
 async def external_urls(
     db_session: AsyncSession,
     ro: ap_object.RemoteObject | OutboxObject | InboxObject,
@@ -78,15 +66,11 @@ async def external_urls(
     tags_hrefs = set()
     for tag in ro.tags:
         if tag_href := tag.get("href"):
-            tags_hrefs.add(tag_href)
+            if tag_href and tag_href not in filter(None, [ro.quote_url]):
+                tags_hrefs.add(tag_href)
         if tag.get("type") == "Mention":
             if tag["href"] != LOCAL_ACTOR.ap_id:
-                try:
-                    mentioned_actor = await fetch_actor(db_session, tag["href"])
-                except (ap.FetchError, ap.NotAnObjectError):
-                    tags_hrefs.add(tag["href"])
-                    continue
-
+                mentioned_actor = await fetch_actor(db_session, tag["href"])
                 tags_hrefs.add(mentioned_actor.url)
                 tags_hrefs.add(mentioned_actor.ap_id)
             else:
@@ -101,22 +85,18 @@ async def external_urls(
             if not h:
                 continue
 
-            try:
-                ph = urlparse(h)
-                mimetype, _ = mimetypes.guess_type(h)
-                if (
-                    ph.scheme in {"http", "https"}
-                    and ph.hostname != note_host
-                    and is_url_valid(h)
-                    and (
-                        not mimetype
-                        or mimetype.split("/")[0] not in ["image", "video", "audio"]
-                    )
-                ):
-                    urls.add(h)
-            except Exception:
-                logger.exception(f"Failed to check {h}")
-                continue
+            ph = urlparse(h)
+            mimetype, _ = mimetypes.guess_type(h)
+            if (
+                ph.scheme in {"http", "https"}
+                and ph.hostname != note_host
+                and is_url_valid(h)
+                and (
+                    not mimetype
+                    or mimetype.split("/")[0] not in ["image", "video", "audio"]
+                )
+            ):
+                urls.add(h)
 
     return urls - tags_hrefs
 
@@ -137,10 +117,7 @@ async def _og_meta_from_url(url: str) -> OpenGraphMeta | None:
         return None
 
     try:
-        return scrap_og_meta(url, resp.text)
-    except TimeoutError:
-        logger.info(f"Timed out when scraping OG meta for {url}")
-        return None
+        return _scrap_og_meta(url, resp.text)
     except Exception:
         logger.info(f"Failed to scrap OG meta for {url}")
         return None

app/utils/text.py

@@ -1,8 +0,0 @@
-import re
-import unicodedata
-
-
-def slugify(text: str) -> str:
-    value = unicodedata.normalize("NFKC", text)
-    value = re.sub(r"[^\w\s-]", "", value.lower())
-    return re.sub(r"[-\s]+", "-", value).strip("-_")

app/utils/url.py

@@ -54,14 +54,10 @@ def is_url_valid(url: str) -> bool:
     if not parsed.hostname or parsed.hostname.lower() in ["localhost"]:
         return False
 
-    if is_hostname_blocked(parsed.hostname):
+    if parsed.hostname in BLOCKED_SERVERS:
         logger.warning(f"{parsed.hostname} is blocked")
         return False
 
-    if parsed.hostname.endswith(".onion"):
-        logger.warning(f"{url} is an onion service")
-        return False
-
     ip_address = _getaddrinfo(
         parsed.hostname, parsed.port or (80 if parsed.scheme == "http" else 443)
     )
@@ -81,11 +77,3 @@ def check_url(url: str) -> None:
         raise InvalidURLError(f'"{url}" is invalid')
 
     return None
-
-
-@functools.lru_cache(maxsize=256)
-def is_hostname_blocked(hostname: str) -> bool:
-    for blocked_hostname in BLOCKED_SERVERS:
-        if hostname == blocked_hostname or hostname.endswith(f".{blocked_hostname}"):
-            return True
-    return False

app/utils/webmentions.py

@@ -24,7 +24,7 @@ async def _discover_webmention_endoint(url: str) -> str | None:
                 follow_redirects=True,
             )
             resp.raise_for_status()
-        except Exception:
+        except (httpx.HTTPError, httpx.HTTPStatusError):
             logger.exception(f"Failed to discover webmention endpoint for {url}")
             return None
 

app/utils/workers.py

@@ -69,5 +69,5 @@ class Worker(Generic[T]):
         logger.info("stopping loop")
 
     async def _shutdown(self, sig: signal.Signals) -> None:
-        logger.info(f"Caught {sig=}")
+        logger.info(f"Caught {signal=}")
         self._stop_event.set()

app/webfinger.py

@@ -12,7 +12,6 @@ async def webfinger(
     resource: str,
 ) -> dict[str, Any] | None:  # noqa: C901
     """Mastodon-like WebFinger resolution to retrieve the activity stream Actor URL."""
-    resource = resource.strip()
     logger.info(f"performing webfinger resolution for {resource}")
     protos = ["https", "http"]
     if resource.startswith("http://"):

app/webmentions.py

@@ -1,5 +1,3 @@
-from urllib.parse import urlparse
-
 import httpx
 from bs4 import BeautifulSoup  # type: ignore
 from fastapi import APIRouter
@@ -8,21 +6,13 @@ from fastapi import HTTPException
 from fastapi import Request
 from fastapi.responses import JSONResponse
 from loguru import logger
-from sqlalchemy import func
 from sqlalchemy import select
 
 from app import models
-from app.boxes import _get_outbox_announces_count
-from app.boxes import _get_outbox_likes_count
-from app.boxes import _get_outbox_replies_count
 from app.boxes import get_outbox_object_by_ap_id
-from app.boxes import get_outbox_object_by_slug_and_short_id
-from app.boxes import is_notification_enabled
 from app.database import AsyncSession
 from app.database import get_db_session
 from app.utils import microformats
-from app.utils.facepile import Face
-from app.utils.facepile import WebmentionReply
 from app.utils.url import check_url
 from app.utils.url import is_url_valid
 
@@ -57,7 +47,6 @@ async def webmention_endpoint(
 
         check_url(source)
         check_url(target)
-        parsed_target_url = urlparse(target)
     except Exception:
         logger.exception("Invalid webmention request")
         raise HTTPException(status_code=400, detail="Invalid payload")
@@ -76,16 +65,6 @@ async def webmention_endpoint(
         logger.info("Found existing Webmention, will try to update or delete")
 
     mentioned_object = await get_outbox_object_by_ap_id(db_session, target)
-
-    if not mentioned_object and parsed_target_url.path.startswith("/articles/"):
-        try:
-            _, _, short_id, slug = parsed_target_url.path.split("/")
-            mentioned_object = await get_outbox_object_by_slug_and_short_id(
-                db_session, slug, short_id
-            )
-        except Exception:
-            logger.exception(f"Failed to match {target}")
-
     if not mentioned_object:
         logger.info(f"Invalid target {target=}")
 
@@ -111,21 +90,15 @@ async def webmention_endpoint(
         logger.warning(f"target {target=} not found in source")
         if existing_webmention_in_db:
             logger.info("Deleting existing Webmention")
+            mentioned_object.webmentions_count = mentioned_object.webmentions_count - 1
             existing_webmention_in_db.is_deleted = True
-            await db_session.flush()
 
-            # Revert side effects
-            await _handle_webmention_side_effects(
-                db_session, existing_webmention_in_db, mentioned_object
+            notif = models.Notification(
+                notification_type=models.NotificationType.DELETED_WEBMENTION,
+                outbox_object_id=mentioned_object.id,
+                webmention_id=existing_webmention_in_db.id,
             )
-
-            if is_notification_enabled(models.NotificationType.DELETED_WEBMENTION):
-                notif = models.Notification(
-                    notification_type=models.NotificationType.DELETED_WEBMENTION,
-                    outbox_object_id=mentioned_object.id,
-                    webmention_id=existing_webmention_in_db.id,
-                )
-                db_session.add(notif)
+            db_session.add(notif)
 
             await db_session.commit()
 
@@ -137,96 +110,36 @@ async def webmention_endpoint(
         else:
             return JSONResponse(content={}, status_code=200)
 
-    webmention_type = models.WebmentionType.UNKNOWN
-    webmention: models.Webmention
     if existing_webmention_in_db:
         # Undelete if needed
         existing_webmention_in_db.is_deleted = False
         existing_webmention_in_db.source_microformats = data
-        await db_session.flush()
-        webmention = existing_webmention_in_db
 
-        if is_notification_enabled(models.NotificationType.UPDATED_WEBMENTION):
-            notif = models.Notification(
-                notification_type=models.NotificationType.UPDATED_WEBMENTION,
-                outbox_object_id=mentioned_object.id,
-                webmention_id=existing_webmention_in_db.id,
-            )
-            db_session.add(notif)
+        notif = models.Notification(
+            notification_type=models.NotificationType.UPDATED_WEBMENTION,
+            outbox_object_id=mentioned_object.id,
+            webmention_id=existing_webmention_in_db.id,
+        )
+        db_session.add(notif)
     else:
         new_webmention = models.Webmention(
             source=source,
             target=target,
             source_microformats=data,
             outbox_object_id=mentioned_object.id,
-            webmention_type=webmention_type,
         )
         db_session.add(new_webmention)
         await db_session.flush()
-        webmention = new_webmention
 
-        if is_notification_enabled(models.NotificationType.NEW_WEBMENTION):
-            notif = models.Notification(
-                notification_type=models.NotificationType.NEW_WEBMENTION,
-                outbox_object_id=mentioned_object.id,
-                webmention_id=new_webmention.id,
-            )
-            db_session.add(notif)
+        notif = models.Notification(
+            notification_type=models.NotificationType.NEW_WEBMENTION,
+            outbox_object_id=mentioned_object.id,
+            webmention_id=new_webmention.id,
+        )
+        db_session.add(notif)
 
-    # Determine the webmention type
-    for item in data.get("items", []):
-        if target in item.get("properties", {}).get(
-            "in-reply-to", []
-        ) and WebmentionReply.from_webmention(webmention):
-            webmention_type = models.WebmentionType.REPLY
-            break
-        elif target in item.get("properties", {}).get(
-            "like-of", []
-        ) and Face.from_webmention(webmention):
-            webmention_type = models.WebmentionType.LIKE
-            break
-        elif target in item.get("properties", {}).get(
-            "repost-of", []
-        ) and Face.from_webmention(webmention):
-            webmention_type = models.WebmentionType.REPOST
-            break
+        mentioned_object.webmentions_count = mentioned_object.webmentions_count + 1
 
-    if webmention_type != models.WebmentionType.UNKNOWN:
-        webmention.webmention_type = webmention_type
-        await db_session.flush()
-
-    # Handle side effect
-    await _handle_webmention_side_effects(db_session, webmention, mentioned_object)
     await db_session.commit()
 
     return JSONResponse(content={}, status_code=200)
-
-
-async def _handle_webmention_side_effects(
-    db_session: AsyncSession,
-    webmention: models.Webmention,
-    mentioned_object: models.OutboxObject,
-) -> None:
-    if webmention.webmention_type == models.WebmentionType.UNKNOWN:
-        # TODO: recount everything
-        mentioned_object.webmentions_count = await db_session.scalar(
-            select(func.count(models.Webmention.id)).where(
-                models.Webmention.is_deleted.is_(False),
-                models.Webmention.outbox_object_id == mentioned_object.id,
-                models.Webmention.webmention_type == models.WebmentionType.UNKNOWN,
-            )
-        )
-    elif webmention.webmention_type == models.WebmentionType.LIKE:
-        mentioned_object.likes_count = await _get_outbox_likes_count(
-            db_session, mentioned_object
-        )
-    elif webmention.webmention_type == models.WebmentionType.REPOST:
-        mentioned_object.announces_count = await _get_outbox_announces_count(
-            db_session, mentioned_object
-        )
-    elif webmention.webmention_type == models.WebmentionType.REPLY:
-        mentioned_object.replies_count = await _get_outbox_replies_count(
-            db_session, mentioned_object
-        )
-    else:
-        raise ValueError(f"Unhandled {webmention.webmention_type} webmention")

data/templates/app

@@ -1 +0,0 @@
-../../app/templates/

data/tests.toml

@@ -5,7 +5,6 @@ admin_password = "$2b$12$OwCyZM33uXQUVrChgER.h.qgFJ4fBp6tdFwArR3Lm1LV8NgMvIxVa"
 name = "test"
 summary = "<p>Hello</p>"
 https = false
-id = "http://localhost:8000"
 icon_url = "https://localhost:8000/static/nopic.png"
 secret = "1dd4079e0474d1a519052b8fe3cb5fa6"
 debug = true

docs/developer_guide.md

@@ -58,24 +58,3 @@ And check out the result by starting a static server using Python standard libra
 cd docs/dist
 python -m http.server 8001
 ```
-
-## Contributing
-
-Contributions/patches are welcome, but please start a discussion in a [ticket](https://todo.sr.ht/~tsileo/microblog.pub) or a [thread in the mailing list](https://lists.sr.ht/~tsileo/microblog.pub-devel) before working on anything consequent.
-
-### Patches
-
-Please ensure your code passes the code quality checks:
-
-```bash
-inv autoformat
-inv lint
-```
-
-And that the tests suite is passing:
-
-```bash
-inv tests
-```
-
-Please also consider adding new test cases if needed.

docs/install.md

@@ -89,12 +89,6 @@ Setup config.
 poetry run inv configuration-wizard
 ```
 
-Setup the database.
-
-```bash
-poetry run inv migrate-db
-```
-
 Grab your virtualenv path.
 
 ```bash
@@ -191,49 +185,6 @@ http {
 }
 ```
 
-## (Advanced) Running from subpath
-
-It is possible to configure microblogpub to run from subpath.
-To achieve this, do the following configuration _between_ config and start steps.
-i.e. _after_ you run `make config` or `poetry run inv configuration-wizard`,
-but _before_ you run `docker compose up` or `poetry run supervisord`.
-Changing this settings on an instance which has some posts or was seen by other instances will likely break links to these posts or federation (i.e. links to your instance, posts and profile from other instances).
-
-The following steps will explain how to configure instance to be available at `https://example.com/subdir`.
-Change them to your actual domain and subdir.
-
-* Edit `data/profile.toml` file, add this line:
-
-		id = "https://example.com/subdir"
-
-* Edit `misc/*-supervisord.conf` file which is relevant to you (it depends on how you start microblogpub - if in doubt, do the same change in all of them) - in `[program:uvicorn]` section, in the line which starts with `command`, add this argument at the very end: ` --root-path /subdir`
-
-Above two steps are enough to configure microblogpub.
-Next, you also need to configure reverse proxy.
-It might slightly differ if you plan to have other services running on the same domain, but for [NGINX config shown above](#reverse-proxy), the following changes are enough:
-
-* Add subdir to location, so location block starts like this:
-
-		location /subdir {
-
-* Add `/` at the end of `proxy_pass` directive, like this:
-
-		proxy_pass http://localhost:8000/;
-
-These two changes will instruct NGINX that requests sent to `https://example.com/subdir/...` should be forwarded to `http://localhost:8000/...`.
-
-* Inside `server` block, add redirects for well-known URLs (add these lines after `client_max_body_size`, remember to replace `subdir` with your actual subdir!):
-
-		location /.well-known/webfinger { return 301 /subdir$request_uri; }
-		location /.well-known/nodeinfo  { return 301 /subdir$request_uri; }
-		location /.well-known/oauth-authorization-server  { return 301 /subdir$request_uri; }
-
-* Optionally, [check robots.txt from a running microblogpub instance](https://microblog.pub/robots.txt) and integrate it into robots.txt file in the root of your server - remember to prepend `subdir` to URLs, so for example `Disallow: /admin` becomes `Disallow: /subdir/admin`.
-
 ## YunoHost edition
 
-[YunoHost](https://yunohost.org/) support is available (although it is not an official package for now): <https://git.sr.ht/~tsileo/microblog.pub_ynh>.
-
-## Available tutorial/guides
-
- - [Opalstack](https://community.opalstack.com/d/1055-howto-install-and-run-microblogpub-on-opalstack), thanks to [@defulmere@mastodon.social](https://mastodon.online/@defulmere).
+[YunoHost](https://yunohost.org/) support is a work in progress.

docs/templates/layout.html

@@ -63,7 +63,7 @@ nav a:hover, main a:hover, header p a:hover {
     max-width: 960px;
     margin: 50px auto;
 }
-pre {
+pre code {
     padding: 10px;
     overflow: auto;
     display: block;

docs/user_guide.md

@@ -98,39 +98,6 @@ privacy_replace = [
 ]
 ```
 
-### Disabling certain notification types
-
-All notifications are enabled by default.
-
-You can disabled specific notifications by adding them to the `disabled_notifications` list.
-
-This example disables likes and shares notifications:
-
-```
-disabled_notifications = ["like", "announce"]
-```
-
-#### Available notification types
-
- - `new_follower`
- - `rejected_follower`
- - `unfollow`
- - `follow_request_accepted`
- - `follow_request_rejected`
- - `move`
- - `like`
- - `undo_like`
- - `announce`
- - `undo_announce`
- - `mention`
- - `new_webmention`
- - `updated_webmention`
- - `deleted_webmention`
- - `blocked`
- - `unblocked`
- - `block`
- - `unblock`
-
 ### Customization
 
 #### Default emoji
@@ -146,7 +113,6 @@ You can copy/paste them from [getemoji.com](https://getemoji.com/).
 #### Custom emoji
 
 You can add custom emoji in the `data/custom_emoji` directory and they will be picked automatically.
-Do not use exotic characters in filename - only letters, numbers, and underscore symbol `_` are allowed.
 
 #### Custom CSS
 
@@ -161,23 +127,9 @@ $secondary-color: #32cd32;
 
 See `app/scss/main.scss` to see what variables can be overridden.
 
-#### Custom templates
-
-If you'd like to customize your instance's theme beyond CSS, you can modify the app's HTML by placing templates in `data/templates` which overwrite the defaults in `app/templates`.
-
-#### Custom Content Security Policy (CSP)
-
-You can override the default Content Security Policy by adding a line in `data/profile.toml`:
-
-```toml
-custom_content_security_policy = "default-src 'self'; style-src 'self' 'sha256-{HIGHLIGHT_CSS_HASH}'; frame-ancestors 'none'; base-uri 'self'; form-action 'self';"
-```
-
-This example will output the default CSP, note that `{HIGHLIGHT_CSS_HASH}` will be dynamically replaced by the correct value (the hash of the CSS needed for syntax highlighting).
-
 #### Code highlighting theme
 
-You can switch to one of the [styles supported by Pygments](https://pygments.org/styles/) by adding a line in `data/profile.toml`:
+You can switch to one of the [styles supported by Pygments](https://pygments.org/styles/) by adding a line in `profile.toml`:
 
 ```toml
 code_highlighting_theme = "solarized-dark"
@@ -381,13 +333,13 @@ make compile-scss
 
 ### Password reset
 
-If have lost your password, you can generate a new one using the `reset-password` task.
+If have lost your password, you can generate a new one using the `password-reset` task.
 
 #### Python edition
 
 ```bash
 # shutdown supervisord
-poetry run inv reset-password
+poetry run inv password-reset
 # edit data/profile.toml
 # restart supervisord
 ```
@@ -396,7 +348,7 @@ poetry run inv reset-password
 
 ```bash
 docker compose stop
-make reset-password
+make password-reset
 # edit data/profile.toml
 docker compose up -d
 ```
@@ -499,7 +451,6 @@ make self-destruct
 
 If the server is not (re)starting, you can:
 
- - [Ensure that the configuration is valid](/user_guide.html#configuration-checking).
- - [Verify if you haven't any syntax error in the custom theme by recompiling the CSS](/user_guide.html#recompiling-css-files).
- - Look at the log files (in `data/uvicorn.log`, `data/incoming.log` and `data/outgoing.log`).
- - If the CSS is not working, ensure your reverse proxy is serving the static file correctly.
+ - [Ensure that the configuration is valid](/user_guide.html#configuration-checking)
+ - [Verify if you haven't any syntax error in the custom theme by recompiling the CSS](/user_guide.html#recompiling-css-files)
+ - Look at the log files

pyproject.toml

@@ -18,6 +18,7 @@ httpx = {extras = ["http2"], version = "^0.23.0"}
 SQLAlchemy = {extras = ["asyncio"], version = "^1.4.39"}
 alembic = "^1.8.0"
 bleach = "^5.0.0"
+Markdown = "^3.3.7"
 prompt-toolkit = "^3.0.29"
 tomli-w = "^1.0.0"
 python-dateutil = "^2.8.2"
@@ -26,6 +27,7 @@ html5lib = "^1.1"
 mf2py = "^1.1.2"
 Pygments = "^2.12.0"
 loguru = "^0.6.0"
+mdx-linkify = "^2.1"
 Pillow = "^9.1.1"
 blurhash-python = "^1.1.3"
 html2text = "^2020.1.16"
@@ -43,8 +45,6 @@ boussole = "^2.0.0"
 uvicorn = {extras = ["standard"], version = "^0.18.3"}
 Brotli = "^1.0.9"
 greenlet = "^1.1.3"
-mistletoe = "^0.9.0"
-Pebble = "^5.0.2"
 
 [tool.poetry.dev-dependencies]
 black = "^22.3.0"

scripts/build_docs.py

@@ -1,115 +1,19 @@
-import re
 import shutil
-import typing
 from pathlib import Path
-from typing import Any
 
 from jinja2 import Environment
 from jinja2 import FileSystemLoader
 from jinja2 import select_autoescape
-from mistletoe import Document  # type: ignore
-from mistletoe import HTMLRenderer  # type: ignore
-from mistletoe import block_token  # type: ignore
-from pygments import highlight  # type: ignore
-from pygments.formatters import HtmlFormatter  # type: ignore
-from pygments.lexers import get_lexer_by_name as get_lexer  # type: ignore
-from pygments.lexers import guess_lexer  # type: ignore
+from markdown import markdown
 
 from app.config import VERSION
-from app.source import CustomRenderer
 from app.utils.datetime import now
 
-_FORMATTER = HtmlFormatter()
-_FORMATTER.noclasses = True
 
-
-class DocRenderer(CustomRenderer):
-    def __init__(
-        self,
-        depth=5,
-        omit_title=True,
-        filter_conds=[],
-    ) -> None:
-        super().__init__(
-            enable_mentionify=False,
-            enable_hashtagify=False,
-        )
-        self._headings: list[tuple[int, str, str]] = []
-        self._ids: set[str] = set()
-        self.depth = depth
-        self.omit_title = omit_title
-        self.filter_conds = filter_conds
-
-    @property
-    def toc(self):
-        """
-        Returns table of contents as a block_token.List instance.
-        """
-
-        def get_indent(level):
-            if self.omit_title:
-                level -= 1
-            return " " * 4 * (level - 1)
-
-        def build_list_item(heading):
-            level, content, title_id = heading
-            template = '{indent}- <a href="#{id}" rel="nofollow">{content}</a>\n'
-            return template.format(
-                indent=get_indent(level), content=content, id=title_id
-            )
-
-        lines = [build_list_item(heading) for heading in self._headings]
-        items = block_token.tokenize(lines)
-        return items[0]
-
-    def render_heading(self, token):
-        """
-        Overrides super().render_heading; stores rendered heading first,
-        then returns it.
-        """
-        template = '<h{level} id="{id}">{inner}</h{level}>'
-        inner = self.render_inner(token)
-        title_id = inner.lower().replace(" ", "-")
-        if title_id in self._ids:
-            i = 1
-            while 1:
-                title_id = f"{title_id}_{i}"
-                if title_id not in self._ids:
-                    break
-        self._ids.add(title_id)
-        rendered = template.format(level=token.level, inner=inner, id=title_id)
-        content = self.parse_rendered_heading(rendered)
-
-        if not (
-            self.omit_title
-            and token.level == 1
-            or token.level > self.depth
-            or any(cond(content) for cond in self.filter_conds)
-        ):
-            self._headings.append((token.level, content, title_id))
-        return rendered
-
-    @staticmethod
-    def parse_rendered_heading(rendered):
-        """
-        Helper method; converts rendered heading to plain text.
-        """
-        return re.sub(r"<.+?>", "", rendered)
-
-    def render_block_code(self, token: typing.Any) -> str:
-        code = token.children[0].content
-        lexer = get_lexer(token.language) if token.language else guess_lexer(code)
-        return highlight(code, lexer, _FORMATTER)
-
-
-def markdownify(content: str) -> tuple[str, Any]:
-    with DocRenderer() as renderer:
-        rendered_content = renderer.render(Document(content))
-
-    with HTMLRenderer() as html_renderer:
-        toc = html_renderer.render(renderer.toc)
-
-    return rendered_content, toc
+def markdownify(content: str) -> str:
+    return markdown(
+        content, extensions=["mdx_linkify", "fenced_code", "codehilite", "toc"]
+    )
 
 
 def main() -> None:
@@ -126,36 +30,32 @@ def main() -> None:
     last_updated = now().replace(second=0, microsecond=0).isoformat()
 
     readme = Path("README.md")
-    content, toc = markdownify(readme.read_text().removeprefix("# microblog.pub"))
     template.stream(
-        content=content,
+        content=markdownify(readme.read_text().removeprefix("# microblog.pub")),
         version=VERSION,
         path="/",
         last_updated=last_updated,
     ).dump("docs/dist/index.html")
 
     install = Path("docs/install.md")
-    content, toc = markdownify(install.read_text())
     template.stream(
-        content=content.replace("[TOC]", toc),
+        content=markdownify(install.read_text()),
         version=VERSION,
         path="/installing.html",
         last_updated=last_updated,
     ).dump("docs/dist/installing.html")
 
     user_guide = Path("docs/user_guide.md")
-    content, toc = markdownify(user_guide.read_text())
     template.stream(
-        content=content.replace("[TOC]", toc),
+        content=markdownify(user_guide.read_text()),
         version=VERSION,
         path="/user_guide.html",
         last_updated=last_updated,
     ).dump("docs/dist/user_guide.html")
 
     developer_guide = Path("docs/developer_guide.md")
-    content, toc = markdownify(developer_guide.read_text())
     template.stream(
-        content=content.replace("[TOC]", toc),
+        content=markdownify(developer_guide.read_text()),
         version=VERSION,
         path="/developer_guide.html",
         last_updated=last_updated,

scripts/config_wizard.py

@@ -75,10 +75,9 @@ def main() -> None:
         proto = "http"
 
     print("Note that you can put your icon/avatar in the static/ directory")
-    if icon_url := prompt(
+    dat["icon_url"] = prompt(
         "icon URL: ", default=f'{proto}://{dat["domain"]}/static/nopic.png'
-    ):
-        dat["icon_url"] = icon_url
+    )
     dat["secret"] = os.urandom(16).hex()
 
     with config_file.open("w") as f:

tasks.py

@@ -1,6 +1,5 @@
 import asyncio
 import io
-import shutil
 import tarfile
 from contextlib import contextmanager
 from pathlib import Path
@@ -46,12 +45,7 @@ def compile_scss(ctx, watch=False):
     # type: (Context, bool) -> None
     from app.utils.favicon import build_favicon
 
-    favicon_file = Path("data/favicon.ico")
-    if not favicon_file.exists():
-        build_favicon()
-    else:
-        shutil.copy2(favicon_file, "app/static/favicon.ico")
-
+    build_favicon()
     theme_file = Path("data/_theme.scss")
     if not theme_file.exists():
         theme_file.write_text("// override vars for theming here")

tests/factories.py

@@ -68,20 +68,6 @@ def build_accept_activity(
     }
 
 
-def build_block_activity(
-    from_remote_actor: actor.RemoteActor,
-    for_remote_actor: actor.RemoteActor,
-    outbox_public_id: str | None = None,
-) -> ap.RawObject:
-    return {
-        "@context": ap.AS_CTX,
-        "type": "Block",
-        "id": from_remote_actor.ap_id + "/block/" + (outbox_public_id or uuid4().hex),
-        "actor": from_remote_actor.ap_id,
-        "object": for_remote_actor.ap_id,
-    }
-
-
 def build_move_activity(
     from_remote_actor: actor.RemoteActor,
     for_remote_object: actor.RemoteActor,

tests/test_inbox.py

@@ -423,53 +423,3 @@ def test_inbox__move_activity(
     ).scalar_one()
     assert notif.actor.ap_id == new_ra.ap_id
     assert notif.inbox_object_id == inbox_activity.id
-
-
-def test_inbox__block_activity(
-    db: Session,
-    client: TestClient,
-    respx_mock: respx.MockRouter,
-) -> None:
-    # Given a remote actor
-    ra = setup_remote_actor(respx_mock)
-
-    # Which is followed by the local actor
-    setup_remote_actor_as_following(ra)
-
-    # When receiving a Block activity
-    follow_activity = RemoteObject(
-        factories.build_block_activity(
-            from_remote_actor=ra,
-            for_remote_actor=LOCAL_ACTOR,
-        ),
-        ra,
-    )
-    with mock_httpsig_checker(ra):
-        response = client.post(
-            "/inbox",
-            headers={"Content-Type": ap.AS_CTX},
-            json=follow_activity.ap_object,
-        )
-
-    # Then the server returns a 202
-    assert response.status_code == 202
-
-    run_process_next_incoming_activity()
-
-    # And the actor was saved in DB
-    saved_actor = db.execute(select(models.Actor)).scalar_one()
-    assert saved_actor.ap_id == ra.ap_id
-
-    # And the Block activity was saved in the inbox
-    inbox_activity = db.execute(
-        select(models.InboxObject).where(models.InboxObject.ap_type == "Block")
-    ).scalar_one()
-
-    # And a notification was created
-    notif = db.execute(
-        select(models.Notification).where(
-            models.Notification.notification_type == models.NotificationType.BLOCKED
-        )
-    ).scalar_one()
-    assert notif.actor.ap_id == ra.ap_id
-    assert notif.inbox_object_id == inbox_activity.id

tests/test_outbox.py

@@ -179,7 +179,7 @@ def test_send_create_activity__with_attachment(
     outbox_object = db.execute(select(models.OutboxObject)).scalar_one()
     assert outbox_object.ap_type == "Note"
     assert outbox_object.summary is None
-    assert outbox_object.content == "<p>hello</p>\n"
+    assert outbox_object.content == "<p>hello</p>"
     assert len(outbox_object.attachments) == 1
     attachment = outbox_object.attachments[0]
     assert attachment.type == "Document"
@@ -227,7 +227,7 @@ def test_send_create_activity__no_content_with_cw_and_attachments(
     outbox_object = db.execute(select(models.OutboxObject)).scalar_one()
     assert outbox_object.ap_type == "Note"
     assert outbox_object.summary is None
-    assert outbox_object.content == "<p>cw</p>\n"
+    assert outbox_object.content == "<p>cw</p>"
     assert len(outbox_object.attachments) == 1
 
 

tests/test_utils.py

@@ -1,19 +0,0 @@
-from unittest import mock
-
-import pytest
-
-from app.utils.url import is_hostname_blocked
-
-
-@pytest.mark.parametrize(
-    "hostname,should_be_blocked",
-    [
-        ("example.com", True),
-        ("subdomain.example.com", True),
-        ("example.xyz", False),
-    ],
-)
-def test_is_hostname_blocked(hostname: str, should_be_blocked: bool) -> None:
-    with mock.patch("app.utils.url.BLOCKED_SERVERS", ["example.com"]):
-        is_hostname_blocked.cache_clear()
-        assert is_hostname_blocked(hostname) is should_be_blocked