Command result notices

This introduces a new notice-like event with more structure so that clients can know if an event is sucessfully written to the database. Clients can't really do much with the current NOTICE messages, but with these structured result messages, it can know if an event was sucessfully saved. Whenever there is an error and we have an id available (event id, sub id, etc) we return these structured OK events instead. Example: When saving the following event: ["EVENT",{"id": "event_id" }] The server will now return events in the following format: ["OK", event_id, "true|false", message] For example, on a successful save: ["OK", "event_id", "true"] If we already have the event: ["OK", "event_id", "true", "duplicate"] If the event is rejected: ["OK", "event_id", "false", "you are blocked"] If a subscription fails: ["OK", "sub_id", "false", "Too many subscriptions"] NIP coming soon!
2022-11-09 07:44:39 -06:00
55 changed files with 2314 additions and 11152 deletions
@@ -7,7 +7,6 @@ environment:
 packages:
  - cargo
  - sqlite-devel
-  - protobuf-compiler
 sources:
  - https://git.sr.ht/~gheartsfield/nostr-rs-relay/
 shell: false
@@ -1,39 +0,0 @@
-name: Test and build
-
-on:
-  push:
-    branches:
-      - master
-
-jobs:
-  test_nostr-rs-relay:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v3
-
-      - name: Update local toolchain
-        run: |
-          sudo apt-get install -y protobuf-compiler
-          rustup update
-          rustup component add clippy
-          rustup install nightly
-
-      - name: Toolchain info
-        run: |
-          cargo --version --verbose
-          rustc --version
-          cargo clippy --version
-
-      # - name: Lint
-      #   run: |
-      #     cargo fmt -- --check
-      #     cargo clippy -- -D warnings
-
-      - name: Test
-        run: |
-          cargo check
-          cargo test --all
-
-      - name: Build
-        run: |
-          cargo build --release --locked
@@ -1,4 +1,2 @@
-**/target/
+/target
 nostr.db
-nostr.db-*
-justfile
@@ -11,6 +11,6 @@ repos:
 - repo: https://github.com/doublify/pre-commit-rust
  rev: v1.0
  hooks:
-#  - id: fmt
+  - id: fmt
  - id: cargo-check
  - id: clippy
@@ -1,24 +1,12 @@
 [package]
 name = "nostr-rs-relay"
-version = "0.8.13"
+version = "0.7.1"
 edition = "2021"
-authors = ["Greg Heartsfield <scsibug@imap.cc>"]
-description = "A relay implementation for the Nostr protocol"
-readme = "README.md"
-homepage = "https://sr.ht/~gheartsfield/nostr-rs-relay/"
-repository = "https://git.sr.ht/~gheartsfield/nostr-rs-relay"
-license = "MIT"
-keywords = ["nostr", "server"]
-categories = ["network-programming", "web-programming"]

 [dependencies]
-clap = { version = "4.0.32", features = ["env", "default", "derive"]}
-tracing = "0.1.37"
-tracing-appender = "0.2.2"
-tracing-subscriber = "0.3.16"
+tracing = "0.1.36"
+tracing-subscriber = "0.2.0"
 tokio = { version = "1", features = ["full", "tracing", "signal"] }
-prost = "0.11"
-tonic = "0.8.3"
 console-subscriber = "0.1.8"
 futures = "0.3"
 futures-util = "0.3"
@@ -32,35 +20,19 @@ secp256k1 = {version = "0.21", features = ["rand", "rand-std", "serde", "bitcoin
 serde = { version = "1.0", features = ["derive"] }
 serde_json = {version = "1.0", features = ["preserve_order"]}
 hex = "0.4"
-rusqlite = { version = "0.26", features = ["limits","bundled","modern_sqlite", "trace"]}
+rusqlite = { version = "0.26", features = ["limits","bundled"]}
 r2d2 = "0.8"
 r2d2_sqlite = "0.19"
 lazy_static = "1.4"
 governor = "0.4"
 nonzero_ext = "0.3"
 hyper = { version="0.14", features=["client", "server","http1","http2","tcp"] }
-hyper-rustls = { version = "0.24" }
+hyper-tls = "0.5"
 http = { version = "0.2" }
 parse_duration = "2"
 rand = "0.8"
 const_format = "0.2.28"
 regex = "1"
-async-trait = "0.1.60"
-async-std = "1.12.0"
-sqlx = { version ="0.6.2", features=["runtime-tokio-rustls", "postgres", "chrono"]}
-chrono = "0.4.23"
-prometheus = "0.13.3"
-indicatif = "0.17.3"
-bech32 = "0.9.1"
-url = "2.3.1"
-qrcode = { version = "0.12.0", default-features = false, features = ["svg"] }
-nostr = { version = "0.18.0", default-features = false, features = ["base", "nip04", "nip19"] }
-[target.'cfg(not(target_env = "msvc"))'.dependencies]
-tikv-jemallocator = "0.5"
-log = "0.4"

 [dev-dependencies]
 anyhow = "1"
-
-[build-dependencies]
-tonic-build = { version="0.8.3", features = ["prost"] }
@@ -1,27 +1,18 @@
-FROM docker.io/library/rust:1-bookworm as builder
-RUN apt-get update \
-    && apt-get install -y cmake protobuf-compiler \
-    && rm -rf /var/lib/apt/lists/*
-RUN USER=root cargo install cargo-auditable
+FROM docker.io/library/rust:1.65.0@sha256:1bca14676a365d0ed37a1e2a1da86c2bcf883fdf6e6886469434763d94d4afd5 as builder
+
 RUN USER=root cargo new --bin nostr-rs-relay
 WORKDIR ./nostr-rs-relay
 COPY ./Cargo.toml ./Cargo.toml
 COPY ./Cargo.lock ./Cargo.lock
-# build dependencies only (caching)
-RUN cargo auditable build --release --locked
-# get rid of starter project code
+RUN cargo build --release
 RUN rm src/*.rs

-# copy project source code
 COPY ./src ./src
-COPY ./proto ./proto
-COPY ./build.rs ./build.rs

-# build auditable release using locked deps
 RUN rm ./target/release/deps/nostr*relay*
-RUN cargo auditable build --release --locked
+RUN cargo build --release

-FROM docker.io/library/debian:bookworm-slim
+FROM docker.io/library/debian:bullseye-20221024-slim@sha256:76cdda8fe5eb597ef5e712e4c9a9f5f1fb119e69f353daaa7bd6d0f6e66e541d

 ARG APP=/usr/src/app
 ARG APP_DATA=/usr/src/app/db
@@ -1,9 +1,8 @@
 # [nostr-rs-relay](https://git.sr.ht/~gheartsfield/nostr-rs-relay)

-This is a [nostr](https://github.com/nostr-protocol/nostr) relay,
-written in Rust.  It currently supports the entire relay protocol, and
-persists data with SQLite.  There is experimental support for
-Postgresql.
+This is a [nostr](https://github.com/nostr-protocol/nostr) relay, written in
+Rust.  It currently supports the entire relay protocol, and has a
+SQLite persistence layer.

 The project master repository is available on
 [sourcehut](https://sr.ht/~gheartsfield/nostr-rs-relay/), and is
@@ -11,9 +10,6 @@ mirrored on [GitHub](https://github.com/scsibug/nostr-rs-relay).

 [![builds.sr.ht status](https://builds.sr.ht/~gheartsfield/nostr-rs-relay/commits/master.svg)](https://builds.sr.ht/~gheartsfield/nostr-rs-relay/commits/master?)

-![Github CI](https://github.com/schlunsen/nostr-rs-relay/actions/workflows/ci.yml/badge.svg)
-
-
 ## Features

 [NIPs](https://github.com/nostr-protocol/nips) with a relay-specific implementation are listed here.
@@ -30,13 +26,8 @@ mirrored on [GitHub](https://github.com/scsibug/nostr-rs-relay).
 - [x] NIP-12: [Generic Tag Queries](https://github.com/nostr-protocol/nips/blob/master/12.md)
 - [x] NIP-15: [End of Stored Events Notice](https://github.com/nostr-protocol/nips/blob/master/15.md)
 - [x] NIP-16: [Event Treatment](https://github.com/nostr-protocol/nips/blob/master/16.md)
- [x] NIP-20: [Command Results](https://github.com/nostr-protocol/nips/blob/master/20.md)
 - [x] NIP-22: [Event `created_at` limits](https://github.com/nostr-protocol/nips/blob/master/22.md) (_future-dated events only_)
- [ ] NIP-26: [Event Delegation](https://github.com/nostr-protocol/nips/blob/master/26.md) (_implemented, but currently disabled_)
- [x] NIP-28: [Public Chat](https://github.com/nostr-protocol/nips/blob/master/28.md)
- [x] NIP-33: [Parameterized Replaceable Events](https://github.com/nostr-protocol/nips/blob/master/33.md)
- [x] NIP-40: [Expiration Timestamp](https://github.com/nostr-protocol/nips/blob/master/40.md)
- [x] NIP-42: [Authentication of clients to relays](https://github.com/nostr-protocol/nips/blob/master/42.md)
+- [x] NIP-26: [Event Delegation](https://github.com/nostr-protocol/nips/blob/master/26.md)

 ## Quick Start

@@ -45,32 +36,15 @@ application.  Use a bind mount to store the SQLite database outside of
 the container image, and map the container's 8080 port to a host port
 (7000 in the example below).

-The examples below start a rootless podman container, mapping a local
-data directory and config file.
-
 ```console
-$ podman build -t nostr-rs-relay .
+$ docker build -t nostr-rs-relay .

-$ mkdir data
+$ docker run -it -p 7000:8080 \
+  --mount src=$(pwd)/data,target=/usr/src/app/db,type=bind nostr-rs-relay

-$ podman unshare chown 100:100 data
-
-$ podman run -it --rm -p 7000:8080 \
-  --user=100:100 \
-  -v $(pwd)/data:/usr/src/app/db:Z \
-  -v $(pwd)/config.toml:/usr/src/app/config.toml:ro,Z \
-  --name nostr-relay nostr-rs-relay:latest
-
-Nov 19 15:31:15.013  INFO nostr_rs_relay: Starting up from main
-Nov 19 15:31:15.017  INFO nostr_rs_relay::server: listening on: 0.0.0.0:8080
-Nov 19 15:31:15.019  INFO nostr_rs_relay::server: db writer created
-Nov 19 15:31:15.019  INFO nostr_rs_relay::server: control message listener started
-Nov 19 15:31:15.019  INFO nostr_rs_relay::db: Built a connection pool "event writer" (min=1, max=4)
-Nov 19 15:31:15.019  INFO nostr_rs_relay::db: opened database "/usr/src/app/db/nostr.db" for writing
-Nov 19 15:31:15.019  INFO nostr_rs_relay::schema: DB version = 0
-Nov 19 15:31:15.054  INFO nostr_rs_relay::schema: database pragma/schema initialized to v7, and ready
-Nov 19 15:31:15.054  INFO nostr_rs_relay::schema: All migration scripts completed successfully.  Welcome to v7.
-Nov 19 15:31:15.521  INFO nostr_rs_relay::db: Built a connection pool "client query" (min=4, max=128)
+[2021-12-31T19:58:31Z INFO  nostr_rs_relay] listening on: 0.0.0.0:8080
+[2021-12-31T19:58:31Z INFO  nostr_rs_relay::db] opened database "/usr/src/app/db/nostr.db" for writing
+[2021-12-31T19:58:31Z INFO  nostr_rs_relay::db] DB version = 2
 ```

 Use a `nostr` client such as
@@ -89,43 +63,6 @@ Text Note [81cf...2652] from 296a...9b92 5 seconds ago
 A pre-built container is also available on DockerHub:
 https://hub.docker.com/r/scsibug/nostr-rs-relay

-## Build and Run (without Docker)
-
-Building `nostr-rs-relay` requires an installation of Cargo & Rust: https://www.rust-lang.org/tools/install
-
-The following OS packages will be helpful; on Debian/Ubuntu:
-```console
-$ sudo apt-get install build-essential cmake protobuf-compiler pkg-config libssl-dev
-```
-
-Clone this repository, and then build a release version of the relay:
-
-```console
-$ git clone -q https://git.sr.ht/\~gheartsfield/nostr-rs-relay
-$ cd nostr-rs-relay
-$ cargo build -q -r
-```
-
-The relay executable is now located in
-`target/release/nostr-rs-relay`.  In order to run it with logging
-enabled, execute it with the `RUST_LOG` variable set:
-
-```console
-$ RUST_LOG=warn,nostr_rs_relay=info ./target/release/nostr-rs-relay
-Dec 26 10:31:56.455  INFO nostr_rs_relay: Starting up from main
-Dec 26 10:31:56.464  INFO nostr_rs_relay::server: listening on: 0.0.0.0:8080
-Dec 26 10:31:56.466  INFO nostr_rs_relay::server: db writer created
-Dec 26 10:31:56.466  INFO nostr_rs_relay::db: Built a connection pool "event writer" (min=1, max=2)
-Dec 26 10:31:56.466  INFO nostr_rs_relay::db: opened database "./nostr.db" for writing
-Dec 26 10:31:56.466  INFO nostr_rs_relay::schema: DB version = 11
-Dec 26 10:31:56.467  INFO nostr_rs_relay::db: Built a connection pool "maintenance writer" (min=1, max=2)
-Dec 26 10:31:56.467  INFO nostr_rs_relay::server: control message listener started
-Dec 26 10:31:56.468  INFO nostr_rs_relay::db: Built a connection pool "client query" (min=4, max=8)
-```
-
-You now have a running relay, on port `8080`.  Use a `nostr` client or
-`websocat` to connect and send/query for events.
-
 ## Configuration

 The sample [`config.toml`](config.toml) file demonstrates the
@@ -146,7 +83,7 @@ settings.

 For examples of putting the relay behind a reverse proxy (for TLS
 termination, load balancing, and other features), see [Reverse
-Proxy](docs/reverse-proxy.md).
+Proxy](reverse-proxy.md).

 ## Dev Channel

@@ -160,8 +97,3 @@ To chat about `nostr-rs-relay` on `nostr` itself; visit our channel on [anigma](
 License
 ---
 This project is MIT licensed.
-
-External Documentation and Links
---
-
-* [BlockChainCaffe's Nostr Relay Setup Guide](https://github.com/BlockChainCaffe/Nostr-Relay-Setup-Guide)
@@ -1,7 +0,0 @@
-fn main() -> Result<(), Box<dyn std::error::Error>> {
-    tonic_build::configure()
-        .build_server(false)
-        .protoc_arg("--experimental_allow_proto3_optional")
-        .compile(&["proto/nauthz.proto"], &["proto"])?;
-    Ok(())
-}
@@ -16,64 +16,28 @@ description = "A newly created nostr-rs-relay.\n\nCustomize this with your own i
 # Administrative contact URI
 #contact = "mailto:contact@example.com"

-# Favicon location.  Relative to the current directory.  Assumes an
-# ICO format.
-#favicon = "favicon.ico"
-
-# URL of Relay's icon.
-#relay_icon = "https://example.test/img.png"
-
 [diagnostics]
 # Enable tokio tracing (for use with tokio-console)
-#tracing = false
+#tracing = true

 [database]
-# Database engine (sqlite/postgres).  Defaults to sqlite.
-# Support for postgres is currently experimental.
-#engine = "sqlite"
-
 # Directory for SQLite files.  Defaults to the current directory.  Can
 # also be specified (and overriden) with the "--db dirname" command
 # line option.
-#data_directory = "."
+data_directory = "."
+

 # Use an in-memory database instead of 'nostr.db'.
-# Requires sqlite engine.
 # Caution; this will not survive a process restart!
 #in_memory = false

 # Database connection pool settings for subscribers:

 # Minimum number of SQLite reader connections
-#min_conn = 0
+#min_conn = 4

-# Maximum number of SQLite reader connections.  Recommend setting this
-# to approx the number of cores.
-#max_conn = 8
-
-# Database connection string.  Required for postgres; not used for
-# sqlite.
-#connection = "postgresql://postgres:nostr@localhost:7500/nostr"
-
-# Optional database connection string for writing.  Use this for
-# postgres clusters where you want to separate reads and writes to
-# different nodes.  Ignore for single-database instances.
-#connection_write = "postgresql://postgres:nostr@localhost:7500/nostr"
-
-[logging]
-# Directory to store log files.  Log files roll over daily.
-#folder_path = "./log"
-#file_prefix = "nostr-relay"
-
-[grpc]
-# gRPC interfaces for externalized decisions and other extensions to
-# functionality.
-#
-# Events can be authorized through an external service, by providing
-# the URL below.  In the event the server is not accessible, events
-# will be permitted.  The protobuf3 schema used is available in
-# `proto/nauthz.proto`.
-# event_admission_server = "http://[::1]:50051"
+# Maximum number of SQLite reader connections
+#max_conn = 128

 [network]
 # Bind to this network address
@@ -98,29 +62,8 @@ reject_future_seconds = 1800

 [limits]
 # Limit events created per second, averaged over one minute.  Must be
-# an integer.  If not set (or set to 0), there is no limit.  Note:
-# this is for the server as a whole, not per-connection.
-#
-# Limiting event creation is highly recommended if your relay is
-# public!
-#
-#messages_per_sec = 5
-
-# Limit client subscriptions created, averaged over one minute.  Must
-# be an integer.  If not set (or set to 0), defaults to unlimited.
-# Strongly recommended to set this to a low value such as 10 to ensure
-# fair service.
-#subscriptions_per_min = 0
-
-# UNIMPLEMENTED...
-# Limit how many concurrent database connections a client can have.
-# This prevents a single client from starting too many expensive
-# database queries.  Must be an integer.  If not set (or set to 0),
-# defaults to unlimited (subject to subscription limits).
-#db_conns_per_client = 0
-
-# Limit blocking threads used for database connections.  Defaults to 16.
-#max_blocking_threads = 16
+# an integer.  If not set (or set to 0), defaults to unlimited.
+#messages_per_sec = 0

 # Limit the maximum size of an EVENT message.  Defaults to 128 KB.
 # Set to 0 for unlimited.
@@ -140,16 +83,6 @@ reject_future_seconds = 1800
 # backpressure to senders if writes are slow.
 #event_persist_buffer = 4096

-# Event kind blacklist. Events with these kinds will be discarded.
-#event_kind_blacklist = [
-#    70202,
-#]
-
-# Event kind allowlist. Events other than these kinds will be discarded.
-#event_kind_allowlist = [
-#    0, 1, 2, 3, 7, 40, 41, 42, 43, 44, 30023,
-#]
-
 [authorization]
 # Pubkey addresses in this array are whitelisted for event publishing.
 # Only valid events by these authors will be accepted, if the variable
@@ -158,10 +91,6 @@ reject_future_seconds = 1800
 #  "35d26e4690cbe1a898af61cc3515661eb5fa763b57bd0b42e45099c8b32fd50f",
 #  "887645fef0ce0c3c1218d2f5d8e6132a19304cdc57cd20281d082f38cfea0072",
 #]
-# Enable NIP-42 authentication
-#nip42_auth = false
-# Send DMs (kind 4 and 44) and gift wraps (kind 1059) only to their authenticated recipients
-#nip42_dms = false

 [verified_users]
 # NIP-05 verification of users.  Can be "enabled" to require NIP-05
@@ -188,47 +117,3 @@ reject_future_seconds = 1800
 # How many consecutive failed checks before we give up on verifying
 # this author.
 #max_consecutive_failures = 20
-
-[pay_to_relay]
-# Enable pay to relay
-#enabled = false
-
-# The cost to be admitted to relay
-#admission_cost = 4200
-
-# The cost in sats per post
-#cost_per_event = 0
-
-# Url of lnbits api
-#node_url = "<node url>"
-
-# LNBits api secret
-#api_secret = "<ln bits api>"
-
-# Nostr direct message on signup
-#direct_message=true
-
-# Terms of service
-#terms_message = """
-#This service (and supporting services) are provided "as is", without warranty of any kind, express or implied.
-#
-#By using this service, you agree:
-#* Not to engage in spam or abuse the relay service
-#* Not to disseminate illegal content
-#* That requests to delete content cannot be guaranteed
-#* To use the service in compliance with all applicable laws
-#* To grant necessary rights to your content for unlimited time
-#* To be of legal age and have capacity to use this service
-#* That the service may be terminated at any time without notice
-#* That the content you publish may be removed at any time without notice
-#* To have your IP address collected to detect abuse or misuse
-#* To cooperate with the relay to combat abuse or misuse
-#* You may be exposed to content that you might find triggering or distasteful
-#* The relay operator is not liable for content produced by users of the relay
-#"""
-
-# Whether or not new sign ups should be allowed
-#sign_ups = false
-
-# optional if `direct_message=false`
-#secret_key = "<nostr nsec>"
@@ -1,14 +0,0 @@
-[Unit]
-Description=nostr-rs-relay
-
-[Service]
-User=REPLACE_WITH_YOUR_USERNAME
-WorkingDirectory=/var/lib/nostr-rs-relay
-Environment=RUST_LOG=warn,nostr_rs_relay=info
-ExecStart=/usr/bin/nostr-rs-relay --config /etc/nostr-rs-relay/config.toml
-TimeoutStopSec=10
-Restart=on-failure
-RestartSec=5
-
-[Install]
-WantedBy=multi-user.target
@@ -1,129 +0,0 @@
-# Database Maintenance
-
-`nostr-rs-relay` uses the SQLite embedded database to minimize
-dependencies and overall footprint of running a relay.  If traffic is
-light, the relay should just run with very little need for
-intervention.  For heavily trafficked relays, there are a number of
-steps that the operator may need to take to maintain performance and
-limit disk usage.
-
-This maintenance guide is current as of version `0.8.2`.  Future
-versions may incorporate and automate some of these steps.
-
-## Backing Up the Database
-
-To prevent data loss, the database should be backed up regularly.  The
-recommended method is to use the `sqlite3` command to perform an
-"Online Backup".  This can be done while the relay is running, queries
-can still run and events will be persisted during the backup.
-
-The following commands will perform a backup of the database to a
-dated file, and then compress to minimize size:
-
-```console
-BACKUP_FILE=/var/backups/nostr/`date +%Y%m%d_%H%M`.db
-sqlite3 -readonly /apps/nostr-relay/nostr.db ".backup $BACKUP_FILE"
-sqlite3 $BACKUP_FILE "vacuum;"
-bzip2 -9 $BACKUP_FILE
-```
-
-Nostr events are very compressible.  Expect a compression ratio on the
-order of 4:1, resulting in a 75% space saving.
-
-## Vacuuming the Database
-
-As the database is updated, it can become fragmented.  Performing a
-full `vacuum` will rebuild the entire database file, and can reduce
-space.  Running this may reduce the size of the database file,
-especially if a large amount of data was updated or deleted.
-
-```console
-vacuum;
-```
-
-## Clearing Hidden Events
-
-When events are deleted, the event is not actually removed from the
-database.  Instead, a flag `HIDDEN` is set to true for the event,
-which excludes it from search results.  High volume replacements from
-profile or other replaceable events are deleted, not hidden, in the
-current version of the relay.
-
-In the current version, removing hidden events should not result in
-significant space savings, but it can still be used if there is no
-desire to hold on to events that can never be re-broadcast.
-
-```console
-PRAGMA foreign_keys = ON;
-delete from event where HIDDEN=true;
-```
-
-## Manually Removing Events
-
-For a variety of reasons, an operator may wish to remove some events
-from the database.  The only way of achieving this today is with
-manually run SQL commands.
-
-It is recommended to have a good backup prior to manually running SQL
-commands!
-
-In all cases, it is mandatory to enable foreign keys, and this must be
-done for every connection.  Otherwise, you will likely orphan rows in
-the `tag` table.
-
-### Deleting Specific Event
-
-```console
-PRAGMA foreign_keys = ON;
-delete from event where event_hash=x'00000000000c1271675dc86e3e1dd1336827bccabb90dc4c9d3b4465efefe00e';
-```
-
-### Querying and Deleting All Events for Pubkey
-
-```console
-PRAGMA foreign_keys = ON;
-
-select lower(hex(author)) as author, count(*) as c from event group by author order by c asc;
-
-delete from event where author=x'000000000002c7831d9c5a99f183afc2813a6f69a16edda7f6fc0ed8110566e6';
-```
-
-### Querying and Deleting All Events of a Kind
-
-
-```console
-PRAGMA foreign_keys = ON;
-
-select printf('%7d', kind), count(*) as c from event group by kind order by c;
-
-delete from event where kind=70202;
-```
-
-### Deleting Old Events
-
-In this scenario, we wish to delete any event that has been stored by
-our relay for more than 1 month.  Crucially, this is based on when the
-event was stored, not when the event says it was created.  If an event
-has a `created` field of 2 years ago, but was first sent to our relay
-yesterday, it would not be deleted in this scenario.  Keep in mind, we
-do not track anything for re-broadcast events that we already have, so
-this is not a very effective way of implementing a "least recently
-seen" policy.
-
-```console
-PRAGMA foreign_keys = ON;
-
-DELETE FROM event WHERE first_seen < CAST(strftime('%s', date('now', '-30 day')) AS INT);
-```
-
-### Delete Profile Events with No Recent Events
-
-Many users create profiles, post a "hello world" event, and then never
-appear again (likely using an ephemeral keypair that was lost in the
-browser cache).  We can find these accounts and remove them after some
-time.
-
-```console
-PRAGMA foreign_keys = ON;
-TODO!
-```
@@ -1,79 +0,0 @@
-# gRPC Extensions Design Document
-
-The relay will be extensible through gRPC endpoints, definable in the
-main configuration file.  These will allow external programs to host
-logic for deciding things such as, should this event be persisted,
-should this connection be allowed, and should this subscription
-request be registered.  The primary goal is allow for relay operator
-specific functionality that allows them to serve smaller communities
-and reduce spam and abuse.
-
-This will likely evolve substantially, the first goal is to get a
-basic one-way service that lets an externalized program decide on
-event persistence.  This does not represent the final state of gRPC
-extensibility in `nostr-rs-relay`.
-
-## Considerations
-
-Write event latency must not be significantly affected.  However, the
-primary reason we are implementing this is spam/abuse protection, so
-we are willing to tolerate some increase in latency if that protects
-us against outages!
-
-The interface should provide enough information to make simple
-decisions, without burdening the relay to do extra queries.  The
-decision endpoint will be mostly responsible for maintaining state and
-gathering additional details.
-
-## Design Overview
-
-A gRPC server may be defined in the `config.toml` file.  If it exists,
-the relay will attempt to connect to it and send a message for each
-`EVENT` command submitted by clients.  If a successful response is
-returned indicating the event is permitted, the relay continues
-processing the event as normal.  All existing whitelist, blacklist,
-and `NIP-05` validation checks are still performed and MAY still
-result in the event being rejected.  If a successful response is
-returned indicated the decision is anything other than permit, then
-the relay MUST reject the event, and return a command result to the
-user (using `NIP-20`) indicating the event was blocked (optionally
-providing a message).
-
-In the event there is an error in the gRPC interface, event processing
-proceeds as if gRPC was disabled (fail open).  This allows gRPC
-servers to be deployed with minimal chance of causing a full relay
-outage.
-
-## Design Details
-
-Currently one procedure call is supported, `EventAdmit`, in the
-`Authorization` service.  It accepts the following data in order to
-support authorization decisions:
-
- The event itself
- The client IP that submitted the event
- The client's HTTP origin header, if one exists
- The client's HTTP user agent header, if one exists
- The public key of the client, if `NIP-42` authentication was
-  performed (not supported in the relay yet!)
- The `NIP-05` associated with the event's public key, if it is known
-  to the relay
-
-A server providing authorization decisions will return the following:
-
- A decision to permit or deny the event
- An optional message that explains why the event was denied, to be
-  transmitted to the client
-
-## Security Issues
-
-There is little attempt to secure this interface, since it is intended
-for use processes running on the same host.  It is recommended to
-ensure that the gRPC server providing the API is not exposed to the
-public Internet.  Authorization server implementations should have
-their own security reviews performed.
-
-A slow gRPC server could cause availability issues for event
-processing, since this is performed on a single thread.  Avoid any
-expensive or long-running processes that could result from submitted
-events, since any client can initiate a gRPC call to the service.
@@ -1,84 +0,0 @@
-# Pay to Relay Design Document
-
-The relay with use payment as a form of spam prevention. In order to post to the relay a user must pay a set rate. There is also the option to require a payment for each note posted to the relay. There is no cost to read from the relay.
-
-## Configuration
-
-Currently, [LNBits](https://github.com/lnbits/lnbits) is implemented as the payment processor.  LNBits exposes a simple API for creating invoices, to use this API create a wallet and on the right side find "API info" you will need to add the invoice/read key to this relays config file.
-
-The below configuration will need to be added to config.toml
-```
-[pay_to_relay]
-# Enable pay to relay
-enabled = true
-# The cost to be admitted to relay
-admission_cost = 1000
-# The cost in sats per post
-cost_per_event = 0
-# Url of lnbits api
-node_url = "https://<IP of node>:5001/api/v1/payments"
-# LNBits api secret
-api_secret = "<LNbits api key>"
-# Terms of service
-terms_message = """This service ....
-"""
-# Whether or not new sign ups should be allowed
-sign_ups = true
-secret_key = "<nostr secret key to send dms>"
-```
-
-The LNBits instance must have a signed HTTPS a self signed certificate will not work.
-
-## Design Overview
-
-### Concepts
-
-All authors are initially not admitted to write to the relay.  There are two ways to gain access write to the relay. The first is by attempting to post the the relay, upon receiving an event from an author that is not admitted, the relay will send a direct message including the terms of service of the relay and a lighting invoice for the admission cost.  Once this invoice is paid the author can write to the relay. For this method to work the author must be reading from the relay. An author can also pay and accept the terms of service via a webpage `https://<relay-url>/join`.
-
-## Design Details
-
-Authors are stored in a dedicated table. This tracks:
-
-* `pubkey`
-* `is_admitted` whether on no the admission invoice has been paid, accepting the terms of service.
-* `balance` the current balance in sats of the author, used if there is a cost per post
-* `tos_accepted_at` the timestamp of when the author accepted the tos
-
-Invoice information is stored in a dedicated table. This tracks:
-* `payment_hash` the payment hash of the lighting invoice
-* `pubkey` of the author the invoice is issued to
-* `invoice` bolt11 invoice
-* `amount` in sats
-* `status` (Paid/Unpaid/Expired)
-* `description`
-* `created_at` timestamp of creation
-* `confirmed_at` timestamp of payment
-
-### Event Handling
-
-If "pay to relay" is enabled, all incoming events are evaluated to determine whether the author is on the relay's whitelist or if they have paid the admission fee and accepted the terms. If "pay per note" is enabled, there is an additional check to ensure that the author has enough balance, which is then reduced by the cost per note. If the author is on the whitelist, this balance check is not necessary.
-
-### Integration
-
-We have an existing database writer thread, which receives events and
-attempts to persist them to disk.  Once validated and persisted, these
-events are broadcast to all subscribers.
-
-When "pay to relay" is enabled, the writer must check if the author is admitted to post. If the author is not admitted to post the event is forwarded to the payment module. Where an invoice is generated, persisted and broadcast as an direct message to the author.
-
-### Threat Scenarios
-
-Some of these mitigation's are fully implemented, others are documented
-simply to demonstrate a mitigation is possible.
-
-### Sign up Spamming
-
-*Threat*: An attacker generates a large number of new pubkeys publishing to the relays. Causing a large number of new invoices to be created for each new pubkey.
-
-*Mitigation*: Rate limit number of new sign ups
-
-### Admitted Author Spamming
-
-*Threat*: An attacker gains write access by paying the admission fee, and then floods the relay with a large number of spam events.
-
-*Mitigation*: The attacker's admission can be revoked and their admission fee will not be refunded. Enabling "cost per event" and increasing the admission cost can also discourage this type of behavior.
@@ -1,199 +0,0 @@
-# Reverse Proxy Setup Guide
-
-It is recommended to run `nostr-rs-relay` behind a reverse proxy such
-as `haproxy`, `nginx` or `traefik` to provide TLS termination.  Simple examples
-for `haproxy`, `nginx` and `traefik` configurations are documented here.
-
-## Minimal HAProxy Configuration
-
-Assumptions:
-
-* HAProxy version is `2.4.10` or greater (older versions not tested).
-* Hostname for the relay is `relay.example.com`.
-* Your relay should be available over wss://relay.example.com
-* Your (NIP-11) relay info page should be available on https://relay.example.com
-* SSL certificate is located in `/etc/certs/example.com.pem`.
-* Relay is running on port 8080.
-* Limit connections to 400 concurrent.
-* HSTS (HTTP Strict Transport Security) is desired.
-* Only TLS 1.2 or greater is allowed.
-
-```
-global
-    ssl-default-bind-ciphersuites TLS_AES_128_GCM_SHA256:TLS_AES_256_GCM_SHA384:TLS_CHACHA20_POLY1305_SHA256
-    ssl-default-bind-options prefer-client-ciphers no-sslv3 no-tlsv10 no-tlsv11 no-tls-tickets
-
-frontend fe_prod
-    mode    http
-    bind    :443 ssl crt /etc/certs/example.com.pem alpn h2,http/1.1
-    bind    :80
-    http-request set-header X-Forwarded-Proto https if { ssl_fc }
-    redirect scheme https code 301 if !{ ssl_fc }
-    acl host_relay hdr(host) -i -m beg relay.example.com
-    use_backend relay if host_relay
-    # HSTS (1 year)
-    http-response set-header Strict-Transport-Security max-age=31536000
-
-backend relay
-    mode http
-    timeout connect 5s
-    timeout client 50s
-    timeout server 50s
-    timeout tunnel 1h
-    timeout client-fin 30s
-    option tcp-check
-    default-server maxconn 400 check inter 20s fastinter 1s
-    server relay 127.0.0.1:8080
-```
-
-### HAProxy Notes
-
-You may experience WebSocket connection problems with Firefox if
-HTTP/2 is enabled, for older versions of HAProxy (2.3.x).  Either
-disable HTTP/2 (`h2`), or upgrade HAProxy.
-
-## Bare-bones Nginx Configuration
-
-Assumptions:
-
-* `Nginx` version is `1.18.0` (other versions not tested).
-* Hostname for the relay is `relay.example.com`.
-* SSL certificate and key are located at `/etc/letsencrypt/live/relay.example.com/`.
-* Relay is running on port `8080`.
-
-```
-http {
-    server {
-        listen 443 ssl;
-        server_name relay.example.com;
-        ssl_certificate /etc/letsencrypt/live/relay.example.com/fullchain.pem;
-        ssl_certificate_key /etc/letsencrypt/live/relay.example.com/privkey.pem;
-        ssl_protocols TLSv1.3 TLSv1.2;
-        ssl_prefer_server_ciphers on;
-        ssl_ecdh_curve secp521r1:secp384r1;
-        ssl_ciphers EECDH+AESGCM:EECDH+AES256;
-
-        # Optional Diffie-Helmann parameters
-        # Generate with openssl dhparam -out /etc/ssl/certs/dhparam.pem 4096
-        #ssl_dhparam /etc/ssl/certs/dhparam.pem;
-
-        ssl_session_cache shared:TLS:2m;
-        ssl_buffer_size 4k;
-
-        # OCSP stapling
-        ssl_stapling on;
-        ssl_stapling_verify on;
-        resolver 1.1.1.1 1.0.0.1 [2606:4700:4700::1111] [2606:4700:4700::1001]; # Cloudflare
-
-        # Set HSTS to 365 days
-        add_header Strict-Transport-Security 'max-age=31536000; includeSubDomains; preload' always;
-        keepalive_timeout 70;
-
-        location / {
-            proxy_pass http://localhost:8080;
-            proxy_http_version 1.1;
-            proxy_read_timeout 1d;
-            proxy_send_timeout 1d;
-            proxy_set_header Upgrade $http_upgrade;
-            proxy_set_header Connection "Upgrade";
-            proxy_set_header Host $host;
-        }
-    }
-}
-```
-
-### Nginx Notes
-
-The above configuration was tested on `nginx` `1.18.0` on `Ubuntu` `20.04` and `22.04`
-
-For help installing `nginx` on `Ubuntu`, see [this guide](https://www.digitalocean.com/community/tutorials/how-to-install-nginx-on-ubuntu-20-04).
-
-For guidance on using `letsencrypt` to obtain a cert on `Ubuntu`, including an `nginx` plugin, see [this post](https://www.digitalocean.com/community/tutorials/how-to-secure-nginx-with-let-s-encrypt-on-ubuntu-20-04).
-
-
-## Example Traefik Configuration
-
-Assumptions:
-
-* `Traefik` version is `2.9` (other versions not tested).
-* `Traefik` is used for provisioning of Let's Encrypt certificates.
-* `Traefik` is running in `Docker`, using `docker compose` and labels for the static configuration. An equivalent setup using a Traefik config file is possible too (but not covered here).
-* Strict Transport Security is enabled.
-* Hostname for the relay is `relay.example.com`, email address for ACME certificates provider is `name@example.com`.
-* ipv6 is enabled, a viable private ipv6 subnet is specified in the example below.
-* Relay is running on port `8080`.
-
-```
-version: '3'
-
-networks:
-  nostr:
-    enable_ipv6: true
-    ipam:
-      config:
-        - subnet: fd00:db8:a::/64
-          gateway: fd00:db8:a::1
-
-services:
-  traefik:
-    image: traefik:v2.9
-    networks:
-      nostr:
-    command:
-      - "--log.level=ERROR"
-      # letsencrypt configuration
-      - "--certificatesResolvers.http.acme.email==name@example.com"
-      - "--certificatesResolvers.http.acme.storage=/certs/acme.json"
-      - "--certificatesResolvers.http.acme.httpChallenge.entryPoint=http"
-      # define entrypoints
-      - "--entryPoints.http.address=:80"
-      - "--entryPoints.http.http.redirections.entryPoint.to=https"
-      - "--entryPoints.http.http.redirections.entryPoint.scheme=https"
-      - "--entryPoints.https.address=:443"
-      - "--entryPoints.https.forwardedHeaders.insecure=true"
-      - "--entryPoints.https.proxyProtocol.insecure=true"
-      # docker provider (get configuration from container labels)
-      - "--providers.docker.endpoint=unix:///var/run/docker.sock"
-      - "--providers.docker.exposedByDefault=false"
-      - "--providers.file.directory=/config"
-      - "--providers.file.watch=true"
-    ports:
-      - "80:80"
-      - "443:443"
-    volumes:
-      - "/var/run/docker.sock:/var/run/docker.sock:ro"
-      - "$(pwd)/traefik/certs:/certs"
-      - "$(pwd)/traefik/config:/config"
-    logging:
-      driver: "local"
-    restart: always
-
-  # example nostr config. only labels: section is relevant for Traefik config
-  nostr:
-   image: nostr-rs-relay:latest
-   container_name: nostr-relay
-   networks:
-     nostr:
-   restart: always
-   user: 100:100
-   volumes:
-   - '$(pwd)/nostr/data:/usr/src/app/db:Z'
-   - '$(pwd)/nostr/config/config.toml:/usr/src/app/config.toml:ro,Z'
-   labels:
-     - "traefik.enable=true"
-     - "traefik.http.routers.nostr.entrypoints=https"
-     - "traefik.http.routers.nostr.rule=Host(`relay.example.com`)"
-     - "traefik.http.routers.nostr.tls.certresolver=http"
-     - "traefik.http.routers.nostr.service=nostr"
-     - "traefik.http.services.nostr.loadbalancer.server.port=8080"
-     - "traefik.http.services.nostr.loadbalancer.passHostHeader=true"
-     - "traefik.http.middlewares.nostr.headers.sslredirect=true"
-     - "traefik.http.middlewares.nostr.headers.stsincludesubdomains=true"
-     - "traefik.http.middlewares.nostr.headers.stspreload=true"
-     - "traefik.http.middlewares.nostr.headers.stsseconds=63072000"
-     - "traefik.http.routers.nostr.middlewares=nostr"
-```
-
-### Traefik Notes
-
-Traefik will take care of the provisioning and renewal of certificates. In case of an ipv4-only relay, simply detele the `enable_ipv6:` and `ipam:` entries in the `networks:` section of the docker-compose file.
@@ -1,39 +0,0 @@
-# Run as a linux system process
-
-Docker makes it easy to spin up and down environments but it's also possible to run `nostr-rs-relay` as a systemd linux process.
-This guide assumes you're on a Linux machine and that Rust is already installed.
-
-## Instructions
-
-### Build nostr-rs-relay from source
-Start by building the application from source. Here is how to do that:
-1. `git clone https://github.com/scsibug/nostr-rs-relay.git`
-2. `cd nostr-rs-relay`
-3. `cargo build --release`
-
-### Place the files where they belong
-We want to place the nostr-rs-relay binary and the config.toml file where they belong. While still in the root level of the nostr-rs-relay folder you cloned in last step, run the following commands:
-1. `sudo cp target/release/nostr-rs-relay /usr/local/bin/`
-2. `sudo mkdir /etc/nostr-rs-relay`
-2. `sudo cp config.toml /etc/nostr-rs-relay`
-
-### Create the Systemd service file
-We need to create a new Systemd service file. These files are placed in the `/etc/systemd/system/` folder where you will find many other services running.
-
-1. `sudo vim /etc/systemd/system/nostr-rs-relay.service`
-2. Paste in the contents of [this service file](../contrib/nostr-rs-relay.service). Remember to replace the `User` value with your own username.
-3. Save the file and exit your text editor
-
-
-### Run the service
-To get the service running, we need to reload the systemd daemon and enable the service.
-
-1. `sudo systemctl daemon-reload`
-2. `sudo systemctl start nostr-rs-relay.service`
-3. `sudo systemctl status nostr-rs-relay.service`
-
-
-### Tips
-
-#### Logs
-The application will write logs to the journal. To read it, execute `sudo journalctl -f -u nostr-rs-relay`
@@ -179,7 +179,7 @@ attempts to persist them to disk.  Once validated and persisted, these
 events are broadcast to all subscribers.

 When verification is enabled, the writer must check to ensure a valid,
-unexpired verification record exists for the author.  All metadata
+unexpired verification record exists for the auther.  All metadata
 events (regardless of verification status) are forwarded to a verifier
 module.  If the verifier determines a new verification record is
 needed, it is also responsible for persisting and broadcasting the
@@ -1,13 +0,0 @@
-[package]
-name = "nauthz-server"
-version = "0.1.0"
-edition = "2021"
-
-[dependencies]
-# Common dependencies
-tokio = { version = "1.0", features = ["rt-multi-thread", "macros"] }
-prost = "0.11"
-tonic = "0.8.3"
-
-[build-dependencies]
-tonic-build = { version="0.8.3", features = ["prost"] }
@@ -1,7 +0,0 @@
-fn main() -> Result<(), Box<dyn std::error::Error>> {
-    tonic_build::configure()
-        .build_server(true)
-        .protoc_arg("--experimental_allow_proto3_optional")
-        .compile(&["../../proto/nauthz.proto"], &["../../proto"])?;
-    Ok(())
-}
@@ -1,60 +0,0 @@
-use tonic::{transport::Server, Request, Response, Status};
-
-use nauthz_grpc::authorization_server::{Authorization, AuthorizationServer};
-use nauthz_grpc::{Decision, EventReply, EventRequest};
-
-pub mod nauthz_grpc {
-    tonic::include_proto!("nauthz");
-}
-
-#[derive(Default)]
-pub struct EventAuthz {
-    allowed_kinds: Vec<u64>,
-}
-
-#[tonic::async_trait]
-impl Authorization for EventAuthz {
-    async fn event_admit(
-        &self,
-        request: Request<EventRequest>,
-    ) -> Result<Response<EventReply>, Status> {
-        let reply;
-        let req = request.into_inner();
-        let event = req.event.unwrap();
-        let content_prefix: String = event.content.chars().take(40).collect();
-        println!("recvd event, [kind={}, origin={:?}, nip05_domain={:?}, tag_count={}, content_sample={:?}]",
-                 event.kind, req.origin, req.nip05.map(|x| x.domain), event.tags.len(), content_prefix);
-        // Permit any event with a whitelisted kind
-        if self.allowed_kinds.contains(&event.kind) {
-            println!("This looks fine! (kind={})", event.kind);
-            reply = nauthz_grpc::EventReply {
-                decision: Decision::Permit as i32,
-                message: None,
-            };
-        } else {
-            println!("Blocked! (kind={})", event.kind);
-            reply = nauthz_grpc::EventReply {
-                decision: Decision::Deny as i32,
-                message: Some(format!("kind {} not permitted", event.kind)),
-            };
-        }
-        Ok(Response::new(reply))
-    }
-}
-
-#[tokio::main]
-async fn main() -> Result<(), Box<dyn std::error::Error>> {
-    let addr = "[::1]:50051".parse().unwrap();
-
-    // A simple authorization engine that allows kinds 0-3
-    let checker = EventAuthz {
-        allowed_kinds: vec![0, 1, 2, 3],
-    };
-    println!("EventAuthz Server listening on {}", addr);
-    // Start serving
-    Server::builder()
-        .add_service(AuthorizationServer::new(checker))
-        .serve(addr)
-        .await?;
-    Ok(())
-}
@@ -0,0 +1,3 @@
+#!/usr/bin/env bash
+sed -E 's/@sha256:[[:alnum:]]+//g' Dockerfile > Dockerfile.any-platform
+echo "Created platform-agnostic Dockerfile in 'Dockerfile.any-platform'"
@@ -1,60 +0,0 @@
-syntax = "proto3";
-
-// Nostr Authorization Services
-package nauthz;
-
-// Authorization for actions against a relay
-service Authorization {
-  // Determine if an event should be admitted to the relay
-  rpc EventAdmit(EventRequest) returns (EventReply) {}
-}
-
-message Event {
-  bytes id = 1;                // 32-byte SHA256 hash of serialized event
-  bytes pubkey = 2;            // 32-byte public key of event creator
-  fixed64 created_at = 3;      // UNIX timestamp provided by event creator
-  uint64 kind = 4;             // event kind
-  string content = 5;          // arbitrary event contents
-  repeated TagEntry tags = 6;  // event tag array
-  bytes sig = 7;               // 32-byte signature of the event id
-  // Individual values for a single tag
-  message TagEntry {
-    repeated string values = 1;
-  }
-}
-
-// Event data and metadata for authorization decisions
-message EventRequest {
-  Event event =
-      1;  // the event to be admitted for further relay processing
-  optional string ip_addr =
-      2;  // IP address of the client that submitted the event
-  optional string origin =
-      3;  // HTTP origin header from the client, if one exists
-  optional string user_agent =
-      4;  // HTTP user-agent header from the client, if one exists
-  optional bytes auth_pubkey =
-      5;  // the public key associated with a NIP-42 AUTH'd session, if
-          // authentication occurred
-  optional Nip05Name nip05 =
-      6; // NIP-05 address associated with the event pubkey, if it is
-         // known and has been validated by the relay
-  // A NIP_05 verification record
-  message Nip05Name {
-    string local = 1;
-    string domain = 2;
-  }
-}
-
-// A permit or deny decision
-enum Decision {
-  DECISION_UNSPECIFIED = 0;
-  DECISION_PERMIT = 1; // Admit this event for further processing
-  DECISION_DENY = 2; // Deny persisting or propagating this event
-}
-
-// Response to a event authorization request
-message EventReply {
-  Decision decision = 1;       // decision to enforce
-  optional string message = 2; // informative message for the client
-}
@@ -0,0 +1,92 @@
+# Reverse Proxy Setup Guide
+
+It is recommended to run `nostr-rs-relay` behind a reverse proxy such
+as `haproxy` or `nginx` to provide TLS termination.  Simple examples
+of `haproxy` and `nginx` configurations are documented here.
+
+## Minimal HAProxy Configuration
+
+Assumptions:
+
+* HAProxy version is `2.4.10` or greater (older versions not tested).
+* Hostname for the relay is `relay.example.com`.
+* Your relay should be available over wss://relay.example.com
+* Your (NIP-11) relay info page should be available on https://relay.example.com
+* SSL certificate is located in `/etc/certs/example.com.pem`.
+* Relay is running on port 8080.
+* Limit connections to 400 concurrent.
+* HSTS (HTTP Strict Transport Security) is desired.
+* Only TLS 1.2 or greater is allowed.
+
+```
+global
+    ssl-default-bind-ciphersuites TLS_AES_128_GCM_SHA256:TLS_AES_256_GCM_SHA384:TLS_CHACHA20_POLY1305_SHA256
+    ssl-default-bind-options prefer-client-ciphers no-sslv3 no-tlsv10 no-tlsv11 no-tls-tickets
+
+frontend fe_prod
+    mode    http
+    bind    :443 ssl crt /etc/certs/example.com.pem alpn h2,http/1.1
+    bind    :80
+    http-request set-header X-Forwarded-Proto https if { ssl_fc }
+    redirect scheme https code 301 if !{ ssl_fc }
+    acl host_relay hdr(host) -i relay.example.com
+    use_backend relay if host_relay
+    # HSTS (1 year)
+    http-response set-header Strict-Transport-Security max-age=31536000
+
+backend relay
+    mode http
+    timeout connect 5s
+    timeout client 50s
+    timeout server 50s
+    timeout tunnel 1h
+    timeout client-fin 30s
+    option tcp-check
+    default-server maxconn 400 check inter 20s fastinter 1s
+    server relay 127.0.0.1:8080
+```
+
+### HAProxy Notes
+
+You may experience WebSocket connection problems with Firefox if
+HTTP/2 is enabled, for older versions of HAProxy (2.3.x).  Either
+disable HTTP/2 (`h2`), or upgrade HAProxy.
+
+## Bare-bones Nginx Configuration
+
+Assumptions:
+
+* `Nginx` version is `1.18.0` (other versions not tested).
+* Hostname for the relay is `relay.example.com`.
+* SSL certificate and key are located at `/etc/letsencrypt/live/relay.example.com/`.
+* Relay is running on port `8080`.
+
+```
+http {
+    server {
+        listen 443 ssl;
+        server_name relay.example.com;
+        ssl_certificate /etc/letsencrypt/live/relay.example.com/fullchain.pem;
+        ssl_certificate_key /etc/letsencrypt/live/relay.example.com/privkey.pem;
+        ssl_protocols TLSv1 TLSv1.1 TLSv1.2;
+        ssl_ciphers HIGH:!aNULL:!MD5;
+        keepalive_timeout 70;
+
+        location / {
+            proxy_pass http://localhost:8080;
+            proxy_http_version 1.1;
+            proxy_set_header Upgrade $http_upgrade;
+            proxy_set_header Connection "Upgrade";
+            proxy_set_header Host $host;
+        }
+    }
+}
+```
+
+### Nginx Notes
+
+The above configuration was tested on `nginx` `1.18.0` was tested on `Ubuntu 20.04`.
+
+For help installing `nginx` on `Ubuntu`, see [this guide](https://www.digitalocean.com/community/tutorials/how-to-install-nginx-on-ubuntu-20-04).
+
+For guidance on using `letsencrypt` to obtain a cert on `Ubuntu`, including an `nginx` plugin, see [this post](https://www.digitalocean.com/community/tutorials/how-to-secure-nginx-with-let-s-encrypt-on-ubuntu-20-04).
@@ -1,4 +1 @@
 edition = "2021"
-#max_width = 140
-#chain_width = 100
-#fn_call_width = 100
@@ -1,181 +0,0 @@
-use nostr_rs_relay::config;
-use nostr_rs_relay::error::{Error, Result};
-use nostr_rs_relay::event::{single_char_tagname, Event};
-use nostr_rs_relay::repo::sqlite::{build_pool, PooledConnection};
-use nostr_rs_relay::repo::sqlite_migration::{curr_db_version, DB_VERSION};
-use nostr_rs_relay::utils::is_lower_hex;
-use rusqlite::params;
-use rusqlite::{OpenFlags, Transaction};
-use std::io;
-use std::path::Path;
-use std::sync::mpsc;
-use std::thread;
-use tracing::info;
-
-/// Bulk load JSONL data from STDIN to the database specified in config.toml (or ./nostr.db as a default).
-/// The database must already exist, this will not create a new one.
-/// Tested against schema v13.
-
-pub fn main() -> Result<()> {
-    let _trace_sub = tracing_subscriber::fmt::try_init();
-    println!("Nostr-rs-relay Bulk Loader");
-    // check for a database file, or create one.
-    let settings = config::Settings::new(&None)?;
-    if !Path::new(&settings.database.data_directory).is_dir() {
-        info!("Database directory does not exist");
-        return Err(Error::DatabaseDirError);
-    }
-    // Get a database pool
-    let pool = build_pool(
-        "bulk-loader",
-        &settings,
-        OpenFlags::SQLITE_OPEN_READ_WRITE | OpenFlags::SQLITE_OPEN_CREATE,
-        1,
-        4,
-        false,
-    );
-    {
-        // check for database schema version
-        let mut conn: PooledConnection = pool.get()?;
-        let version = curr_db_version(&mut conn)?;
-        info!("current version is: {:?}", version);
-        // ensure the schema version is current.
-        if version != DB_VERSION {
-            info!("version is not current, exiting");
-            panic!("cannot write to schema other than v{DB_VERSION}");
-        }
-    }
-    // this channel will contain parsed events ready to be inserted
-    let (event_tx, event_rx) = mpsc::sync_channel(100_000);
-    // Thread for reading events
-    let _stdin_reader_handler = thread::spawn(move || {
-        let stdin = io::stdin();
-        for readline in stdin.lines() {
-            if let Ok(line) = readline {
-                // try to parse a nostr event
-                let eres: Result<Event, serde_json::Error> = serde_json::from_str(&line);
-                if let Ok(mut e) = eres {
-                    if let Ok(()) = e.validate() {
-                        e.build_index();
-                        //debug!("Event: {:?}", e);
-                        event_tx.send(Some(e)).ok();
-                    } else {
-                        info!("could not validate event");
-                    }
-                } else {
-                    info!("error reading event: {:?}", eres);
-                }
-            } else {
-                // error reading
-                info!("error reading: {:?}", readline);
-            }
-        }
-        info!("finished parsing events");
-        event_tx.send(None).ok();
-        let ok: Result<()> = Ok(());
-        ok
-    });
-    let mut conn: PooledConnection = pool.get()?;
-    let mut events_read = 0;
-    let event_batch_size = 50_000;
-    let mut new_events = 0;
-    let mut has_more_events = true;
-    while has_more_events {
-        // begin a transaction
-        let tx = conn.transaction()?;
-        // read in batch_size events and commit
-        for _ in 0..event_batch_size {
-            match event_rx.recv() {
-                Ok(Some(e)) => {
-                    events_read += 1;
-                    // ignore ephemeral events
-                    if !(e.kind >= 20000 && e.kind < 30000) {
-                        match write_event(&tx, e) {
-                            Ok(c) => {
-                                new_events += c;
-                            }
-                            Err(e) => {
-                                info!("error inserting event: {:?}", e);
-                            }
-                        }
-                    }
-                }
-                Ok(None) => {
-                    // signal that the sender will never produce more
-                    // events
-                    has_more_events = false;
-                    break;
-                }
-                Err(_) => {
-                    info!("sender is closed");
-                    // sender is done
-                }
-            }
-        }
-        info!("committed {} events...", new_events);
-        tx.commit()?;
-        conn.execute_batch("pragma wal_checkpoint(truncate)")?;
-    }
-    info!("processed {} events", events_read);
-    info!("stored {} new events", new_events);
-    // get a connection for writing events
-    // read standard in.
-    info!("finished reading input");
-    Ok(())
-}
-
-/// Write an event and update the tag table.
-/// Assumes the event has its index built.
-fn write_event(tx: &Transaction, e: Event) -> Result<usize> {
-    let id_blob = hex::decode(&e.id).ok();
-    let pubkey_blob: Option<Vec<u8>> = hex::decode(&e.pubkey).ok();
-    let delegator_blob: Option<Vec<u8>> = e.delegated_by.as_ref().and_then(|d| hex::decode(d).ok());
-    let event_str = serde_json::to_string(&e).ok();
-    // ignore if the event hash is a duplicate.
-    let ins_count = tx.execute(
-	"INSERT OR IGNORE INTO event (event_hash, created_at, kind, author, delegated_by, content, first_seen, hidden) VALUES (?1, ?2, ?3, ?4, ?5, ?6, strftime('%s','now'), FALSE);",
-	params![id_blob, e.created_at, e.kind, pubkey_blob, delegator_blob, event_str]
-    )?;
-    if ins_count == 0 {
-        return Ok(0);
-    }
-    // we want to capture the event_id that had the tag, the tag name, and the tag hex value.
-    let event_id = tx.last_insert_rowid();
-    // look at each event, and each tag, creating new tag entries if appropriate.
-    for t in e.tags.iter().filter(|x| x.len() > 1) {
-        let tagname = t.get(0).unwrap();
-        let tagnamechar_opt = single_char_tagname(tagname);
-        if tagnamechar_opt.is_none() {
-            continue;
-        }
-        // safe because len was > 1
-        let tagval = t.get(1).unwrap();
-        // insert as BLOB if we can restore it losslessly.
-        // this means it needs to be even length and lowercase.
-        if (tagval.len() % 2 == 0) && is_lower_hex(tagval) {
-            tx.execute(
-                "INSERT INTO tag (event_id, name, value_hex) VALUES (?1, ?2, ?3);",
-                params![event_id, tagname, hex::decode(tagval).ok()],
-            )?;
-        } else {
-            // otherwise, insert as text
-            tx.execute(
-                "INSERT INTO tag (event_id, name, value) VALUES (?1, ?2, ?3);",
-                params![event_id, tagname, &tagval],
-            )?;
-        }
-    }
-    if e.is_replaceable() {
-        //let query = "SELECT id FROM event WHERE kind=? AND author=? ORDER BY created_at DESC LIMIT 1;";
-        //let count: usize = tx.query_row(query, params![e.kind, pubkey_blob], |row| row.get(0))?;
-        //info!("found {} rows that /would/ be preserved", count);
-        match tx.execute(
-	    "DELETE FROM event WHERE kind=? and author=? and id NOT IN (SELECT id FROM event WHERE kind=? AND author=? ORDER BY created_at DESC LIMIT 1);",
-	    params![e.kind, pubkey_blob, e.kind, pubkey_blob],
-	) {
-	    Ok(_) => {},
-	    Err(x) => {info!("error deleting replaceable event: {:?}",x);}
-	}
-    }
-    Ok(ins_count)
-}
@@ -1,20 +0,0 @@
-use clap::Parser;
-
-#[derive(Parser)]
-#[command(about = "A nostr relay written in Rust", author = env!("CARGO_PKG_AUTHORS"), version = env!("CARGO_PKG_VERSION"))]
-pub struct CLIArgs {
-    #[arg(
-        short,
-        long,
-        help = "Use the <directory> as the location of the database",
-        required = false
-    )]
-    pub db: Option<String>,
-    #[arg(
-        short,
-        long,
-        help = "Use the <file name> as the location of the config file",
-        required = false
-    )]
-    pub config: Option<String>,
-}
@@ -1,8 +1,8 @@
 //! Configuration file and settings management
-use crate::payment::Processor;
 use config::{Config, ConfigError, File};
 use serde::{Deserialize, Serialize};
 use std::time::Duration;
+use tracing::warn;

 #[derive(Debug, Serialize, Deserialize, Clone)]
 #[allow(unused)]
@@ -12,26 +12,15 @@ pub struct Info {
    pub description: Option<String>,
    pub pubkey: Option<String>,
    pub contact: Option<String>,
-    pub favicon: Option<String>,
-    pub relay_icon: Option<String>,
 }

 #[derive(Debug, Clone, Serialize, Deserialize)]
 #[allow(unused)]
 pub struct Database {
    pub data_directory: String,
-    pub engine: String,
    pub in_memory: bool,
    pub min_conn: u32,
    pub max_conn: u32,
-    pub connection: String,
-    pub connection_write: Option<String>,
-}
-
-#[derive(Debug, Clone, Serialize, Deserialize)]
-#[allow(unused)]
-pub struct Grpc {
-    pub event_admission_server: Option<String>,
 }

 #[derive(Debug, Clone, Serialize, Deserialize)]
@@ -63,39 +52,17 @@ pub struct Retention {
 #[allow(unused)]
 pub struct Limits {
    pub messages_per_sec: Option<u32>, // Artificially slow down event writing to limit disk consumption (averaged over 1 minute)
-    pub subscriptions_per_min: Option<u32>, // Artificially slow down request (db query) creation to prevent abuse (averaged over 1 minute)
-    pub db_conns_per_client: Option<u32>, // How many concurrent database queries (not subscriptions) may a client have?
-    pub max_blocking_threads: usize,
    pub max_event_bytes: Option<usize>, // Maximum size of an EVENT message
    pub max_ws_message_bytes: Option<usize>,
    pub max_ws_frame_bytes: Option<usize>,
    pub broadcast_buffer: usize, // events to buffer for subscribers (prevents slow readers from consuming memory)
    pub event_persist_buffer: usize, // events to buffer for database commits (block senders if database writes are too slow)
-    pub event_kind_blacklist: Option<Vec<u64>>,
-    pub event_kind_allowlist: Option<Vec<u64>>,
 }

 #[derive(Debug, Clone, Serialize, Deserialize)]
 #[allow(unused)]
 pub struct Authorization {
    pub pubkey_whitelist: Option<Vec<String>>, // If present, only allow these pubkeys to publish events
-    pub nip42_auth: bool,                      // if true enables NIP-42 authentication
-    pub nip42_dms: bool, // if true send DMs only to their authenticated recipients
-}
-
-#[derive(Debug, Clone, Serialize, Deserialize)]
-#[allow(unused)]
-pub struct PayToRelay {
-    pub enabled: bool,
-    pub admission_cost: u64, // Cost to have pubkey whitelisted
-    pub cost_per_event: u64, // Cost author to pay per event
-    pub node_url: String,
-    pub api_secret: String,
-    pub terms_message: String,
-    pub sign_ups: bool,       // allow new users to sign up to relay
-    pub direct_message: bool, // Send direct message to user with invoice and terms
-    pub secret_key: Option<String>,
-    pub processor: Processor,
 }

 #[derive(Debug, Clone, Serialize, Deserialize)]
@@ -166,66 +133,42 @@ impl VerifiedUsers {
    }
 }

-#[derive(Debug, Clone, Serialize, Deserialize)]
-#[allow(unused)]
-pub struct Logging {
-    pub folder_path: Option<String>,
-    pub file_prefix: Option<String>,
-}
-
 #[derive(Debug, Clone, Serialize, Deserialize)]
 #[allow(unused)]
 pub struct Settings {
    pub info: Info,
    pub diagnostics: Diagnostics,
    pub database: Database,
-    pub grpc: Grpc,
    pub network: Network,
    pub limits: Limits,
    pub authorization: Authorization,
-    pub pay_to_relay: PayToRelay,
    pub verified_users: VerifiedUsers,
    pub retention: Retention,
    pub options: Options,
-    pub logging: Logging,
 }

 impl Settings {
-    pub fn new(config_file_name: &Option<String>) -> Result<Self, ConfigError> {
+    #[must_use]
+    pub fn new() -> Self {
        let default_settings = Self::default();
        // attempt to construct settings with file
-        let from_file = Self::new_from_default(&default_settings, config_file_name);
+        let from_file = Self::new_from_default(&default_settings);
        match from_file {
+            Ok(f) => f,
            Err(e) => {
-                // pass up the parse error if the config file was specified,
-                // otherwise use the default config (with a warning).
-                if config_file_name.is_some() {
-                    Err(e)
-                } else {
-                    eprintln!("Error reading config file ({:?})", e);
-                    eprintln!("WARNING: Default configuration settings will be used");
-                    Ok(default_settings)
-                }
+                warn!("Error reading config file ({:?})", e);
+                default_settings
            }
-            ok => ok,
        }
    }

-    fn new_from_default(
-        default: &Settings,
-        config_file_name: &Option<String>,
-    ) -> Result<Self, ConfigError> {
-        let default_config_file_name = "config.toml".to_string();
-        let config: &String = match config_file_name {
-            Some(value) => value,
-            None => &default_config_file_name,
-        };
+    fn new_from_default(default: &Settings) -> Result<Self, ConfigError> {
        let builder = Config::builder();
        let config: Config = builder
            // use defaults
            .add_source(Config::try_from(default)?)
            // override with file contents
-            .add_source(File::with_name(config))
+            .add_source(File::with_name("config.toml"))
            .build()?;
        let mut settings: Settings = config.try_deserialize()?;
        // ensure connection pool size is logical
@@ -242,23 +185,6 @@ impl Settings {
        );
        // initialize durations for verified users
        settings.verified_users.init();
-
-        // Validate pay to relay settings
-        if settings.pay_to_relay.enabled {
-            assert_ne!(settings.pay_to_relay.api_secret, "");
-            // Should check that url is valid
-            assert_ne!(settings.pay_to_relay.node_url, "");
-            assert_ne!(settings.pay_to_relay.terms_message, "");
-
-            if settings.pay_to_relay.direct_message {
-                assert_ne!(
-                    settings.pay_to_relay.secret_key,
-                    Some("<nostr nsec>".to_string())
-                );
-                assert!(settings.pay_to_relay.secret_key.is_some());
-            }
-        }
-
        Ok(settings)
    }
 }
@@ -272,21 +198,13 @@ impl Default for Settings {
                description: None,
                pubkey: None,
                contact: None,
-                favicon: None,
-                relay_icon: None,
            },
            diagnostics: Diagnostics { tracing: false },
            database: Database {
                data_directory: ".".to_owned(),
-                engine: "sqlite".to_owned(),
                in_memory: false,
                min_conn: 4,
-                max_conn: 8,
-                connection: "".to_owned(),
-                connection_write: None,
-            },
-            grpc: Grpc {
-                event_admission_server: None,
+                max_conn: 128,
            },
            network: Network {
                port: 8080,
@@ -296,33 +214,14 @@ impl Default for Settings {
            },
            limits: Limits {
                messages_per_sec: None,
-                subscriptions_per_min: None,
-                db_conns_per_client: None,
-                max_blocking_threads: 16,
                max_event_bytes: Some(2 << 17),      // 128K
                max_ws_message_bytes: Some(2 << 17), // 128K
                max_ws_frame_bytes: Some(2 << 17),   // 128K
                broadcast_buffer: 16384,
                event_persist_buffer: 4096,
-                event_kind_blacklist: None,
-                event_kind_allowlist: None,
            },
            authorization: Authorization {
                pubkey_whitelist: None, // Allow any address to publish
-                nip42_auth: false,      // Disable NIP-42 authentication
-                nip42_dms: false,       // Send DMs to everybody
-            },
-            pay_to_relay: PayToRelay {
-                enabled: false,
-                admission_cost: 4200,
-                cost_per_event: 0,
-                terms_message: "".to_string(),
-                node_url: "".to_string(),
-                api_secret: "".to_string(),
-                sign_ups: false,
-                direct_message: true,
-                secret_key: None,
-                processor: Processor::LNBits,
            },
            verified_users: VerifiedUsers {
                mode: VerifiedUsersMode::Disabled,
@@ -343,10 +242,6 @@ impl Default for Settings {
            options: Options {
                reject_future_seconds: None, // Reject events in the future if defined
            },
-            logging: Logging {
-                folder_path: None,
-                file_prefix: None,
-            },
        }
    }
 }
@@ -1,42 +1,26 @@
 //! Client connection state
-use std::collections::HashMap;
-
-use tracing::{debug, trace};
-use uuid::Uuid;
-
 use crate::close::Close;
-use crate::conn::Nip42AuthState::{AuthPubkey, Challenge, NoAuth};
 use crate::error::Error;
 use crate::error::Result;
-use crate::event::Event;
+
 use crate::subscription::Subscription;
-use crate::utils::{host_str, unix_time};
+use std::collections::HashMap;
+use tracing::{debug, info};
+use uuid::Uuid;

 /// A subscription identifier has a maximum length
 const MAX_SUBSCRIPTION_ID_LEN: usize = 256;

-/// NIP-42 authentication state
-pub enum Nip42AuthState {
-    /// The client is not authenticated yet
-    NoAuth,
-    /// The AUTH challenge sent
-    Challenge(String),
-    /// The client is authenticated
-    AuthPubkey(String),
-}
-
 /// State for a client connection
 pub struct ClientConn {
    /// Client IP (either from socket, or configured proxy header
-    client_ip_addr: String,
+    client_ip: String,
    /// Unique client identifier generated at connection time
    client_id: Uuid,
    /// The current set of active client subscriptions
    subscriptions: HashMap<String, Subscription>,
    /// Per-connection maximum concurrent subscriptions
    max_subs: usize,
-    /// NIP-42 AUTH
-    auth: Nip42AuthState,
 }

 impl Default for ClientConn {
@@ -48,28 +32,20 @@ impl Default for ClientConn {
 impl ClientConn {
    /// Create a new, empty connection state.
    #[must_use]
-    pub fn new(client_ip_addr: String) -> Self {
+    pub fn new(client_ip: String) -> Self {
        let client_id = Uuid::new_v4();
        ClientConn {
-            client_ip_addr,
+            client_ip,
            client_id,
            subscriptions: HashMap::new(),
            max_subs: 32,
-            auth: NoAuth,
        }
    }

-    #[must_use]
    pub fn subscriptions(&self) -> &HashMap<String, Subscription> {
        &self.subscriptions
    }

-    /// Check if the given subscription already exists
-    #[must_use]
-    pub fn has_subscription(&self, sub: &Subscription) -> bool {
-        self.subscriptions.values().any(|x| x == sub)
-    }
-
    /// Get a short prefix of the client's unique identifier, suitable
    /// for logging.
    #[must_use]
@@ -79,23 +55,7 @@ impl ClientConn {

    #[must_use]
    pub fn ip(&self) -> &str {
-        &self.client_ip_addr
-    }
-
-    #[must_use]
-    pub fn auth_pubkey(&self) -> Option<&String> {
-        match &self.auth {
-            AuthPubkey(pubkey) => Some(pubkey),
-            _ => None,
-        }
-    }
-
-    #[must_use]
-    pub fn auth_challenge(&self) -> Option<&String> {
-        match &self.auth {
-            Challenge(pubkey) => Some(pubkey),
-            _ => None,
-        }
+        &self.client_ip
    }

    /// Add a new subscription for this connection.
@@ -109,7 +69,7 @@ impl ClientConn {
        // prevent arbitrarily long subscription identifiers from
        // being used.
        if sub_id_len > MAX_SUBSCRIPTION_ID_LEN {
-            debug!(
+            info!(
                "ignoring sub request with excessive length: ({})",
                sub_id_len
            );
@@ -118,12 +78,8 @@ impl ClientConn {
        // check if an existing subscription exists, and replace if so
        if self.subscriptions.contains_key(&k) {
            self.subscriptions.remove(&k);
-            self.subscriptions.insert(k, s.clone());
-            trace!(
-                "replaced existing subscription (cid: {}, sub: {:?})",
-                self.get_client_prefix(),
-                s.get_id()
-            );
+            self.subscriptions.insert(k, s);
+            debug!("replaced existing subscription");
            return Ok(());
        }

@@ -133,10 +89,9 @@ impl ClientConn {
        }
        // add subscription
        self.subscriptions.insert(k, s);
-        trace!(
-            "registered new subscription, currently have {} active subs (cid: {})",
-            self.subscriptions.len(),
-            self.get_client_prefix(),
+        debug!(
+            "registered new subscription, currently have {} active subs",
+            self.subscriptions.len()
        );
        Ok(())
    }
@@ -145,85 +100,10 @@ impl ClientConn {
    pub fn unsubscribe(&mut self, c: &Close) {
        // TODO: return notice if subscription did not exist.
        self.subscriptions.remove(&c.id);
-        trace!(
-            "removed subscription, currently have {} active subs (cid: {})",
+        debug!(
+            "removed subscription, currently have {} active subs (cid={})",
            self.subscriptions.len(),
            self.get_client_prefix(),
        );
    }
-
-    pub fn generate_auth_challenge(&mut self) {
-        self.auth = Challenge(Uuid::new_v4().to_string());
-    }
-
-    pub fn authenticate(&mut self, event: &Event, relay_url: &str) -> Result<()> {
-        match &self.auth {
-            Challenge(_) => (),
-            AuthPubkey(_) => {
-                // already authenticated
-                return Ok(());
-            }
-            NoAuth => {
-                // unexpected AUTH request
-                return Err(Error::AuthFailure);
-            }
-        }
-        match event.validate() {
-            Ok(_) => {
-                if event.kind != 22242 {
-                    return Err(Error::AuthFailure);
-                }
-
-                let curr_time = unix_time();
-                let past_cutoff = curr_time - 600; // 10 minutes
-                let future_cutoff = curr_time + 600; // 10 minutes
-                if event.created_at < past_cutoff || event.created_at > future_cutoff {
-                    return Err(Error::AuthFailure);
-                }
-
-                let mut challenge: Option<&str> = None;
-                let mut relay: Option<&str> = None;
-
-                for tag in &event.tags {
-                    if tag.len() == 2 && tag.get(0) == Some(&"challenge".into()) {
-                        challenge = tag.get(1).map(|x| x.as_str());
-                    }
-                    if tag.len() == 2 && tag.get(0) == Some(&"relay".into()) {
-                        relay = tag.get(1).map(|x| x.as_str());
-                    }
-                }
-
-                match (challenge, &self.auth) {
-                    (Some(received_challenge), Challenge(sent_challenge)) => {
-                        if received_challenge != sent_challenge {
-                            return Err(Error::AuthFailure);
-                        }
-                    }
-                    (_, _) => {
-                        return Err(Error::AuthFailure);
-                    }
-                }
-
-                match (relay.and_then(host_str), host_str(relay_url)) {
-                    (Some(received_relay), Some(our_relay)) => {
-                        if received_relay != our_relay {
-                            return Err(Error::AuthFailure);
-                        }
-                    }
-                    (_, _) => {
-                        return Err(Error::AuthFailure);
-                    }
-                }
-
-                self.auth = AuthPubkey(event.pubkey.clone());
-                trace!(
-                    "authenticated pubkey {} (cid: {})",
-                    event.pubkey.chars().take(8).collect::<String>(),
-                    self.get_client_prefix()
-                );
-                Ok(())
-            }
-            Err(_) => Err(Error::AuthFailure),
-        }
-    }
 }
@@ -80,11 +80,10 @@ impl FromStr for Operator {

 #[derive(Serialize, Deserialize, PartialEq, Eq, Debug, Clone)]
 pub struct ConditionQuery {
-    pub conditions: Vec<Condition>,
+    pub(crate) conditions: Vec<Condition>,
 }

 impl ConditionQuery {
-    #[must_use]
    pub fn allows_event(&self, event: &Event) -> bool {
        // check each condition, to ensure that the event complies
        // with the restriction.
@@ -102,7 +101,6 @@ impl ConditionQuery {
 }

 // Verify that the delegator approved the delegation; return a ConditionQuery if so.
-#[must_use]
 pub fn validate_delegation(
    delegator: &str,
    delegatee: &str,
@@ -110,7 +108,7 @@ pub fn validate_delegation(
    sigstr: &str,
 ) -> Option<ConditionQuery> {
    // form the token
-    let tok = format!("nostr:delegation:{delegatee}:{cond_query}");
+    let tok = format!("nostr:delegation:{}:{}", delegatee, cond_query);
    // form SHA256 hash
    let digest: sha256::Hash = sha256::Hash::hash(tok.as_bytes());
    let sig = schnorr::Signature::from_str(sigstr).unwrap();
@@ -135,18 +133,17 @@ pub fn validate_delegation(
 }

 /// Parsed delegation condition
-/// see <https://github.com/nostr-protocol/nips/pull/28#pullrequestreview-1084903800>
-/// An example complex condition would be:  `kind=1,2,3&created_at<1665265999`
+/// see https://github.com/nostr-protocol/nips/pull/28#pullrequestreview-1084903800
+/// An example complex condition would be:  kind=1,2,3&created_at<1665265999
 #[derive(Serialize, Deserialize, PartialEq, Eq, Debug, Clone)]
 pub struct Condition {
-    pub field: Field,
-    pub operator: Operator,
-    pub values: Vec<u64>,
+    pub(crate) field: Field,
+    pub(crate) operator: Operator,
+    pub(crate) values: Vec<u64>,
 }

 impl Condition {
    /// Check if this condition allows the given event to be delegated
-    #[must_use]
    pub fn allows_event(&self, event: &Event) -> bool {
        // determine what the right-hand side of the operator is
        let resolved_field = match &self.field {
@@ -326,7 +323,7 @@ mod tests {
                Condition {
                    field: Field::CreatedAt,
                    operator: Operator::LessThan,
-                    values: vec![1_665_867_123],
+                    values: vec![1665867123],
                },
            ],
        };
@@ -335,6 +332,19 @@ mod tests {
        assert_eq!(parsed, cq);
        Ok(())
    }
+    fn simple_event() -> Event {
+        Event {
+            id: "0".to_owned(),
+            pubkey: "0".to_owned(),
+            delegated_by: None,
+            created_at: 0,
+            kind: 0,
+            tags: vec![],
+            content: "".to_owned(),
+            sig: "0".to_owned(),
+            tagidx: None,
+        }
+    }
    // Check for condition logic on event w/ empty values
    #[test]
    fn condition_with_empty_values() {
@@ -343,7 +353,7 @@ mod tests {
            operator: Operator::GreaterThan,
            values: vec![],
        };
-        let e = Event::simple_event();
+        let e = simple_event();
        assert!(!c.allows_event(&e));
        c.operator = Operator::LessThan;
        assert!(!c.allows_event(&e));
@@ -363,7 +373,7 @@ mod tests {
            operator: Operator::GreaterThan,
            values: vec![10],
        };
-        let mut e = Event::simple_event();
+        let mut e = simple_event();
        // kind is not greater than 10, not allowed
        e.kind = 1;
        assert!(!c.allows_event(&e));
@@ -382,7 +392,7 @@ mod tests {
            operator: Operator::Equals,
            values: vec![0, 10, 20],
        };
-        let mut e = Event::simple_event();
+        let mut e = simple_event();
        // Allow if event kind is in list for Equals
        e.kind = 10;
        assert!(c.allows_event(&e));
@@ -17,16 +17,10 @@ pub enum Error {
    ConnWriteError,
    #[error("EVENT parse failed")]
    EventParseFailed,
-    #[error("CLOSE message parse failed")]
+    #[error("ClOSE message parse failed")]
    CloseParseFailed,
-    #[error("Event invalid signature")]
-    EventInvalidSignature,
-    #[error("Event invalid id")]
-    EventInvalidId,
-    #[error("Event malformed pubkey")]
-    EventMalformedPubkey,
-    #[error("Event could not canonicalize")]
-    EventCouldNotCanonicalize,
+    #[error("Event validation failed")]
+    EventInvalid,
    #[error("Event too large")]
    EventMaxLengthError(usize),
    #[error("Subscription identifier max length exceeded")]
@@ -48,10 +42,6 @@ pub enum Error {
    DatabaseDirError,
    #[error("Database Connection Pool Error")]
    DatabasePoolError(r2d2::Error),
-    #[error("SQL error")]
-    SqlxError(sqlx::Error),
-    #[error("Database Connection Pool Error")]
-    SqlxDatabasePoolError(sqlx::Error),
    #[error("Custom Error : {0}")]
    CustomError(String),
    #[error("Task join error")]
@@ -62,26 +52,6 @@ pub enum Error {
    HexError(hex::FromHexError),
    #[error("Delegation parse error")]
    DelegationParseError,
-    #[error("Channel closed error")]
-    ChannelClosed,
-    #[error("Authz error")]
-    AuthzError,
-    #[error("Tonic GRPC error")]
-    TonicError(tonic::Status),
-    #[error("Invalid AUTH message")]
-    AuthFailure,
-    #[error("I/O Error")]
-    IoError(std::io::Error),
-    #[error("Event builder error")]
-    EventError(nostr::event::builder::Error),
-    #[error("Nostr key error")]
-    NostrKeyError(nostr::key::Error),
-    #[error("Payment hash mismatch")]
-    PaymentHash,
-    #[error("Error parsing url")]
-    URLParseError(url::ParseError),
-    #[error("HTTP error")]
-    HTTPError(http::Error),
    #[error("Unknown/Undocumented")]
    UnknownError,
 }
@@ -124,12 +94,6 @@ impl From<rusqlite::Error> for Error {
    }
 }

-impl From<sqlx::Error> for Error {
-    fn from(d: sqlx::Error) -> Self {
-        Error::SqlxDatabasePoolError(d)
-    }
-}
-
 impl From<serde_json::Error> for Error {
    /// Wrap JSON error
    fn from(r: serde_json::Error) -> Self {
@@ -150,43 +114,3 @@ impl From<config::ConfigError> for Error {
        Error::ConfigError(r)
    }
 }
-
-impl From<tonic::Status> for Error {
-    /// Wrap Config error
-    fn from(r: tonic::Status) -> Self {
-        Error::TonicError(r)
-    }
-}
-
-impl From<std::io::Error> for Error {
-    fn from(r: std::io::Error) -> Self {
-        Error::IoError(r)
-    }
-}
-impl From<nostr::event::builder::Error> for Error {
-    /// Wrap event builder error
-    fn from(r: nostr::event::builder::Error) -> Self {
-        Error::EventError(r)
-    }
-}
-
-impl From<nostr::key::Error> for Error {
-    /// Wrap nostr key error
-    fn from(r: nostr::key::Error) -> Self {
-        Error::NostrKeyError(r)
-    }
-}
-
-impl From<url::ParseError> for Error {
-    /// Wrap nostr key error
-    fn from(r: url::ParseError) -> Self {
-        Error::URLParseError(r)
-    }
-}
-
-impl From<http::Error> for Error {
-    /// Wrap nostr key error
-    fn from(r: http::Error) -> Self {
-        Error::HTTPError(r)
-    }
-}
@@ -1,12 +1,7 @@
 //! Event parsing and validation
 use crate::delegation::validate_delegation;
-use crate::error::Error::{
-    CommandUnknownError, EventCouldNotCanonicalize, EventInvalidId, EventInvalidSignature,
-    EventMalformedPubkey,
-};
+use crate::error::Error::*;
 use crate::error::Result;
-use crate::event::EventWrapper::WrappedAuth;
-use crate::event::EventWrapper::WrappedEvent;
 use crate::nip05;
 use crate::utils::unix_time;
 use bitcoin_hashes::{sha256, Hash};
@@ -32,30 +27,23 @@ pub struct EventCmd {
    event: Event,
 }

-impl EventCmd {
-    #[must_use]
-    pub fn event_id(&self) -> &str {
-        &self.event.id
-    }
-}
-
 /// Parsed nostr event.
 #[derive(Serialize, Deserialize, PartialEq, Eq, Debug, Clone)]
 pub struct Event {
    pub id: String,
-    pub pubkey: String,
+    pub(crate) pubkey: String,
    #[serde(skip)]
-    pub delegated_by: Option<String>,
-    pub created_at: u64,
-    pub kind: u64,
+    pub(crate) delegated_by: Option<String>,
+    pub(crate) created_at: u64,
+    pub(crate) kind: u64,
    #[serde(deserialize_with = "tag_from_string")]
    // NOTE: array-of-arrays may need to be more general than a string container
-    pub tags: Vec<Vec<String>>,
-    pub content: String,
-    pub sig: String,
+    pub(crate) tags: Vec<Vec<String>>,
+    pub(crate) content: String,
+    pub(crate) sig: String,
    // Optimization for tag search, built on demand.
    #[serde(skip)]
-    pub tagidx: Option<HashMap<char, HashSet<String>>>,
+    pub(crate) tagidx: Option<HashMap<char, HashSet<String>>>,
 }

 /// Simple tag type for array of array of strings.
@@ -71,7 +59,6 @@ where
 }

 /// Attempt to form a single-char tag name.
-#[must_use]
 pub fn single_char_tagname(tagname: &str) -> Option<char> {
    // We return the tag character if and only if the tagname consists
    // of a single char.
@@ -90,124 +77,29 @@ pub fn single_char_tagname(tagname: &str) -> Option<char> {
    }
 }

-pub enum EventWrapper {
-    WrappedEvent(Event),
-    WrappedAuth(Event),
-}
-
 /// Convert network event to parsed/validated event.
-impl From<EventCmd> for Result<EventWrapper> {
-    fn from(ec: EventCmd) -> Result<EventWrapper> {
+impl From<EventCmd> for Result<Event> {
+    fn from(ec: EventCmd) -> Result<Event> {
        // ensure command is correct
-        if ec.cmd == "EVENT" {
-            ec.event.validate().map(|_| {
-                let mut e = ec.event;
-                e.build_index();
-                e.update_delegation();
-                WrappedEvent(e)
-            })
-        } else if ec.cmd == "AUTH" {
-            // we don't want to validate the event here, because NIP-42 can be disabled
-            // it will be validated later during the authentication process
-            Ok(WrappedAuth(ec.event))
-        } else {
+        if ec.cmd != "EVENT" {
            Err(CommandUnknownError)
+        } else if ec.event.is_valid() {
+            let mut e = ec.event;
+            e.build_index();
+            e.update_delegation();
+            Ok(e)
+        } else {
+            Err(EventInvalid)
        }
    }
 }

 impl Event {
-    #[cfg(test)]
-    #[must_use]
-    pub fn simple_event() -> Event {
-        Event {
-            id: "0".to_owned(),
-            pubkey: "0".to_owned(),
-            delegated_by: None,
-            created_at: 0,
-            kind: 0,
-            tags: vec![],
-            content: "".to_owned(),
-            sig: "0".to_owned(),
-            tagidx: None,
-        }
-    }
-
-    #[must_use]
    pub fn is_kind_metadata(&self) -> bool {
        self.kind == 0
    }

-    /// Should this event be persisted?
-    #[must_use]
-    pub fn is_ephemeral(&self) -> bool {
-        self.kind >= 20000 && self.kind < 30000
-    }
-
-    /// Is this event currently expired?
-    pub fn is_expired(&self) -> bool {
-        if let Some(exp) = self.expiration() {
-            exp <= unix_time()
-        } else {
-            false
-        }
-    }
-
-    /// Determine the time at which this event should expire
-    pub fn expiration(&self) -> Option<u64> {
-        let default = "".to_string();
-        let dvals: Vec<&String> = self
-            .tags
-            .iter()
-            .filter(|x| !x.is_empty())
-            .filter(|x| x.get(0).unwrap() == "expiration")
-            .map(|x| x.get(1).unwrap_or(&default))
-            .take(1)
-            .collect();
-        let val_first = dvals.get(0);
-        val_first.and_then(|t| t.parse::<u64>().ok())
-    }
-
-    /// Should this event be replaced with newer timestamps from same author?
-    #[must_use]
-    pub fn is_replaceable(&self) -> bool {
-        self.kind == 0
-            || self.kind == 3
-            || self.kind == 41
-            || (self.kind >= 10000 && self.kind < 20000)
-    }
-
-    /// Should this event be replaced with newer timestamps from same author, for distinct `d` tag values?
-    #[must_use]
-    pub fn is_param_replaceable(&self) -> bool {
-        self.kind >= 30000 && self.kind < 40000
-    }
-
-    /// Should this event be replaced with newer timestamps from same author, for distinct `d` tag values?
-    #[must_use]
-    pub fn distinct_param(&self) -> Option<String> {
-        if self.is_param_replaceable() {
-            let default = "".to_string();
-            let dvals: Vec<&String> = self
-                .tags
-                .iter()
-                .filter(|x| !x.is_empty())
-                .filter(|x| x.get(0).unwrap() == "d")
-                .map(|x| x.get(1).unwrap_or(&default))
-                .take(1)
-                .collect();
-            let dval_first = dvals.get(0);
-            match dval_first {
-                Some(_) => dval_first.map(|x| x.to_string()),
-                None => Some(default),
-            }
-        } else {
-            None
-        }
-    }
-
    /// Pull a NIP-05 Name out of the event, if one exists
-    #[must_use]
    pub fn get_nip05_addr(&self) -> Option<nip05::Nip05Name> {
        if self.is_kind_metadata() {
            // very quick check if we should attempt to parse this json
@@ -225,7 +117,6 @@ impl Event {
    // is this event delegated (properly)?
    // does the signature match, and are conditions valid?
    // if so, return an alternate author for the event
-    #[must_use]
    pub fn delegated_author(&self) -> Option<String> {
        // is there a delegation tag?
        let delegation_tag: Vec<String> = self
@@ -235,7 +126,7 @@ impl Event {
            .filter(|x| x.get(0).unwrap() == "delegation")
            .take(1)
            .next()?
-            .clone(); // get first tag
+            .to_vec(); // get first tag

        //let delegation_tag = self.tag_values_by_name("delegation");
        // delegation tags should have exactly 3 elements after the name (pubkey, condition, sig)
@@ -264,11 +155,11 @@ impl Event {
    }

    /// Update delegation status
-    pub fn update_delegation(&mut self) {
+    fn update_delegation(&mut self) {
        self.delegated_by = self.delegated_author();
    }
    /// Build an event tag index
-    pub fn build_index(&mut self) {
+    fn build_index(&mut self) {
        // if there are no tags; just leave the index as None
        if self.tags.is_empty() {
            return;
@@ -295,27 +186,23 @@ impl Event {
    }

    /// Create a short event identifier, suitable for logging.
-    #[must_use]
    pub fn get_event_id_prefix(&self) -> String {
        self.id.chars().take(8).collect()
    }
-    #[must_use]
    pub fn get_author_prefix(&self) -> String {
        self.pubkey.chars().take(8).collect()
    }

    /// Retrieve tag initial values across all tags matching the name
-    #[must_use]
    pub fn tag_values_by_name(&self, tag_name: &str) -> Vec<String> {
        self.tags
            .iter()
            .filter(|x| x.len() > 1)
            .filter(|x| x.get(0).unwrap() == tag_name)
-            .map(|x| x.get(1).unwrap().clone())
+            .map(|x| x.get(1).unwrap().to_owned())
            .collect()
    }

-    #[must_use]
    pub fn is_valid_timestamp(&self, reject_future_seconds: Option<usize>) -> bool {
        if let Some(allowable_future) = reject_future_seconds {
            let curr_time = unix_time();
@@ -333,7 +220,7 @@ impl Event {
    }

    /// Check if this event has a valid signature.
-    pub fn validate(&self) -> Result<()> {
+    fn is_valid(&self) -> bool {
        // TODO: return a Result with a reason for invalid events
        // validation is performed by:
        // * parsing JSON string into event fields
@@ -342,43 +229,43 @@ impl Event {
        // * serialize with no spaces/newlines
        let c_opt = self.to_canonical();
        if c_opt.is_none() {
-            debug!("could not canonicalize");
-            return Err(EventCouldNotCanonicalize);
+            debug!("event could not be canonicalized");
+            return false;
        }
        let c = c_opt.unwrap();
        // * compute the sha256sum.
        let digest: sha256::Hash = sha256::Hash::hash(c.as_bytes());
-        let hex_digest = format!("{digest:x}");
+        let hex_digest = format!("{:x}", digest);
        // * ensure the id matches the computed sha256sum.
        if self.id != hex_digest {
            debug!("event id does not match digest");
-            return Err(EventInvalidId);
+            return false;
        }
        // * validate the message digest (sig) using the pubkey & computed sha256 message hash.
-        let sig = schnorr::Signature::from_str(&self.sig).map_err(|_| EventInvalidSignature)?;
+        let sig = schnorr::Signature::from_str(&self.sig).unwrap();
        if let Ok(msg) = secp256k1::Message::from_slice(digest.as_ref()) {
            if let Ok(pubkey) = XOnlyPublicKey::from_str(&self.pubkey) {
-                SECP.verify_schnorr(&sig, &msg, &pubkey)
-                    .map_err(|_| EventInvalidSignature)
+                let verify = SECP.verify_schnorr(&sig, &msg, &pubkey);
+                matches!(verify, Ok(()))
            } else {
                debug!("client sent malformed pubkey");
-                Err(EventMalformedPubkey)
+                false
            }
        } else {
            info!("error converting digest to secp256k1 message");
-            Err(EventInvalidSignature)
+            false
        }
    }

    /// Convert event to canonical representation for signing.
-    pub fn to_canonical(&self) -> Option<String> {
+    fn to_canonical(&self) -> Option<String> {
        // create a JsonValue for each event element
        let mut c: Vec<Value> = vec![];
        // id must be set to 0
        let id = Number::from(0_u64);
        c.push(serde_json::Value::Number(id));
        // public key
-        c.push(Value::String(self.pubkey.clone()));
+        c.push(Value::String(self.pubkey.to_owned()));
        // creation time
        let created_at = Number::from(self.created_at);
        c.push(serde_json::Value::Number(created_at));
@@ -388,7 +275,7 @@ impl Event {
        // tags
        c.push(self.tags_to_canonical());
        // content
-        c.push(Value::String(self.content.clone()));
+        c.push(Value::String(self.content.to_owned()));
        serde_json::to_string(&Value::Array(c)).ok()
    }

@@ -396,11 +283,11 @@ impl Event {
    fn tags_to_canonical(&self) -> Value {
        let mut tags = Vec::<Value>::new();
        // iterate over self tags,
-        for t in &self.tags {
+        for t in self.tags.iter() {
            // each tag is a vec of strings
            let mut a = Vec::<Value>::new();
            for v in t.iter() {
-                a.push(serde_json::Value::String(v.clone()));
+                a.push(serde_json::Value::String(v.to_owned()));
            }
            tags.push(serde_json::Value::Array(a));
        }
@@ -408,7 +295,6 @@ impl Event {
    }

    /// Determine if the given tag and value set intersect with tags in this event.
-    #[must_use]
    pub fn generic_tag_val_intersect(&self, tagname: char, check: &HashSet<String>) -> bool {
        match &self.tagidx {
            // check if this is indexable tagname
@@ -424,37 +310,34 @@ impl Event {
    }
 }

-impl From<nostr::Event> for Event {
-    fn from(nostr_event: nostr::Event) -> Self {
-        Event {
-            id: nostr_event.id.to_hex(),
-            pubkey: nostr_event.pubkey.to_string(),
-            created_at: nostr_event.created_at.as_u64(),
-            kind: nostr_event.kind.as_u64(),
-            tags: nostr_event.tags.iter().map(|x| x.as_vec()).collect(),
-            content: nostr_event.content,
-            sig: nostr_event.sig.to_string(),
-            delegated_by: None,
-            tagidx: None,
-        }
-    }
-}
-
 #[cfg(test)]
 mod tests {
    use super::*;
+    fn simple_event() -> Event {
+        Event {
+            id: "0".to_owned(),
+            pubkey: "0".to_owned(),
+            delegated_by: None,
+            created_at: 0,
+            kind: 0,
+            tags: vec![],
+            content: "".to_owned(),
+            sig: "0".to_owned(),
+            tagidx: None,
+        }
+    }

    #[test]
    fn event_creation() {
        // create an event
-        let event = Event::simple_event();
+        let event = simple_event();
        assert_eq!(event.id, "0");
    }

    #[test]
    fn event_serialize() -> Result<()> {
        // serialize an event to JSON string
-        let event = Event::simple_event();
+        let event = simple_event();
        let j = serde_json::to_string(&event)?;
        assert_eq!(j, "{\"id\":\"0\",\"pubkey\":\"0\",\"created_at\":0,\"kind\":0,\"tags\":[],\"content\":\"\",\"sig\":\"0\"}");
        Ok(())
@@ -462,28 +345,29 @@ mod tests {

    #[test]
    fn empty_event_tag_match() {
-        let event = Event::simple_event();
+        let event = simple_event();
        assert!(!event
            .generic_tag_val_intersect('e', &HashSet::from(["foo".to_owned(), "bar".to_owned()])));
    }

    #[test]
    fn single_event_tag_match() {
-        let mut event = Event::simple_event();
+        let mut event = simple_event();
        event.tags = vec![vec!["e".to_owned(), "foo".to_owned()]];
        event.build_index();
-        assert!(
+        assert_eq!(
            event.generic_tag_val_intersect(
                'e',
                &HashSet::from(["foo".to_owned(), "bar".to_owned()])
-            )
+            ),
+            true
        );
    }

    #[test]
    fn event_tags_serialize() -> Result<()> {
        // serialize an event with tags to JSON string
-        let mut event = Event::simple_event();
+        let mut event = simple_event();
        event.tags = vec![
            vec![
                "e".to_owned(),
@@ -516,7 +400,7 @@ mod tests {
            id: "999".to_owned(),
            pubkey: "012345".to_owned(),
            delegated_by: None,
-            created_at: 501_234,
+            created_at: 501234,
            kind: 1,
            tags: vec![],
            content: "this is a test".to_owned(),
@@ -534,7 +418,7 @@ mod tests {
            id: "999".to_owned(),
            pubkey: "012345".to_owned(),
            delegated_by: None,
-            created_at: 501_234,
+            created_at: 501234,
            kind: 1,
            tags: vec![
                vec!["j".to_owned(), "abc".to_owned()],
@@ -561,7 +445,7 @@ mod tests {
            id: "999".to_owned(),
            pubkey: "012345".to_owned(),
            delegated_by: None,
-            created_at: 501_234,
+            created_at: 501234,
            kind: 1,
            tags: vec![
                vec!["j".to_owned(), "abc".to_owned()],
@@ -588,7 +472,7 @@ mod tests {
            id: "999".to_owned(),
            pubkey: "012345".to_owned(),
            delegated_by: None,
-            created_at: 501_234,
+            created_at: 501234,
            kind: 1,
            tags: vec![
                vec!["#e".to_owned(), "aoeu".to_owned()],
@@ -607,188 +491,4 @@ mod tests {
        let expected = Some(expected_json.to_owned());
        assert_eq!(c, expected);
    }
-
-    #[test]
-    fn ephemeral_event() {
-        let mut event = Event::simple_event();
-        event.kind = 20000;
-        assert!(event.is_ephemeral());
-        event.kind = 29999;
-        assert!(event.is_ephemeral());
-        event.kind = 30000;
-        assert!(!event.is_ephemeral());
-        event.kind = 19999;
-        assert!(!event.is_ephemeral());
-    }
-
-    #[test]
-    fn replaceable_event() {
-        let mut event = Event::simple_event();
-        event.kind = 0;
-        assert!(event.is_replaceable());
-        event.kind = 3;
-        assert!(event.is_replaceable());
-        event.kind = 10000;
-        assert!(event.is_replaceable());
-        event.kind = 19999;
-        assert!(event.is_replaceable());
-        event.kind = 20000;
-        assert!(!event.is_replaceable());
-    }
-
-    #[test]
-    fn param_replaceable_event() {
-        let mut event = Event::simple_event();
-        event.kind = 30000;
-        assert!(event.is_param_replaceable());
-        event.kind = 39999;
-        assert!(event.is_param_replaceable());
-        event.kind = 29999;
-        assert!(!event.is_param_replaceable());
-        event.kind = 40000;
-        assert!(!event.is_param_replaceable());
-    }
-
-    #[test]
-    fn param_replaceable_value_case_1() {
-        // NIP case #1: "tags":[["d",""]]
-        let mut event = Event::simple_event();
-        event.kind = 30000;
-        event.tags = vec![vec!["d".to_owned(), "".to_owned()]];
-        assert_eq!(event.distinct_param(), Some("".to_string()));
-    }
-
-    #[test]
-    fn param_replaceable_value_case_2() {
-        // NIP case #2: "tags":[]: implicit d tag with empty value
-        let mut event = Event::simple_event();
-        event.kind = 30000;
-        assert_eq!(event.distinct_param(), Some("".to_string()));
-    }
-
-    #[test]
-    fn param_replaceable_value_case_3() {
-        // NIP case #3: "tags":[["d"]]: implicit empty value ""
-        let mut event = Event::simple_event();
-        event.kind = 30000;
-        event.tags = vec![vec!["d".to_owned()]];
-        assert_eq!(event.distinct_param(), Some("".to_string()));
-    }
-
-    #[test]
-    fn param_replaceable_value_case_4() {
-        // NIP case #4: "tags":[["d",""],["d","not empty"]]: only first d tag is considered
-        let mut event = Event::simple_event();
-        event.kind = 30000;
-        event.tags = vec![
-            vec!["d".to_owned(), "".to_string()],
-            vec!["d".to_owned(), "not empty".to_string()],
-        ];
-        assert_eq!(event.distinct_param(), Some("".to_string()));
-    }
-
-    #[test]
-    fn param_replaceable_value_case_4b() {
-        // Variation of #4 with
-        // NIP case #4: "tags":[["d","not empty"],["d",""]]: only first d tag is considered
-        let mut event = Event::simple_event();
-        event.kind = 30000;
-        event.tags = vec![
-            vec!["d".to_owned(), "not empty".to_string()],
-            vec!["d".to_owned(), "".to_string()],
-        ];
-        assert_eq!(event.distinct_param(), Some("not empty".to_string()));
-    }
-
-    #[test]
-    fn param_replaceable_value_case_5() {
-        // NIP case #5: "tags":[["d"],["d","some value"]]: only first d tag is considered
-        let mut event = Event::simple_event();
-        event.kind = 30000;
-        event.tags = vec![
-            vec!["d".to_owned()],
-            vec!["d".to_owned(), "second value".to_string()],
-            vec!["d".to_owned(), "third value".to_string()],
-        ];
-        assert_eq!(event.distinct_param(), Some("".to_string()));
-    }
-
-    #[test]
-    fn param_replaceable_value_case_6() {
-        // NIP case #6: "tags":[["e"]]: same as no tags
-        let mut event = Event::simple_event();
-        event.kind = 30000;
-        event.tags = vec![vec!["e".to_owned()]];
-        assert_eq!(event.distinct_param(), Some("".to_string()));
-    }
-
-    #[test]
-    fn expiring_event_none() {
-        // regular events do not expire
-        let mut event = Event::simple_event();
-        event.kind = 7;
-        event.tags = vec![vec!["test".to_string(), "foo".to_string()]];
-        assert_eq!(event.expiration(), None);
-    }
-
-    #[test]
-    fn expiring_event_empty() {
-        // regular events do not expire
-        let mut event = Event::simple_event();
-        event.kind = 7;
-        event.tags = vec![vec!["expiration".to_string()]];
-        assert_eq!(event.expiration(), None);
-    }
-
-    #[test]
-    fn expiring_event_future() {
-        // a normal expiring event
-        let exp: u64 = 1676264138;
-        let mut event = Event::simple_event();
-        event.kind = 1;
-        event.tags = vec![vec!["expiration".to_string(), exp.to_string()]];
-        assert_eq!(event.expiration(), Some(exp));
-    }
-
-    #[test]
-    fn expiring_event_negative() {
-        // expiration set to a negative value (invalid)
-        let exp: i64 = -90;
-        let mut event = Event::simple_event();
-        event.kind = 1;
-        event.tags = vec![vec!["expiration".to_string(), exp.to_string()]];
-        assert_eq!(event.expiration(), None);
-    }
-
-    #[test]
-    fn expiring_event_zero() {
-        // a normal expiring event set to zero
-        let exp: i64 = 0;
-        let mut event = Event::simple_event();
-        event.kind = 1;
-        event.tags = vec![vec!["expiration".to_string(), exp.to_string()]];
-        assert_eq!(event.expiration(), Some(0));
-    }
-
-    #[test]
-    fn expiring_event_fraction() {
-        // expiration is fractional (invalid)
-        let exp: f64 = 23.334;
-        let mut event = Event::simple_event();
-        event.kind = 1;
-        event.tags = vec![vec!["expiration".to_string(), exp.to_string()]];
-        assert_eq!(event.expiration(), None);
-    }
-
-    #[test]
-    fn expiring_event_multiple() {
-        // multiple values, we just take the first
-        let mut event = Event::simple_event();
-        event.kind = 1;
-        event.tags = vec![
-            vec!["expiration".to_string(), (10).to_string()],
-            vec!["expiration".to_string(), (20).to_string()],
-        ];
-        assert_eq!(event.expiration(), Some(10));
-    }
 }
@@ -3,7 +3,7 @@ use crate::utils::is_hex;
 use hex;

 /// Types of hexadecimal queries.
-#[derive(PartialEq, Eq, PartialOrd, Ord, Debug, Clone)]
+#[derive(PartialEq, Eq, Debug, Clone)]
 pub enum HexSearch {
    // when no range is needed, exact 32-byte
    Exact(Vec<u8>),
@@ -19,16 +19,16 @@ fn is_all_fs(s: &str) -> bool {
 }

 /// Find the next hex sequence greater than the argument.
-#[must_use]
 pub fn hex_range(s: &str) -> Option<HexSearch> {
-    let mut hash_base = s.to_owned();
-    if !is_hex(&hash_base) || hash_base.len() > 64 {
+    // handle special cases
+    if !is_hex(s) || s.len() > 64 {
        return None;
    }
-    if hash_base.len() == 64 {
-        return Some(HexSearch::Exact(hex::decode(&hash_base).ok()?));
+    if s.len() == 64 {
+        return Some(HexSearch::Exact(hex::decode(s).ok()?));
    }
    // if s is odd, add a zero
+    let mut hash_base = s.to_owned();
    let mut odd = hash_base.len() % 2 != 0;
    if odd {
        // extend the string to make it even
@@ -57,9 +57,8 @@ pub fn hex_range(s: &str) -> Option<HexSearch> {
        } else if odd {
            // check if first char in this byte is NOT 'f'
            if b < 240 {
-                // bump up the first character in this byte
-                upper[byte_len] = b + 16;
-                // increment done, stop iterating through the vec
+                upper[byte_len] = b + 16; // bump up the first character in this byte
+                                          // increment done, stop iterating through the vec
                break;
            }
            // if it is 'f', reset the byte to 0 and do a carry
@@ -1,35 +1,9 @@
 //! Relay metadata using NIP-11
 /// Relay Info
-use crate::config::Settings;
+use crate::config;
 use serde::{Deserialize, Serialize};

 pub const CARGO_PKG_VERSION: Option<&'static str> = option_env!("CARGO_PKG_VERSION");
-pub const UNIT: &str = "sats";
-
-/// Limitations of the relay as specified in NIP-111
-/// (This nip isn't finalized so may change)
-#[derive(Debug, Serialize, Deserialize)]
-#[allow(unused)]
-pub struct Limitation {
-    #[serde(skip_serializing_if = "Option::is_none")]
-    payment_required: Option<bool>,
-}
-
-#[derive(Serialize, Deserialize, Debug)]
-#[allow(unused)]
-pub struct Fees {
-    #[serde(skip_serializing_if = "Option::is_none")]
-    admission: Option<Vec<Fee>>,
-    #[serde(skip_serializing_if = "Option::is_none")]
-    publication: Option<Vec<Fee>>,
-}
-
-#[derive(Serialize, Deserialize, Debug)]
-#[allow(unused)]
-pub struct Fee {
-    amount: u64,
-    unit: String,
-}

 #[derive(Debug, Serialize, Deserialize)]
 #[allow(unused)]
@@ -45,88 +19,25 @@ pub struct RelayInfo {
    #[serde(skip_serializing_if = "Option::is_none")]
    pub contact: Option<String>,
    #[serde(skip_serializing_if = "Option::is_none")]
-    pub icon: Option<String>,        
-    #[serde(skip_serializing_if = "Option::is_none")]
    pub supported_nips: Option<Vec<i64>>,
    #[serde(skip_serializing_if = "Option::is_none")]
    pub software: Option<String>,
    #[serde(skip_serializing_if = "Option::is_none")]
    pub version: Option<String>,
-    #[serde(skip_serializing_if = "Option::is_none")]
-    pub limitation: Option<Limitation>,
-    #[serde(skip_serializing_if = "Option::is_none")]
-    pub payment_url: Option<String>,
-    #[serde(skip_serializing_if = "Option::is_none")]
-    pub fees: Option<Fees>,
 }

 /// Convert an Info configuration into public Relay Info
-impl From<Settings> for RelayInfo {
-    fn from(c: Settings) -> Self {
-        let mut supported_nips = vec![1, 2, 9, 11, 12, 15, 16, 20, 22, 33, 40];
-
-        if c.authorization.nip42_auth {
-            supported_nips.push(42);
-            supported_nips.sort();
-        }
-
-        let i = c.info;
-        let p = c.pay_to_relay;
-
-        let limitations = Limitation {
-            payment_required: Some(p.enabled),
-        };
-
-        let (payment_url, fees) = if p.enabled {
-            let admission_fee = if p.admission_cost > 0 {
-                Some(vec![Fee {
-                    amount: p.admission_cost,
-                    unit: UNIT.to_string(),
-                }])
-            } else {
-                None
-            };
-
-            let post_fee = if p.cost_per_event > 0 {
-                Some(vec![Fee {
-                    amount: p.cost_per_event,
-                    unit: UNIT.to_string(),
-                }])
-            } else {
-                None
-            };
-
-            let fees = Fees {
-                admission: admission_fee,
-                publication: post_fee,
-            };
-
-            let payment_url = if p.enabled && i.relay_url.is_some() {
-                Some(format!(
-                    "{}join",
-                    i.relay_url.clone().unwrap().replace("ws", "http")
-                ))
-            } else {
-                None
-            };
-            (payment_url, Some(fees))
-        } else {
-            (None, None)
-        };
-
+impl From<config::Info> for RelayInfo {
+    fn from(i: config::Info) -> Self {
        RelayInfo {
            id: i.relay_url,
            name: i.name,
            description: i.description,
            pubkey: i.pubkey,
            contact: i.contact,
-            supported_nips: Some(supported_nips),
+            supported_nips: Some(vec![1, 2, 9, 11, 12, 15, 16, 22, 26]),
            software: Some("https://git.sr.ht/~gheartsfield/nostr-rs-relay".to_owned()),
-            version: CARGO_PKG_VERSION.map(std::borrow::ToOwned::to_owned),
-            limitation: Some(limitations),
-            payment_url,
-            fees,
-            icon: i.relay_icon,
+            version: CARGO_PKG_VERSION.map(|x| x.to_owned()),
        }
    }
 }
@@ -1,4 +1,3 @@
-pub mod cli;
 pub mod close;
 pub mod config;
 pub mod conn;
@@ -8,12 +7,10 @@ pub mod error;
 pub mod event;
 pub mod hexrange;
 pub mod info;
-pub mod nauthz;
 pub mod nip05;
 pub mod notice;
-pub mod repo;
+pub mod schema;
 pub mod subscription;
 pub mod utils;
-// Public API for creating relays programmatically
-pub mod payment;
+// Public API for creating relays programatically
 pub mod server;
@@ -1,108 +1,50 @@
 //! Server process
-use clap::Parser;
-use console_subscriber::ConsoleLayer;
-use nostr_rs_relay::cli::CLIArgs;
+
 use nostr_rs_relay::config;
 use nostr_rs_relay::server::start_server;
-use std::fs;
-use std::path::Path;
-use std::process;
+use std::env;
 use std::sync::mpsc as syncmpsc;
 use std::sync::mpsc::{Receiver as MpscReceiver, Sender as MpscSender};
 use std::thread;
-#[cfg(not(target_env = "msvc"))]
-use tikv_jemallocator::Jemalloc;
 use tracing::info;
-use tracing_appender::non_blocking::WorkerGuard;
-use tracing_subscriber::EnvFilter;

-#[cfg(not(target_env = "msvc"))]
-#[global_allocator]
-static GLOBAL: Jemalloc = Jemalloc;
+use console_subscriber::ConsoleLayer;
+
+/// Return a requested DB name from command line arguments.
+fn db_from_args(args: &[String]) -> Option<String> {
+    if args.len() == 3 && args.get(1) == Some(&"--db".to_owned()) {
+        return args.get(2).map(std::clone::Clone::clone);
+    }
+    None
+}

 /// Start running a Nostr relay server.
 fn main() {
-    let args = CLIArgs::parse();
-
-    // get config file name from args
-    let config_file_arg = args.config;
-
-    // Ensure the config file is readable if it was explicitly set
-    if let Some(config_path) = config_file_arg.as_ref() {
-        let path = Path::new(&config_path);
-        if !path.exists() {
-            eprintln!("Config file not found: {}", &config_path);
-            process::exit(1);
-        }
-        if !path.is_file() {
-            eprintln!("Invalid config file path: {}", &config_path);
-            process::exit(1);
-        }
-        if let Err(err) = fs::metadata(path) {
-            eprintln!("Error while accessing file metadata: {}", err);
-            process::exit(1);
-        }
-        if let Err(err) = fs::File::open(path) {
-            eprintln!("Config file is not readable: {}", err);
-            process::exit(1);
-        }
-    }
-
-    let mut _log_guard: Option<WorkerGuard> = None;
-
-    // configure settings from the config file (defaults to config.toml)
-    // replace default settings with those read from the config file
-    let mut settings = config::Settings::new(&config_file_arg).unwrap_or_else(|e| {
-        eprintln!("Error reading config file ({:?})", e);
-        process::exit(1);
-    });
-
    // setup tracing
+    let _trace_sub = tracing_subscriber::fmt::try_init();
+    info!("Starting up from main");
+    // get database directory from args
+    let args: Vec<String> = env::args().collect();
+    let db_dir: Option<String> = db_from_args(&args);
+    // configure settings from config.toml
+    // replace default settings with those read from config.toml
+    let mut settings = config::Settings::new();
+
    if settings.diagnostics.tracing {
        // enable tracing with tokio-console
        ConsoleLayer::builder().with_default_env().init();
-    } else {
-        // standard logging
-        if let Some(path) = &settings.logging.folder_path {
-            // write logs to a folder
-            let prefix = match &settings.logging.file_prefix {
-                Some(p) => p.as_str(),
-                None => "relay",
-            };
-            let file_appender = tracing_appender::rolling::daily(path, prefix);
-            let (non_blocking, guard) = tracing_appender::non_blocking(file_appender);
-            let filter = EnvFilter::from_default_env();
-            // assign to a variable that is not dropped till the program ends
-            _log_guard = Some(guard);
-
-            tracing_subscriber::fmt()
-                .with_env_filter(filter)
-                .with_writer(non_blocking)
-                .try_init()
-                .unwrap();
-        } else {
-            // write to stdout
-            tracing_subscriber::fmt::try_init().unwrap();
-        }
    }
-    info!("Starting up from main");
-
-    // get database directory from args
-    let db_dir_arg = args.db;
-
-    // update with database location from args, if provided
-    if let Some(db_dir) = db_dir_arg {
-        settings.database.data_directory = db_dir;
+    // update with database location
+    if let Some(db) = db_dir {
+        settings.database.data_directory = db;
    }
-    // we should have a 'control plane' channel to monitor and bump
-    // the server.  this will let us do stuff like clear the database,
-    // shutdown, etc.; for now all this does is initiate shutdown if
-    // `()` is sent.  This will change in the future, this is just a
-    // stopgap to shutdown the relay when it is used as a library.
+
    let (_, ctrl_rx): (MpscSender<()>, MpscReceiver<()>) = syncmpsc::channel();
    // run this in a new thread
-    let handle = thread::spawn(move || {
-        let _svr = start_server(&settings, ctrl_rx);
+    let handle = thread::spawn(|| {
+        // we should have a 'control plane' channel to monitor and bump the server.
+        // this will let us do stuff like clear the database, shutdown, etc.
+        let _svr = start_server(settings, ctrl_rx);
    });
    // block on nostr thread to finish.
    handle.join().unwrap();
@@ -1,111 +0,0 @@
-use crate::error::{Error, Result};
-use crate::{event::Event, nip05::Nip05Name};
-use nauthz_grpc::authorization_client::AuthorizationClient;
-use nauthz_grpc::event::TagEntry;
-use nauthz_grpc::{Decision, Event as GrpcEvent, EventReply, EventRequest};
-use tracing::{info, warn};
-
-pub mod nauthz_grpc {
-    tonic::include_proto!("nauthz");
-}
-
-// A decision for the DB to act upon
-pub trait AuthzDecision: Send + Sync {
-    fn permitted(&self) -> bool;
-    fn message(&self) -> Option<String>;
-}
-
-impl AuthzDecision for EventReply {
-    fn permitted(&self) -> bool {
-        self.decision == Decision::Permit as i32
-    }
-    fn message(&self) -> Option<String> {
-        self.message.clone()
-    }
-}
-
-// A connection to an event admission GRPC server
-pub struct EventAuthzService {
-    server_addr: String,
-    conn: Option<AuthorizationClient<tonic::transport::Channel>>,
-}
-
-// conversion of Nip05Names into GRPC type
-impl std::convert::From<Nip05Name> for nauthz_grpc::event_request::Nip05Name {
-    fn from(value: Nip05Name) -> Self {
-        nauthz_grpc::event_request::Nip05Name {
-            local: value.local.clone(),
-            domain: value.domain,
-        }
-    }
-}
-
-// conversion of event tags into gprc struct
-fn tags_to_protobuf(tags: &[Vec<String>]) -> Vec<TagEntry> {
-    tags.iter()
-        .map(|x| TagEntry { values: x.clone() })
-        .collect()
-}
-
-impl EventAuthzService {
-    pub async fn connect(server_addr: &str) -> EventAuthzService {
-        let mut eas = EventAuthzService {
-            server_addr: server_addr.to_string(),
-            conn: None,
-        };
-        eas.ready_connection().await;
-        eas
-    }
-
-    pub async fn ready_connection(&mut self) {
-        if self.conn.is_none() {
-            let client = AuthorizationClient::connect(self.server_addr.to_string()).await;
-            if let Err(ref msg) = client {
-                warn!("could not connect to nostr authz GRPC server: {:?}", msg);
-            } else {
-                info!("connected to nostr authorization GRPC server");
-            }
-            self.conn = client.ok();
-        }
-    }
-
-    pub async fn admit_event(
-        &mut self,
-        event: &Event,
-        ip: &str,
-        origin: Option<String>,
-        user_agent: Option<String>,
-        nip05: Option<Nip05Name>,
-        auth_pubkey: Option<Vec<u8>>,
-    ) -> Result<Box<dyn AuthzDecision>> {
-        self.ready_connection().await;
-        let id_blob = hex::decode(&event.id)?;
-        let pubkey_blob = hex::decode(&event.pubkey)?;
-        let sig_blob = hex::decode(&event.sig)?;
-        if let Some(ref mut c) = self.conn {
-            let gevent = GrpcEvent {
-                id: id_blob,
-                pubkey: pubkey_blob,
-                sig: sig_blob,
-                created_at: event.created_at,
-                kind: event.kind,
-                content: event.content.clone(),
-                tags: tags_to_protobuf(&event.tags),
-            };
-            let svr_res = c
-                .event_admit(EventRequest {
-                    event: Some(gevent),
-                    ip_addr: Some(ip.to_string()),
-                    origin,
-                    user_agent,
-                    auth_pubkey,
-                    nip05: nip05.map(nauthz_grpc::event_request::Nip05Name::from),
-                })
-                .await?;
-            let reply = svr_res.into_inner();
-            Ok(Box::new(reply))
-        } else {
-            Err(Error::AuthzError)
-        }
-    }
-}
@@ -5,14 +5,16 @@
 //! consumes a stream of metadata events, and keeps a database table
 //! updated with the current NIP-05 verification status.
 use crate::config::VerifiedUsers;
+use crate::db;
 use crate::error::{Error, Result};
 use crate::event::Event;
-use crate::repo::NostrRepo;
+use crate::utils::unix_time;
 use hyper::body::HttpBody;
 use hyper::client::connect::HttpConnector;
 use hyper::Client;
-use hyper_rustls::HttpsConnector;
-use std::sync::Arc;
+use hyper_tls::HttpsConnector;
+use rand::Rng;
+use rusqlite::params;
 use std::time::Duration;
 use std::time::Instant;
 use std::time::SystemTime;
@@ -21,12 +23,14 @@ use tracing::{debug, info, warn};

 /// NIP-05 verifier state
 pub struct Verifier {
-    /// Repository for saving/retrieving events and records
-    repo: Arc<dyn NostrRepo>,
    /// Metadata events for us to inspect
    metadata_rx: tokio::sync::broadcast::Receiver<Event>,
    /// Newly validated events get written and then broadcast on this channel to subscribers
    event_tx: tokio::sync::broadcast::Sender<Event>,
+    /// SQLite read query pool
+    read_pool: db::SqlitePool,
+    /// SQLite write query pool
+    write_pool: db::SqlitePool,
    /// Settings
    settings: crate::config::Settings,
    /// HTTP client
@@ -42,13 +46,12 @@ pub struct Verifier {
 /// A NIP-05 identifier is a local part and domain.
 #[derive(PartialEq, Eq, Debug, Clone)]
 pub struct Nip05Name {
-    pub local: String,
-    pub domain: String,
+    local: String,
+    domain: String,
 }

 impl Nip05Name {
    /// Does this name represent the entire domain?
-    #[must_use]
    pub fn is_domain_only(&self) -> bool {
        self.local == "_"
    }
@@ -70,7 +73,9 @@ impl std::convert::TryFrom<&str> for Nip05Name {
    fn try_from(inet: &str) -> Result<Self, Self::Error> {
        // break full name at the @ boundary.
        let components: Vec<&str> = inet.split('@').collect();
-        if components.len() == 2 {
+        if components.len() != 2 {
+            Err(Error::CustomError("too many/few components".to_owned()))
+        } else {
            // check if local name is valid
            let local = components[0];
            let domain = components[1];
@@ -96,8 +101,6 @@ impl std::convert::TryFrom<&str> for Nip05Name {
                    "invalid character in local part".to_owned(),
                ))
            }
-        } else {
-            Err(Error::CustomError("too many/few components".to_owned()))
        }
    }
 }
@@ -108,37 +111,57 @@ impl std::fmt::Display for Nip05Name {
    }
 }

+// Current time, with a slight foward jitter in seconds
+fn now_jitter(sec: u64) -> u64 {
+    // random time between now, and 10min in future.
+    let mut rng = rand::thread_rng();
+    let jitter_amount = rng.gen_range(0..sec);
+    let now = unix_time();
+    now.saturating_add(jitter_amount)
+}
+
 /// Check if the specified username and address are present and match in this response body
-fn body_contains_user(username: &str, address: &str, bytes: &hyper::body::Bytes) -> Result<bool> {
+fn body_contains_user(username: &str, address: &str, bytes: hyper::body::Bytes) -> Result<bool> {
    // convert the body into json
-    let body: serde_json::Value = serde_json::from_slice(bytes)?;
+    let body: serde_json::Value = serde_json::from_slice(&bytes)?;
    // ensure we have a names object.
    let names_map = body
        .as_object()
        .and_then(|x| x.get("names"))
-        .and_then(serde_json::Value::as_object)
+        .and_then(|x| x.as_object())
        .ok_or_else(|| Error::CustomError("not a map".to_owned()))?;
    // get the pubkey for the requested user
-    let check_name = names_map.get(username).and_then(serde_json::Value::as_str);
+    let check_name = names_map.get(username).and_then(|x| x.as_str());
    // ensure the address is a match
-    Ok(check_name.map_or(false, |x| x == address))
+    Ok(check_name.map(|x| x == address).unwrap_or(false))
 }

 impl Verifier {
    pub fn new(
-        repo: Arc<dyn NostrRepo>,
        metadata_rx: tokio::sync::broadcast::Receiver<Event>,
        event_tx: tokio::sync::broadcast::Sender<Event>,
        settings: crate::config::Settings,
    ) -> Result<Self> {
        info!("creating NIP-05 verifier");
+        // build a database connection for reading and writing.
+        let write_pool = db::build_pool(
+            "nip05 writer",
+            &settings,
+            rusqlite::OpenFlags::SQLITE_OPEN_READ_WRITE,
+            1,    // min conns
+            4,    // max conns
+            true, // wait for DB
+        );
+        let read_pool = db::build_pool(
+            "nip05 reader",
+            &settings,
+            rusqlite::OpenFlags::SQLITE_OPEN_READ_ONLY,
+            1,    // min conns
+            8,    // max conns
+            true, // wait for DB
+        );
        // setup hyper client
-        let https = hyper_rustls::HttpsConnectorBuilder::new()
-            .with_native_roots()
-            .https_or_http()
-            .enable_http1()
-            .build();
-
+        let https = HttpsConnector::new();
        let client = Client::builder().build::<_, hyper::Body>(https);

        // After all accounts have been re-verified, don't check again
@@ -152,9 +175,10 @@ impl Verifier {
        // duration.
        let reverify_interval = tokio::time::interval(http_wait_duration);
        Ok(Verifier {
-            repo,
            metadata_rx,
            event_tx,
+            read_pool,
+            write_pool,
            settings,
            client,
            wait_after_finish,
@@ -222,40 +246,44 @@ impl Verifier {

        let response_fut = self.client.request(req);

-        if let Ok(response_res) = tokio::time::timeout(Duration::from_secs(5), response_fut).await {
-            // limit size of verification document to 1MB.
-            const MAX_ALLOWED_RESPONSE_SIZE: u64 = 1024 * 1024;
-            let response = response_res?;
-            // determine content length from response
-            let response_content_length = match response.body().size_hint().upper() {
-                Some(v) => v,
-                None => MAX_ALLOWED_RESPONSE_SIZE + 1, // reject missing content length
-            };
-            // TODO: test how hyper handles the client providing an inaccurate content-length.
-            if response_content_length <= MAX_ALLOWED_RESPONSE_SIZE {
-                let (parts, body) = response.into_parts();
-                // TODO: consider redirects
-                if parts.status == http::StatusCode::OK {
-                    // parse body, determine if the username / key / address is present
-                    let body_bytes = hyper::body::to_bytes(body).await?;
-                    let body_matches = body_contains_user(&nip.local, pubkey, &body_bytes)?;
-                    if body_matches {
-                        return Ok(UserWebVerificationStatus::Verified);
+        // HTTP request with timeout
+        match tokio::time::timeout(Duration::from_secs(5), response_fut).await {
+            Ok(response_res) => {
+                // limit size of verification document to 1MB.
+                const MAX_ALLOWED_RESPONSE_SIZE: u64 = 1024 * 1024;
+                let response = response_res?;
+                // determine content length from response
+                let response_content_length = match response.body().size_hint().upper() {
+                    Some(v) => v,
+                    None => MAX_ALLOWED_RESPONSE_SIZE + 1, // reject missing content length
+                };
+                // TODO: test how hyper handles the client providing an inaccurate content-length.
+                if response_content_length <= MAX_ALLOWED_RESPONSE_SIZE {
+                    let (parts, body) = response.into_parts();
+                    // TODO: consider redirects
+                    if parts.status == http::StatusCode::OK {
+                        // parse body, determine if the username / key / address is present
+                        let body_bytes = hyper::body::to_bytes(body).await?;
+                        let body_matches = body_contains_user(&nip.local, pubkey, body_bytes)?;
+                        if body_matches {
+                            return Ok(UserWebVerificationStatus::Verified);
+                        }
+                        // successful response, parsed as a nip-05
+                        // document, but this name/pubkey was not
+                        // present.
+                        return Ok(UserWebVerificationStatus::Unverified);
                    }
-                    // successful response, parsed as a nip-05
-                    // document, but this name/pubkey was not
-                    // present.
-                    return Ok(UserWebVerificationStatus::Unverified);
+                } else {
+                    info!(
+                        "content length missing or exceeded limits for account: {:?}",
+                        nip.to_string()
+                    );
                }
-            } else {
-                info!(
-                    "content length missing or exceeded limits for account: {:?}",
-                    nip.to_string()
-                );
            }
-        } else {
-            info!("timeout verifying account {:?}", nip);
-            return Ok(UserWebVerificationStatus::Unknown);
+            Err(_) => {
+                info!("timeout verifying account {:?}", nip);
+                return Ok(UserWebVerificationStatus::Unknown);
+            }
        }
        Ok(UserWebVerificationStatus::Unknown)
    }
@@ -266,15 +294,8 @@ impl Verifier {
        // run a loop, restarting on failure
        loop {
            let res = self.run_internal().await;
-            match res {
-                Err(Error::ChannelClosed) => {
-                    // channel was closed, we are shutting down
-                    return;
-                }
-                Err(e) => {
-                    info!("error in verifier: {:?}", e);
-                }
-                _ => {}
+            if let Err(e) = res {
+                info!("error in verifier: {:?}", e);
            }
        }
    }
@@ -288,7 +309,7 @@ impl Verifier {
                        if let Some(naddr) = e.get_nip05_addr() {
                            info!("got metadata event for ({:?},{:?})", naddr.to_string() ,e.get_author_prefix());
                            // Process a new author, checking if they are verified:
-                            let check_verified = self.repo.get_latest_user_verification(&e.pubkey).await;
+                            let check_verified = get_latest_user_verification(self.read_pool.get().expect("could not get connection"), &e.pubkey).await;
                            // ensure the event we got is more recent than the one we have, otherwise we can ignore it.
                            if let Ok(last_check) = check_verified {
                                if e.created_at <= last_check.event_created {
@@ -321,7 +342,6 @@ impl Verifier {
                    }
                    Err(tokio::sync::broadcast::error::RecvError::Closed) => {
                        info!("metadata broadcast channel closed");
-                        return Err(Error::ChannelClosed);
                    }
                }
            },
@@ -350,7 +370,7 @@ impl Verifier {
            .duration_since(SystemTime::UNIX_EPOCH)
            .map(|x| x.as_secs())
            .unwrap_or(0);
-        let vr = self.repo.get_oldest_user_verification(earliest_epoch).await;
+        let vr = get_oldest_user_verification(self.read_pool.get()?, earliest_epoch).await;
        match vr {
            Ok(ref v) => {
                let new_status = self.get_web_verification(&v.name, &v.address).await;
@@ -358,8 +378,8 @@ impl Verifier {
                    UserWebVerificationStatus::Verified => {
                        // freshly verified account, update the
                        // timestamp.
-                        self.repo.update_verification_timestamp(v.rowid).await?;
-                        info!("verification updated for {}", v.to_string());
+                        self.update_verification_record(self.write_pool.get()?, v)
+                            .await?;
                    }
                    UserWebVerificationStatus::DomainNotAllowed
                    | UserWebVerificationStatus::Unknown => {
@@ -374,25 +394,23 @@ impl Verifier {
                                "giving up on verifying {:?} after {} failures",
                                v.name, v.failure_count
                            );
-                            self.repo.delete_verification(v.rowid).await?;
+                            self.delete_verification_record(self.write_pool.get()?, v)
+                                .await?;
                        } else {
                            // record normal failure, incrementing failure count
-                            info!("verification failed for {}", v.to_string());
-                            self.repo.fail_verification(v.rowid).await?;
+                            self.fail_verification_record(self.write_pool.get()?, v)
+                                .await?;
                        }
                    }
                    UserWebVerificationStatus::Unverified => {
                        // domain has removed the verification, drop
                        // the record on our side.
-                        info!("verification rescinded for {}", v.to_string());
-                        self.repo.delete_verification(v.rowid).await?;
+                        self.delete_verification_record(self.write_pool.get()?, v)
+                            .await?;
                    }
                }
            }
-            Err(
-                Error::SqlError(rusqlite::Error::QueryReturnedNoRows)
-                | Error::SqlxError(sqlx::Error::RowNotFound),
-            ) => {
+            Err(Error::SqlError(rusqlite::Error::QueryReturnedNoRows)) => {
                // No users need verification.  Reset the interval to
                // the next verification attempt.
                let start = tokio::time::Instant::now() + self.wait_after_finish;
@@ -408,6 +426,80 @@ impl Verifier {
        Ok(())
    }

+    /// Reset the verification timestamp on a VerificationRecord
+    pub async fn update_verification_record(
+        &mut self,
+        mut conn: db::PooledConnection,
+        vr: &VerificationRecord,
+    ) -> Result<()> {
+        let vr_id = vr.rowid;
+        let vr_str = vr.to_string();
+        tokio::task::spawn_blocking(move || {
+            // add some jitter to the verification to prevent everything from stacking up together.
+            let verif_time = now_jitter(600);
+            let tx = conn.transaction()?;
+            {
+                // update verification time and reset any failure count
+                let query =
+                    "UPDATE user_verification SET verified_at=?, failure_count=0 WHERE id=?";
+                let mut stmt = tx.prepare(query)?;
+                stmt.execute(params![verif_time, vr_id])?;
+            }
+            tx.commit()?;
+            info!("verification updated for {}", vr_str);
+            let ok: Result<()> = Ok(());
+            ok
+        })
+        .await?
+    }
+    /// Reset the failure timestamp on a VerificationRecord
+    pub async fn fail_verification_record(
+        &mut self,
+        mut conn: db::PooledConnection,
+        vr: &VerificationRecord,
+    ) -> Result<()> {
+        let vr_id = vr.rowid;
+        let vr_str = vr.to_string();
+        let fail_count = vr.failure_count.saturating_add(1);
+        tokio::task::spawn_blocking(move || {
+            // add some jitter to the verification to prevent everything from stacking up together.
+            let fail_time = now_jitter(600);
+            let tx = conn.transaction()?;
+            {
+                let query = "UPDATE user_verification SET failed_at=?, failure_count=? WHERE id=?";
+                let mut stmt = tx.prepare(query)?;
+                stmt.execute(params![fail_time, fail_count, vr_id])?;
+            }
+            tx.commit()?;
+            info!("verification failed for {}", vr_str);
+            let ok: Result<()> = Ok(());
+            ok
+        })
+        .await?
+    }
+    /// Delete a VerificationRecord that is no longer valid
+    pub async fn delete_verification_record(
+        &mut self,
+        mut conn: db::PooledConnection,
+        vr: &VerificationRecord,
+    ) -> Result<()> {
+        let vr_id = vr.rowid;
+        let vr_str = vr.to_string();
+        tokio::task::spawn_blocking(move || {
+            let tx = conn.transaction()?;
+            {
+                let query = "DELETE FROM user_verification WHERE id=?;";
+                let mut stmt = tx.prepare(query)?;
+                stmt.execute(params![vr_id])?;
+            }
+            tx.commit()?;
+            info!("verification rescinded for {}", vr_str);
+            let ok: Result<()> = Ok(());
+            ok
+        })
+        .await?
+    }
+
    /// Persist an event, create a verification record, and broadcast.
    // TODO: have more event-writing logic handled in the db module.
    // Right now, these events avoid the rate limit.  That is
@@ -421,11 +513,11 @@ impl Verifier {
        // disabled/passive, the event has already been persisted.
        let should_write_event = self.settings.verified_users.is_enabled();
        if should_write_event {
-            match self.repo.write_event(event).await {
+            match db::write_event(&mut self.write_pool.get()?, event) {
                Ok(updated) => {
                    if updated != 0 {
                        info!(
-                            "persisted event (new verified pubkey): {:?} in {:?}",
+                            "persisted event: {:?} in {:?}",
                            event.get_event_id_prefix(),
                            start.elapsed()
                        );
@@ -441,9 +533,7 @@ impl Verifier {
            }
        }
        // write the verification record
-        self.repo
-            .create_verification_record(&event.id, name)
-            .await?;
+        save_verification_record(self.write_pool.get()?, event, name).await?;
        Ok(())
    }
 }
@@ -473,7 +563,6 @@ pub struct VerificationRecord {

 /// Check with settings to determine if a given domain is allowed to
 /// publish.
-#[must_use]
 pub fn is_domain_allowed(
    domain: &str,
    whitelist: &Option<Vec<String>>,
@@ -494,7 +583,6 @@ pub fn is_domain_allowed(
 impl VerificationRecord {
    /// Check if the record is recent enough to be considered valid,
    /// and the domain is allowed.
-    #[must_use]
    pub fn is_valid(&self, verified_users_settings: &VerifiedUsers) -> bool {
        //let settings = SETTINGS.read().unwrap();
        // how long a verification record is good for
@@ -542,6 +630,130 @@ impl std::fmt::Display for VerificationRecord {
    }
 }

+/// Create a new verification record based on an event
+pub async fn save_verification_record(
+    mut conn: db::PooledConnection,
+    event: &Event,
+    name: &str,
+) -> Result<()> {
+    let e = hex::decode(&event.id).ok();
+    let n = name.to_owned();
+    let a_prefix = event.get_author_prefix();
+    tokio::task::spawn_blocking(move || {
+        let tx = conn.transaction()?;
+        {
+            // if we create a /new/ one, we should get rid of any old ones.  or group the new ones by name and only consider the latest.
+            let query = "INSERT INTO user_verification (metadata_event, name, verified_at) VALUES ((SELECT id from event WHERE event_hash=?), ?, strftime('%s','now'));";
+            let mut stmt = tx.prepare(query)?;
+            stmt.execute(params![e, n])?;
+            // get the row ID
+            let v_id = tx.last_insert_rowid();
+            // delete everything else by this name
+            let del_query = "DELETE FROM user_verification WHERE name = ? AND id != ?;";
+            let mut del_stmt = tx.prepare(del_query)?;
+            let count = del_stmt.execute(params![n,v_id])?;
+            if count > 0 {
+                info!("removed {} old verification records for ({:?},{:?})", count, n, a_prefix);
+            }
+        }
+        tx.commit()?;
+        info!("saved new verification record for ({:?},{:?})", n, a_prefix);
+        let ok: Result<()> = Ok(());
+        ok
+    }).await?
+}
+
+/// Retrieve the most recent verification record for a given pubkey (async).
+pub async fn get_latest_user_verification(
+    conn: db::PooledConnection,
+    pubkey: &str,
+) -> Result<VerificationRecord> {
+    let p = pubkey.to_owned();
+    tokio::task::spawn_blocking(move || query_latest_user_verification(conn, p)).await?
+}
+
+/// Query database for the latest verification record for a given pubkey.
+pub fn query_latest_user_verification(
+    mut conn: db::PooledConnection,
+    pubkey: String,
+) -> Result<VerificationRecord> {
+    let tx = conn.transaction()?;
+    let query = "SELECT v.id, v.name, e.event_hash, e.created_at, v.verified_at, v.failed_at, v.failure_count FROM user_verification v LEFT JOIN event e ON e.id=v.metadata_event WHERE e.author=? ORDER BY e.created_at DESC, v.verified_at DESC, v.failed_at DESC LIMIT 1;";
+    let mut stmt = tx.prepare_cached(query)?;
+    let fields = stmt.query_row(params![hex::decode(&pubkey).ok()], |r| {
+        let rowid: u64 = r.get(0)?;
+        let rowname: String = r.get(1)?;
+        let eventid: Vec<u8> = r.get(2)?;
+        let created_at: u64 = r.get(3)?;
+        // create a tuple since we can't throw non-rusqlite errors in this closure
+        Ok((
+            rowid,
+            rowname,
+            eventid,
+            created_at,
+            r.get(4).ok(),
+            r.get(5).ok(),
+            r.get(6)?,
+        ))
+    })?;
+    Ok(VerificationRecord {
+        rowid: fields.0,
+        name: Nip05Name::try_from(&fields.1[..])?,
+        address: pubkey,
+        event: hex::encode(fields.2),
+        event_created: fields.3,
+        last_success: fields.4,
+        last_failure: fields.5,
+        failure_count: fields.6,
+    })
+}
+
+/// Retrieve the oldest user verification (async)
+pub async fn get_oldest_user_verification(
+    conn: db::PooledConnection,
+    earliest: u64,
+) -> Result<VerificationRecord> {
+    tokio::task::spawn_blocking(move || query_oldest_user_verification(conn, earliest)).await?
+}
+
+pub fn query_oldest_user_verification(
+    mut conn: db::PooledConnection,
+    earliest: u64,
+) -> Result<VerificationRecord> {
+    let tx = conn.transaction()?;
+    let query = "SELECT v.id, v.name, e.event_hash, e.author, e.created_at, v.verified_at, v.failed_at, v.failure_count FROM user_verification v LEFT JOIN event e ON e.id=v.metadata_event WHERE (v.verified_at < ? OR v.verified_at IS NULL) AND (v.failed_at < ? OR v.failed_at IS NULL) ORDER BY v.verified_at ASC, v.failed_at ASC LIMIT 1;";
+    let mut stmt = tx.prepare_cached(query)?;
+    let fields = stmt.query_row(params![earliest, earliest], |r| {
+        let rowid: u64 = r.get(0)?;
+        let rowname: String = r.get(1)?;
+        let eventid: Vec<u8> = r.get(2)?;
+        let pubkey: Vec<u8> = r.get(3)?;
+        let created_at: u64 = r.get(4)?;
+        // create a tuple since we can't throw non-rusqlite errors in this closure
+        Ok((
+            rowid,
+            rowname,
+            eventid,
+            pubkey,
+            created_at,
+            r.get(5).ok(),
+            r.get(6).ok(),
+            r.get(7)?,
+        ))
+    })?;
+    let vr = VerificationRecord {
+        rowid: fields.0,
+        name: Nip05Name::try_from(&fields.1[..])?,
+        address: hex::encode(fields.3),
+        event: hex::encode(fields.2),
+        event_created: fields.4,
+        last_success: fields.5,
+        last_failure: fields.6,
+        failure_count: fields.7,
+    };
+    Ok(vr)
+}
+
 #[cfg(test)]
 mod tests {
    use super::*;
@@ -550,7 +762,7 @@ mod tests {
    fn local_from_inet() {
        let addr = "bob@example.com";
        let parsed = Nip05Name::try_from(addr);
-        assert!(parsed.is_ok());
+        assert!(!parsed.is_err());
        let v = parsed.unwrap();
        assert_eq!(v.local, "bob");
        assert_eq!(v.domain, "example.com");
@@ -1,99 +1,48 @@
+use crate::error;
+
 pub enum EventResultStatus {
    Saved,
    Duplicate,
-    Invalid,
-    Blocked,
-    RateLimited,
-    Error,
-    Restricted,
+    Error(String),
 }

 pub struct EventResult {
    pub id: String,
-    pub msg: String,
    pub status: EventResultStatus,
 }

 pub enum Notice {
    Message(String),
    EventResult(EventResult),
-    AuthChallenge(String),
-}
-
-impl EventResultStatus {
-    #[must_use]
-    pub fn to_bool(&self) -> bool {
-        match self {
-            Self::Duplicate | Self::Saved => true,
-            Self::Invalid | Self::Blocked | Self::RateLimited | Self::Error | Self::Restricted => false,
-        }
-    }
-
-    #[must_use]
-    pub fn prefix(&self) -> &'static str {
-        match self {
-            Self::Saved => "saved",
-            Self::Duplicate => "duplicate",
-            Self::Invalid => "invalid",
-            Self::Blocked => "blocked",
-            Self::RateLimited => "rate-limited",
-            Self::Error => "error",
-            Self::Restricted => "restricted",
-        }
-    }
 }

 impl Notice {
-    //pub fn err(err: error::Error, id: String) -> Notice {
-    //    Notice::err_msg(format!("{}", err), id)
-    //}
+    pub fn err(err: error::Error, id: String) -> Notice {
+        Notice::err_msg(format!("{}", err), id)
+    }

-    #[must_use]
    pub fn message(msg: String) -> Notice {
        Notice::Message(msg)
    }

-    fn prefixed(id: String, msg: &str, status: EventResultStatus) -> Notice {
-        let msg = format!("{}: {}", status.prefix(), msg);
-        Notice::EventResult(EventResult { id, msg, status })
-    }
-
-    #[must_use]
-    pub fn invalid(id: String, msg: &str) -> Notice {
-        Notice::prefixed(id, msg, EventResultStatus::Invalid)
-    }
-
-    #[must_use]
-    pub fn blocked(id: String, msg: &str) -> Notice {
-        Notice::prefixed(id, msg, EventResultStatus::Blocked)
-    }
-
-    #[must_use]
-    pub fn rate_limited(id: String, msg: &str) -> Notice {
-        Notice::prefixed(id, msg, EventResultStatus::RateLimited)
-    }
-
-    #[must_use]
-    pub fn duplicate(id: String) -> Notice {
-        Notice::prefixed(id, "", EventResultStatus::Duplicate)
-    }
-
-    #[must_use]
-    pub fn error(id: String, msg: &str) -> Notice {
-        Notice::prefixed(id, msg, EventResultStatus::Error)
-    }
-
-    #[must_use]
-    pub fn restricted(id: String, msg: &str) -> Notice {
-        Notice::prefixed(id, msg, EventResultStatus::Restricted)
-    }
-
-    #[must_use]
    pub fn saved(id: String) -> Notice {
        Notice::EventResult(EventResult {
            id,
-            msg: "".into(),
            status: EventResultStatus::Saved,
        })
    }
+
+    pub fn duplicate(id: String) -> Notice {
+        Notice::EventResult(EventResult {
+            id,
+            status: EventResultStatus::Duplicate,
+        })
+    }
+
+    pub fn err_msg(msg: String, id: String) -> Notice {
+        Notice::EventResult(EventResult {
+            id,
+            status: EventResultStatus::Error(msg),
+        })
+    }
 }
@@ -1,176 +0,0 @@
-//! LNBits payment processor
-use http::Uri;
-use hyper::client::connect::HttpConnector;
-use hyper::Client;
-use hyper_rustls::HttpsConnector;
-use nostr::Keys;
-use serde::{Deserialize, Serialize};
-use serde_json::Value;
-
-use async_trait::async_trait;
-use rand::Rng;
-
-use std::str::FromStr;
-use url::Url;
-
-use crate::{config::Settings, error::Error};
-
-use super::{InvoiceInfo, InvoiceStatus, PaymentProcessor};
-
-const APIPATH: &str = "/api/v1/payments/";
-
-/// Info LNBits expects in create invoice request
-#[derive(Serialize, Deserialize, Debug)]
-pub struct LNBitsCreateInvoice {
-    out: bool,
-    amount: u64,
-    memo: String,
-    webhook: String,
-    unit: String,
-    internal: bool,
-    expiry: u64,
-}
-
-/// Invoice response for LN bits
-#[derive(Debug, Serialize, Deserialize)]
-pub struct LNBitsCreateInvoiceResponse {
-    payment_hash: String,
-    payment_request: String,
-}
-
-/// LNBits call back response
-/// Used when an invoice is paid
-/// lnbits to post the status change to relay
-#[derive(Serialize, Deserialize, Debug, Clone)]
-pub struct LNBitsCallback {
-    pub checking_id: String,
-    pub pending: bool,
-    pub amount: u64,
-    pub memo: String,
-    pub time: u64,
-    pub bolt11: String,
-    pub preimage: String,
-    pub payment_hash: String,
-    pub wallet_id: String,
-    pub webhook: String,
-    pub webhook_status: Option<String>,
-}
-
-/// LN Bits repose for check invoice endpoint
-#[derive(Serialize, Deserialize, Debug, Clone)]
-pub struct LNBitsCheckInvoiceResponse {
-    paid: bool,
-}
-
-#[derive(Clone)]
-pub struct LNBitsPaymentProcessor {
-    /// HTTP client
-    client: hyper::Client<HttpsConnector<HttpConnector>, hyper::Body>,
-    settings: Settings,
-}
-
-impl LNBitsPaymentProcessor {
-    pub fn new(settings: &Settings) -> Self {
-        // setup hyper client
-        let https = hyper_rustls::HttpsConnectorBuilder::new()
-            .with_native_roots()
-            .https_only()
-            .enable_http1()
-            .build();
-        let client = Client::builder().build::<_, hyper::Body>(https);
-
-        Self {
-            client,
-            settings: settings.clone(),
-        }
-    }
-}
-
-#[async_trait]
-impl PaymentProcessor for LNBitsPaymentProcessor {
-    /// Calls LNBits api to ger new invoice
-    async fn get_invoice(&self, key: &Keys, amount: u64) -> Result<InvoiceInfo, Error> {
-        let random_number: u16 = rand::thread_rng().gen();
-        let memo = format!("{}: {}", random_number, key.public_key());
-
-        let callback_url = Url::parse(
-            &self
-                .settings
-                .info
-                .relay_url
-                .clone()
-                .unwrap()
-                .replace("ws", "http"),
-        )?
-        .join("lnbits")?;
-
-        let body = LNBitsCreateInvoice {
-            out: false,
-            amount,
-            memo: memo.clone(),
-            webhook: callback_url.to_string(),
-            unit: "sat".to_string(),
-            internal: false,
-            expiry: 3600,
-        };
-        let url = Url::parse(&self.settings.pay_to_relay.node_url)?.join(APIPATH)?;
-        let uri = Uri::from_str(url.as_str().strip_suffix('/').unwrap_or(url.as_str())).unwrap();
-
-        let req = hyper::Request::builder()
-            .method(hyper::Method::POST)
-            .uri(uri)
-            .header("X-Api-Key", &self.settings.pay_to_relay.api_secret)
-            .body(hyper::Body::from(serde_json::to_string(&body)?))
-            .expect("request builder");
-
-        let res = self.client.request(req).await?;
-
-        // Json to Struct of LNbits callback
-        let body = hyper::body::to_bytes(res.into_body()).await?;
-        let invoice_response: LNBitsCreateInvoiceResponse = serde_json::from_slice(&body)?;
-
-        Ok(InvoiceInfo {
-            pubkey: key.public_key().to_string(),
-            payment_hash: invoice_response.payment_hash,
-            bolt11: invoice_response.payment_request,
-            amount,
-            memo,
-            status: InvoiceStatus::Unpaid,
-            confirmed_at: None,
-        })
-    }
-
-    /// Calls LNBits Api to check the payment status of invoice
-    async fn check_invoice(&self, payment_hash: &str) -> Result<InvoiceStatus, Error> {
-        let url = Url::parse(&self.settings.pay_to_relay.node_url)?
-            .join(APIPATH)?
-            .join(payment_hash)?;
-        let uri = Uri::from_str(url.as_str()).unwrap();
-
-        let req = hyper::Request::builder()
-            .method(hyper::Method::GET)
-            .uri(uri)
-            .header("X-Api-Key", &self.settings.pay_to_relay.api_secret)
-            .body(hyper::Body::empty())
-            .expect("request builder");
-
-        let res = self.client.request(req).await?;
-        // Json to Struct of LNbits callback
-        let body = hyper::body::to_bytes(res.into_body()).await?;
-        let invoice_response: Value = serde_json::from_slice(&body)?;
-
-        let status = if let Ok(invoice_response) =
-            serde_json::from_value::<LNBitsCheckInvoiceResponse>(invoice_response)
-        {
-            if invoice_response.paid {
-                InvoiceStatus::Paid
-            } else {
-                InvoiceStatus::Unpaid
-            }
-        } else {
-            InvoiceStatus::Expired
-        };
-
-        Ok(status)
-    }
-}
@@ -1,276 +0,0 @@
-use crate::error::{Error, Result};
-use crate::event::Event;
-use crate::payment::lnbits::LNBitsPaymentProcessor;
-use crate::repo::NostrRepo;
-use serde::{Deserialize, Serialize};
-use std::sync::Arc;
-use tracing::{info, warn};
-
-use async_trait::async_trait;
-use nostr::key::{FromPkStr, FromSkStr};
-use nostr::{key::Keys, Event as NostrEvent, EventBuilder};
-
-pub mod lnbits;
-
-/// Payment handler
-pub struct Payment {
-    /// Repository for saving/retrieving events and events
-    repo: Arc<dyn NostrRepo>,
-    /// Newly validated events get written and then broadcast on this channel to subscribers
-    event_tx: tokio::sync::broadcast::Sender<Event>,
-    /// Payment message sender
-    payment_tx: tokio::sync::broadcast::Sender<PaymentMessage>,
-    /// Payment message receiver
-    payment_rx: tokio::sync::broadcast::Receiver<PaymentMessage>,
-    /// Settings
-    settings: crate::config::Settings,
-    // Nostr Keys
-    nostr_keys: Option<Keys>,
-    /// Payment Processor
-    processor: Arc<dyn PaymentProcessor>,
-}
-
-#[async_trait]
-pub trait PaymentProcessor: Send + Sync {
-    /// Get invoice from processor
-    async fn get_invoice(&self, keys: &Keys, amount: u64) -> Result<InvoiceInfo, Error>;
-    /// Check payment status of an invoice
-    async fn check_invoice(&self, payment_hash: &str) -> Result<InvoiceStatus, Error>;
-}
-
-#[derive(Debug, Serialize, Deserialize, Clone, PartialEq)]
-pub enum Processor {
-    LNBits,
-}
-
-/// Possible states of an invoice
-#[derive(Debug, Serialize, Deserialize, Clone, PartialEq, sqlx::Type)]
-#[sqlx(type_name = "status")]
-pub enum InvoiceStatus {
-    Unpaid,
-    Paid,
-    Expired,
-}
-
-impl ToString for InvoiceStatus {
-    fn to_string(&self) -> String {
-        match self {
-            InvoiceStatus::Paid => "Paid".to_string(),
-            InvoiceStatus::Unpaid => "Unpaid".to_string(),
-            InvoiceStatus::Expired => "Expired".to_string(),
-        }
-    }
-}
-
-/// Invoice information
-#[derive(Debug, Clone)]
-pub struct InvoiceInfo {
-    pub pubkey: String,
-    pub payment_hash: String,
-    pub bolt11: String,
-    pub amount: u64,
-    pub status: InvoiceStatus,
-    pub memo: String,
-    pub confirmed_at: Option<u64>,
-}
-
-/// Message variants for the payment channel
-#[derive(Debug, Clone)]
-pub enum PaymentMessage {
-    /// New account
-    NewAccount(String),
-    /// Check account,
-    CheckAccount(String),
-    /// Account Admitted
-    AccountAdmitted(String),
-    /// Invoice generated
-    Invoice(String, InvoiceInfo),
-    /// Invoice call back
-    /// Payment hash is passed
-    // This may have to be changed to better support other processors
-    InvoicePaid(String),
-}
-
-impl Payment {
-    pub fn new(
-        repo: Arc<dyn NostrRepo>,
-        payment_tx: tokio::sync::broadcast::Sender<PaymentMessage>,
-        payment_rx: tokio::sync::broadcast::Receiver<PaymentMessage>,
-        event_tx: tokio::sync::broadcast::Sender<Event>,
-        settings: crate::config::Settings,
-    ) -> Result<Self> {
-        info!("Create payment handler");
-
-        // Create nostr key from sk string
-        let nostr_keys = if let Some(secret_key) = &settings.pay_to_relay.secret_key {
-            Some(Keys::from_sk_str(secret_key)?)
-        } else {
-            None
-        };
-
-        // Create processor kind defined in settings
-        let processor = match &settings.pay_to_relay.processor {
-            Processor::LNBits => Arc::new(LNBitsPaymentProcessor::new(&settings)),
-        };
-
-        Ok(Payment {
-            repo,
-            payment_tx,
-            payment_rx,
-            event_tx,
-            settings,
-            nostr_keys,
-            processor,
-        })
-    }
-
-    /// Perform Payment tasks
-    pub async fn run(&mut self) {
-        loop {
-            let res = self.run_internal().await;
-            if let Err(e) = res {
-                info!("error in payment: {:?}", e);
-            }
-        }
-    }
-
-    /// Internal select loop for preforming payment operations
-    async fn run_internal(&mut self) -> Result<()> {
-        tokio::select! {
-            m = self.payment_rx.recv() => {
-                match m {
-                    Ok(PaymentMessage::NewAccount(pubkey)) => {
-                        info!("payment event for {:?}", pubkey);
-                        // REVIEW: This will need to change for cost per event
-                        let amount = self.settings.pay_to_relay.admission_cost;
-                        let invoice_info = self.get_invoice_info(&pubkey, amount).await?;
-                        // TODO: should handle this error
-                        self.payment_tx.send(PaymentMessage::Invoice(pubkey, invoice_info)).ok();
-                    },
-                    // Gets the most recent unpaid invoice from database
-                    // Checks LNbits to verify if paid/unpaid
-                    Ok(PaymentMessage::CheckAccount(pubkey)) => {
-                        let keys = Keys::from_pk_str(&pubkey)?;
-
-                        if let Ok(Some(invoice_info)) = self.repo.get_unpaid_invoice(&keys).await {
-                            match self.check_invoice_status(&invoice_info.payment_hash).await? {
-                                InvoiceStatus::Paid => {
-                                    self.repo.admit_account(&keys, self.settings.pay_to_relay.admission_cost).await?;
-                                    self.payment_tx.send(PaymentMessage::AccountAdmitted(pubkey)).ok();
-                                }
-                                _ => {
-                                    self.payment_tx.send(PaymentMessage::Invoice(pubkey, invoice_info)).ok();
-                                }
-                            }
-                        } else {
-                        let amount = self.settings.pay_to_relay.admission_cost;
-                            let invoice_info = self.get_invoice_info(&pubkey, amount).await?;
-                            self.payment_tx.send(PaymentMessage::Invoice(pubkey, invoice_info)).ok();
-                        }
-                    }
-                    Ok(PaymentMessage::InvoicePaid(payment_hash)) => {
-                        if self.check_invoice_status(&payment_hash).await?.eq(&InvoiceStatus::Paid) {
-                            let pubkey = self.repo
-                                .update_invoice(&payment_hash, InvoiceStatus::Paid)
-                                .await?;
-
-                            let key = Keys::from_pk_str(&pubkey)?;
-                            self.repo.admit_account(&key, self.settings.pay_to_relay.admission_cost).await?;
-                        }
-                    }
-                    Ok(_) => {
-                        // For this variant nothing need to be done here
-                        // it is used by `server`
-                    }
-                    Err(err) => warn!("Payment RX: {err}")
-                }
-            }
-        }
-
-        Ok(())
-    }
-
-    /// Sends Nostr DM to pubkey that requested invoice
-    /// Two events the terms followed by the bolt11 invoice
-    pub async fn send_admission_message(
-        &self,
-        pubkey: &str,
-        invoice_info: &InvoiceInfo,
-    ) -> Result<()> {
-        let nostr_keys = match &self.nostr_keys {
-            Some(key) => key,
-            None => return Err(Error::CustomError("Nostr key not defined".to_string())),
-        };
-
-        // Create Nostr key from pk
-        let key = Keys::from_pk_str(pubkey)?;
-
-        let pubkey = key.public_key();
-
-        // Event DM with terms of service
-        let message_event: NostrEvent = EventBuilder::new_encrypted_direct_msg(
-            nostr_keys,
-            pubkey,
-            &self.settings.pay_to_relay.terms_message,
-        )?
-        .to_event(nostr_keys)?;
-
-        // Event DM with invoice
-        let invoice_event: NostrEvent =
-            EventBuilder::new_encrypted_direct_msg(nostr_keys, pubkey, &invoice_info.bolt11)?
-                .to_event(nostr_keys)?;
-
-        // Persist DM events to DB
-        self.repo.write_event(&message_event.clone().into()).await?;
-        self.repo.write_event(&invoice_event.clone().into()).await?;
-
-        // Broadcast DM events
-        self.event_tx.send(message_event.clone().into()).ok();
-        self.event_tx.send(invoice_event.clone().into()).ok();
-
-        Ok(())
-    }
-
-    /// Get Invoice Info
-    /// If the has an active invoice that will be return
-    /// Otherwise a new invoice will be generated by the payment processor
-    pub async fn get_invoice_info(&self, pubkey: &str, amount: u64) -> Result<InvoiceInfo> {
-        // If user is already in DB this will be false
-        // This avoids recreating admission invoices
-        // I think it will continue to send DMs with the invoice
-        // If client continues to try and write to the relay (will be same invoice)
-        let key = Keys::from_pk_str(pubkey)?;
-        if !self.repo.create_account(&key).await? {
-            if let Ok(Some(invoice_info)) = self.repo.get_unpaid_invoice(&key).await {
-                return Ok(invoice_info);
-            }
-        }
-
-        let key = Keys::from_pk_str(pubkey)?;
-
-        let invoice_info = self.processor.get_invoice(&key, amount).await?;
-
-        // Persist invoice to DB
-        self.repo
-            .create_invoice_record(&key, invoice_info.clone())
-            .await?;
-
-        if self.settings.pay_to_relay.direct_message {
-            // Admission event invoice and terms to pubkey that is joining
-            self.send_admission_message(pubkey, &invoice_info).await?;
-        }
-
-        Ok(invoice_info)
-    }
-
-    /// Check paid status of invoice with LNbits
-    pub async fn check_invoice_status(&self, payment_hash: &str) -> Result<InvoiceStatus, Error> {
-        // Check base if passed expiry time
-        let status = self.processor.check_invoice(payment_hash).await?;
-        self.repo
-            .update_invoice(payment_hash, status.clone())
-            .await?;
-
-        Ok(status)
-    }
-}
@@ -1,98 +0,0 @@
-use crate::db::QueryResult;
-use crate::error::Result;
-use crate::event::Event;
-use crate::nip05::VerificationRecord;
-use crate::payment::{InvoiceInfo, InvoiceStatus};
-use crate::subscription::Subscription;
-use crate::utils::unix_time;
-use async_trait::async_trait;
-use nostr::Keys;
-use rand::Rng;
-
-pub mod postgres;
-pub mod postgres_migration;
-pub mod sqlite;
-pub mod sqlite_migration;
-
-#[async_trait]
-pub trait NostrRepo: Send + Sync {
-    /// Start the repository (any initialization or maintenance tasks can be kicked off here)
-    async fn start(&self) -> Result<()>;
-
-    /// Run migrations and return current version
-    async fn migrate_up(&self) -> Result<usize>;
-
-    /// Persist event to database
-    async fn write_event(&self, e: &Event) -> Result<u64>;
-
-    /// Perform a database query using a subscription.
-    ///
-    /// The [`Subscription`] is converted into a SQL query.  Each result
-    /// is published on the `query_tx` channel as it is returned.  If a
-    /// message becomes available on the `abandon_query_rx` channel, the
-    /// query is immediately aborted.
-    async fn query_subscription(
-        &self,
-        sub: Subscription,
-        client_id: String,
-        query_tx: tokio::sync::mpsc::Sender<QueryResult>,
-        mut abandon_query_rx: tokio::sync::oneshot::Receiver<()>,
-    ) -> Result<()>;
-
-    /// Perform normal maintenance
-    async fn optimize_db(&self) -> Result<()>;
-
-    /// Create a new verification record connected to a specific event
-    async fn create_verification_record(&self, event_id: &str, name: &str) -> Result<()>;
-
-    /// Update verification timestamp
-    async fn update_verification_timestamp(&self, id: u64) -> Result<()>;
-
-    /// Update verification record as failed
-    async fn fail_verification(&self, id: u64) -> Result<()>;
-
-    /// Delete verification record
-    async fn delete_verification(&self, id: u64) -> Result<()>;
-
-    /// Get the latest verification record for a given pubkey.
-    async fn get_latest_user_verification(&self, pub_key: &str) -> Result<VerificationRecord>;
-
-    /// Get oldest verification before timestamp
-    async fn get_oldest_user_verification(&self, before: u64) -> Result<VerificationRecord>;
-
-    /// Create a new account
-    async fn create_account(&self, pubkey: &Keys) -> Result<bool>;
-
-    /// Admit an account
-    async fn admit_account(&self, pubkey: &Keys, admission_cost: u64) -> Result<()>;
-
-    /// Gets user balance if they are an admitted pubkey
-    async fn get_account_balance(&self, pubkey: &Keys) -> Result<(bool, u64)>;
-
-    /// Update account balance
-    async fn update_account_balance(
-        &self,
-        pub_key: &Keys,
-        positive: bool,
-        new_balance: u64,
-    ) -> Result<()>;
-
-    /// Create invoice record
-    async fn create_invoice_record(&self, pubkey: &Keys, invoice_info: InvoiceInfo) -> Result<()>;
-
-    /// Update Invoice for given payment hash
-    async fn update_invoice(&self, payment_hash: &str, status: InvoiceStatus) -> Result<String>;
-
-    /// Get the most recent invoice for a given pubkey
-    /// invoice must be unpaid and not expired
-    async fn get_unpaid_invoice(&self, pubkey: &Keys) -> Result<Option<InvoiceInfo>>;
-}
-
-// Current time, with a slight forward jitter in seconds
-pub(crate) fn now_jitter(sec: u64) -> u64 {
-    // random time between now, and 10min in future.
-    let mut rng = rand::thread_rng();
-    let jitter_amount = rng.gen_range(0..sec);
-    let now = unix_time();
-    now.saturating_add(jitter_amount)
-}
@@ -1,965 +0,0 @@
-use crate::db::QueryResult;
-use crate::error::Result;
-use crate::event::{single_char_tagname, Event};
-use crate::nip05::{Nip05Name, VerificationRecord};
-use crate::payment::{InvoiceInfo, InvoiceStatus};
-use crate::repo::{now_jitter, NostrRepo};
-use crate::subscription::{ReqFilter, Subscription};
-use async_std::stream::StreamExt;
-use async_trait::async_trait;
-use chrono::{DateTime, TimeZone, Utc};
-use sqlx::postgres::PgRow;
-use sqlx::Error::RowNotFound;
-use sqlx::{Error, Execute, FromRow, Postgres, QueryBuilder, Row};
-use std::time::{Duration, Instant};
-
-use crate::error;
-use crate::hexrange::{hex_range, HexSearch};
-use crate::repo::postgres_migration::run_migrations;
-use crate::server::NostrMetrics;
-use crate::utils::{self, is_hex, is_lower_hex};
-use nostr::key::Keys;
-use tokio::sync::mpsc::Sender;
-use tokio::sync::oneshot::Receiver;
-use tracing::{debug, error, info, trace, warn};
-
-pub type PostgresPool = sqlx::pool::Pool<Postgres>;
-
-pub struct PostgresRepo {
-    conn: PostgresPool,
-    conn_write: PostgresPool,
-    metrics: NostrMetrics,
-}
-
-impl PostgresRepo {
-    pub fn new(c: PostgresPool, cw: PostgresPool, m: NostrMetrics) -> PostgresRepo {
-        PostgresRepo {
-            conn: c,
-            conn_write: cw,
-            metrics: m,
-        }
-    }
-}
-
-/// Cleanup expired events on a regular basis
-async fn cleanup_expired(conn: PostgresPool, frequency: Duration) -> Result<()> {
-    tokio::task::spawn(async move {
-        loop {
-            tokio::select! {
-                _ = tokio::time::sleep(frequency) => {
-                    let start = Instant::now();
-                    let exp_res = delete_expired(conn.clone()).await;
-                    match exp_res {
-                        Ok(exp_count) => {
-                            if exp_count > 0 {
-                                info!("removed {} expired events in: {:?}", exp_count, start.elapsed());
-                            }
-                        },
-                        Err(e) => {
-                            warn!("could not remove expired events due to error: {:?}", e);
-                        }
-                    }
-                }
-            };
-        }
-    });
-    Ok(())
-}
-
-/// One-time deletion of all expired events
-async fn delete_expired(conn: PostgresPool) -> Result<u64> {
-    let mut tx = conn.begin().await?;
-    let update_count = sqlx::query("DELETE FROM \"event\" WHERE expires_at <= $1;")
-        .bind(Utc.timestamp_opt(utils::unix_time() as i64, 0).unwrap())
-        .execute(&mut tx)
-        .await?
-        .rows_affected();
-    tx.commit().await?;
-    Ok(update_count)
-}
-
-#[async_trait]
-impl NostrRepo for PostgresRepo {
-    async fn start(&self) -> Result<()> {
-        // begin a cleanup task for expired events.
-        cleanup_expired(self.conn_write.clone(), Duration::from_secs(600)).await?;
-        Ok(())
-    }
-
-    async fn migrate_up(&self) -> Result<usize> {
-        Ok(run_migrations(&self.conn_write).await?)
-    }
-
-    async fn write_event(&self, e: &Event) -> Result<u64> {
-        // start transaction
-        let mut tx = self.conn_write.begin().await?;
-        let start = Instant::now();
-
-        // get relevant fields from event and convert to blobs.
-        let id_blob = hex::decode(&e.id).ok();
-        let pubkey_blob: Option<Vec<u8>> = hex::decode(&e.pubkey).ok();
-        let delegator_blob: Option<Vec<u8>> =
-            e.delegated_by.as_ref().and_then(|d| hex::decode(d).ok());
-        let event_str = serde_json::to_string(&e).unwrap();
-
-        // determine if this event would be shadowed by an existing
-        // replaceable event or parameterized replaceable event.
-        if e.is_replaceable() {
-            let repl_count = sqlx::query(
-                "SELECT e.id FROM event e WHERE e.pub_key=$1 AND e.kind=$2 AND e.created_at >= $3 LIMIT 1;")
-                .bind(&pubkey_blob)
-                .bind(e.kind as i64)
-                .bind(Utc.timestamp_opt(e.created_at as i64, 0).unwrap())
-                .fetch_optional(&mut tx)
-                .await?;
-            if repl_count.is_some() {
-                return Ok(0);
-            }
-        }
-        if let Some(d_tag) = e.distinct_param() {
-            let repl_count: i64 = if is_lower_hex(&d_tag) && (d_tag.len() % 2 == 0) {
-                sqlx::query_scalar(
-                    "SELECT count(*) AS count FROM event e LEFT JOIN tag t ON e.id=t.event_id WHERE e.pub_key=$1 AND e.kind=$2 AND t.name='d' AND t.value_hex=$3 AND e.created_at >= $4 LIMIT 1;")
-                    .bind(hex::decode(&e.pubkey).ok())
-                    .bind(e.kind as i64)
-                    .bind(hex::decode(d_tag).ok())
-                    .bind(Utc.timestamp_opt(e.created_at as i64, 0).unwrap())
-                    .fetch_one(&mut tx)
-                    .await?
-            } else {
-                sqlx::query_scalar(
-                    "SELECT count(*) AS count FROM event e LEFT JOIN tag t ON e.id=t.event_id WHERE e.pub_key=$1 AND e.kind=$2 AND t.name='d' AND t.value=$3 AND e.created_at >= $4 LIMIT 1;")
-                    .bind(hex::decode(&e.pubkey).ok())
-                    .bind(e.kind as i64)
-                    .bind(d_tag.as_bytes())
-                    .bind(Utc.timestamp_opt(e.created_at as i64, 0).unwrap())
-                    .fetch_one(&mut tx)
-                    .await?
-            };
-            // if any rows were returned, then some newer event with
-            // the same author/kind/tag value exist, and we can ignore
-            // this event.
-            if repl_count > 0 {
-                return Ok(0);
-            }
-        }
-        // ignore if the event hash is a duplicate.
-        let mut ins_count = sqlx::query(
-            r#"INSERT INTO "event"
-(id, pub_key, created_at, expires_at, kind, "content", delegated_by)
-VALUES($1, $2, $3, $4, $5, $6, $7)
-ON CONFLICT (id) DO NOTHING"#,
-        )
-        .bind(&id_blob)
-        .bind(&pubkey_blob)
-        .bind(Utc.timestamp_opt(e.created_at as i64, 0).unwrap())
-        .bind(
-            e.expiration()
-                .and_then(|x| Utc.timestamp_opt(x as i64, 0).latest()),
-        )
-        .bind(e.kind as i64)
-        .bind(event_str.into_bytes())
-        .bind(delegator_blob)
-        .execute(&mut tx)
-        .await?
-        .rows_affected();
-
-        if ins_count == 0 {
-            // if the event was a duplicate, no need to insert event or
-            // pubkey references.  This will abort the txn.
-            return Ok(0);
-        }
-
-        // add all tags to the tag table
-        for tag in e.tags.iter() {
-            // ensure we have 2 values.
-            if tag.len() >= 2 {
-                let tag_name = &tag[0];
-                let tag_val = &tag[1];
-                // only single-char tags are searchable
-                let tag_char_opt = single_char_tagname(tag_name);
-                match &tag_char_opt {
-                    Some(_) => {
-                        // if tag value is lowercase hex;
-                        if is_lower_hex(tag_val) && (tag_val.len() % 2 == 0) {
-                            sqlx::query("INSERT INTO tag (event_id, \"name\", value, value_hex) VALUES($1, $2, NULL, $3) \
-                    ON CONFLICT (event_id, \"name\", value, value_hex) DO NOTHING")
-                                .bind(&id_blob)
-                                .bind(tag_name)
-                                .bind(hex::decode(tag_val).ok())
-                                .execute(&mut tx)
-                                .await
-                                .unwrap();
-                        } else {
-                            sqlx::query("INSERT INTO tag (event_id, \"name\", value, value_hex) VALUES($1, $2, $3, NULL) \
-                    ON CONFLICT (event_id, \"name\", value, value_hex) DO NOTHING")
-                                .bind(&id_blob)
-                                .bind(tag_name)
-                                .bind(tag_val.as_bytes())
-                                .execute(&mut tx)
-                                .await
-                                .unwrap();
-                        }
-                    }
-                    None => {}
-                }
-            }
-        }
-        if e.is_replaceable() {
-            let update_count = sqlx::query("DELETE FROM \"event\" WHERE kind=$1 and pub_key = $2 and id not in (select id from \"event\" where kind=$1 and pub_key=$2 order by created_at desc limit 1);")
-                .bind(e.kind as i64)
-                .bind(hex::decode(&e.pubkey).ok())
-                .execute(&mut tx)
-                .await?.rows_affected();
-            if update_count > 0 {
-                info!(
-                    "hid {} older replaceable kind {} events for author: {:?}",
-                    update_count,
-                    e.kind,
-                    e.get_author_prefix()
-                );
-            }
-        }
-        // parameterized replaceable events
-        // check for parameterized replaceable events that would be hidden; don't insert these either.
-        if let Some(d_tag) = e.distinct_param() {
-            let update_count = if is_lower_hex(&d_tag) && (d_tag.len() % 2 == 0) {
-                sqlx::query("DELETE FROM event WHERE kind=$1 AND pub_key=$2 AND id IN (SELECT e.id FROM event e LEFT JOIN tag t ON e.id=t.event_id WHERE e.kind=$1 AND e.pub_key=$2 AND t.name='d' AND t.value_hex=$3 ORDER BY created_at DESC OFFSET 1);")
-                    .bind(e.kind as i64)
-                    .bind(hex::decode(&e.pubkey).ok())
-                    .bind(hex::decode(d_tag).ok())
-                    .execute(&mut tx)
-                    .await?.rows_affected()
-            } else {
-                sqlx::query("DELETE FROM event WHERE kind=$1 AND pub_key=$2 AND id IN (SELECT e.id FROM event e LEFT JOIN tag t ON e.id=t.event_id WHERE e.kind=$1 AND e.pub_key=$2 AND t.name='d' AND t.value=$3 ORDER BY created_at DESC OFFSET 1);")
-                    .bind(e.kind as i64)
-                    .bind(hex::decode(&e.pubkey).ok())
-                    .bind(d_tag.as_bytes())
-                    .execute(&mut tx)
-                    .await?.rows_affected()
-            };
-            if update_count > 0 {
-                info!(
-                    "removed {} older parameterized replaceable kind {} events for author: {:?}",
-                    update_count,
-                    e.kind,
-                    e.get_author_prefix()
-                );
-            }
-        }
-        // if this event is a deletion, hide the referenced events from the same author.
-        if e.kind == 5 {
-            let event_candidates = e.tag_values_by_name("e");
-            let pub_keys: Vec<Vec<u8>> = event_candidates
-                .iter()
-                .filter(|x| is_hex(x) && x.len() == 64)
-                .filter_map(|x| hex::decode(x).ok())
-                .collect();
-
-            let mut builder = QueryBuilder::new(
-                "UPDATE \"event\" SET hidden = 1::bit(1) WHERE kind != 5 AND pub_key = ",
-            );
-            builder.push_bind(hex::decode(&e.pubkey).ok());
-            builder.push(" AND id IN (");
-
-            let mut sep = builder.separated(", ");
-            for pk in pub_keys {
-                sep.push_bind(pk);
-            }
-            sep.push_unseparated(")");
-
-            let update_count = builder.build().execute(&mut tx).await?.rows_affected();
-            info!(
-                "hid {} deleted events for author {:?}",
-                update_count,
-                e.get_author_prefix()
-            );
-        } else {
-            // check if a deletion has already been recorded for this event.
-            // Only relevant for non-deletion events
-            let del_count = sqlx::query(
-                "SELECT e.id FROM \"event\" e \
-            LEFT JOIN tag t ON e.id = t.event_id \
-            WHERE e.pub_key = $1 AND t.\"name\" = 'e' AND e.kind = 5 AND t.value = $2 LIMIT 1",
-            )
-            .bind(&pubkey_blob)
-            .bind(&id_blob)
-            .fetch_optional(&mut tx)
-            .await?;
-
-            // check if a the query returned a result, meaning we should
-            // hid the current event
-            if del_count.is_some() {
-                // a deletion already existed, mark original event as hidden.
-                info!(
-                    "hid event: {:?} due to existing deletion by author: {:?}",
-                    e.get_event_id_prefix(),
-                    e.get_author_prefix()
-                );
-                sqlx::query("UPDATE \"event\" SET hidden = 1::bit(1) WHERE id = $1")
-                    .bind(&id_blob)
-                    .execute(&mut tx)
-                    .await?;
-                // event was deleted, so let caller know nothing new
-                // arrived, preventing this from being sent to active
-                // subscriptions
-                ins_count = 0;
-            }
-        }
-        tx.commit().await?;
-        self.metrics
-            .write_events
-            .observe(start.elapsed().as_secs_f64());
-        Ok(ins_count)
-    }
-
-    async fn query_subscription(
-        &self,
-        sub: Subscription,
-        client_id: String,
-        query_tx: Sender<QueryResult>,
-        mut abandon_query_rx: Receiver<()>,
-    ) -> Result<()> {
-        let start = Instant::now();
-        let mut row_count: usize = 0;
-        let metrics = &self.metrics;
-
-        for filter in sub.filters.iter() {
-            let start = Instant::now();
-            // generate SQL query
-            let q_filter = query_from_filter(filter);
-            if q_filter.is_none() {
-                debug!("Failed to generate query!");
-                continue;
-            }
-
-            debug!("SQL generated in {:?}", start.elapsed());
-
-            // cutoff for displaying slow queries
-            let slow_cutoff = Duration::from_millis(2000);
-
-            // any client that doesn't cause us to generate new rows in 5
-            // seconds gets dropped.
-            let abort_cutoff = Duration::from_secs(5);
-
-            let start = Instant::now();
-            let mut slow_first_event;
-            let mut last_successful_send = Instant::now();
-
-            // execute the query. Don't cache, since queries vary so much.
-            let mut q_filter = q_filter.unwrap();
-            let q_build = q_filter.build();
-            let sql = q_build.sql();
-            let mut results = q_build.fetch(&self.conn);
-
-            let mut first_result = true;
-            while let Some(row) = results.next().await {
-                if let Err(e) = row {
-                    error!("Query failed: {} {} {:?}", e, sql, filter);
-                    break;
-                }
-                let first_event_elapsed = start.elapsed();
-                slow_first_event = first_event_elapsed >= slow_cutoff;
-                if first_result {
-                    debug!(
-                        "first result in {:?} (cid: {}, sub: {:?})",
-                        first_event_elapsed, client_id, sub.id
-                    );
-                    first_result = false;
-                }
-
-                // logging for slow queries; show sub and SQL.
-                // to reduce logging; only show 1/16th of clients (leading 0)
-                if slow_first_event && client_id.starts_with("00") {
-                    debug!(
-                        "query req (slow): {:?} (cid: {}, sub: {:?})",
-                        &sub, client_id, sub.id
-                    );
-                } else {
-                    trace!(
-                        "query req: {:?} (cid: {}, sub: {:?})",
-                        &sub,
-                        client_id,
-                        sub.id
-                    );
-                }
-
-                // check if this is still active; every 100 rows
-                if row_count % 100 == 0 && abandon_query_rx.try_recv().is_ok() {
-                    debug!(
-                        "query cancelled by client (cid: {}, sub: {:?})",
-                        client_id, sub.id
-                    );
-                    return Ok(());
-                }
-
-                row_count += 1;
-                let event_json: Vec<u8> = row.unwrap().get(0);
-                loop {
-                    if query_tx.capacity() != 0 {
-                        // we have capacity to add another item
-                        break;
-                    } else {
-                        // the queue is full
-                        trace!("db reader thread is stalled");
-                        if last_successful_send + abort_cutoff < Instant::now() {
-                            // the queue has been full for too long, abort
-                            info!("aborting database query due to slow client");
-                            metrics
-                                .query_aborts
-                                .with_label_values(&["slowclient"])
-                                .inc();
-                            return Ok(());
-                        }
-                        // give the queue a chance to clear before trying again
-                        async_std::task::sleep(Duration::from_millis(100)).await;
-                    }
-                }
-
-                // TODO: we could use try_send, but we'd have to juggle
-                // getting the query result back as part of the error
-                // result.
-                query_tx
-                    .send(QueryResult {
-                        sub_id: sub.get_id(),
-                        event: String::from_utf8(event_json).unwrap(),
-                    })
-                    .await
-                    .ok();
-                last_successful_send = Instant::now();
-            }
-        }
-        query_tx
-            .send(QueryResult {
-                sub_id: sub.get_id(),
-                event: "EOSE".to_string(),
-            })
-            .await
-            .ok();
-        self.metrics
-            .query_sub
-            .observe(start.elapsed().as_secs_f64());
-        debug!(
-            "query completed in {:?} (cid: {}, sub: {:?}, db_time: {:?}, rows: {})",
-            start.elapsed(),
-            client_id,
-            sub.id,
-            start.elapsed(),
-            row_count
-        );
-        Ok(())
-    }
-
-    async fn optimize_db(&self) -> Result<()> {
-        // Not implemented
-        Ok(())
-    }
-
-    async fn create_verification_record(&self, event_id: &str, name: &str) -> Result<()> {
-        let mut tx = self.conn_write.begin().await?;
-
-        sqlx::query("DELETE FROM user_verification WHERE \"name\" = $1")
-            .bind(name)
-            .execute(&mut tx)
-            .await?;
-
-        sqlx::query("INSERT INTO user_verification (event_id, \"name\", verified_at) VALUES ($1, $2, now())")
-            .bind(hex::decode(event_id).ok())
-            .bind(name)
-            .execute(&mut tx)
-            .await?;
-
-        tx.commit().await?;
-        info!("saved new verification record for ({:?})", name);
-        Ok(())
-    }
-
-    async fn update_verification_timestamp(&self, id: u64) -> Result<()> {
-        // add some jitter to the verification to prevent everything from stacking up together.
-        let verify_time = now_jitter(600);
-
-        // update verification time and reset any failure count
-        sqlx::query("UPDATE user_verification SET verified_at = $1, fail_count = 0 WHERE id = $2")
-            .bind(Utc.timestamp_opt(verify_time as i64, 0).unwrap())
-            .bind(id as i64)
-            .execute(&self.conn_write)
-            .await?;
-
-        info!("verification updated for {}", id);
-        Ok(())
-    }
-
-    async fn fail_verification(&self, id: u64) -> Result<()> {
-        sqlx::query("UPDATE user_verification SET failed_at = now(), fail_count = fail_count + 1 WHERE id = $1")
-            .bind(id as i64)
-            .execute(&self.conn_write)
-            .await?;
-        Ok(())
-    }
-
-    async fn delete_verification(&self, id: u64) -> Result<()> {
-        sqlx::query("DELETE FROM user_verification WHERE id = $1")
-            .bind(id as i64)
-            .execute(&self.conn_write)
-            .await?;
-        Ok(())
-    }
-
-    async fn get_latest_user_verification(&self, pub_key: &str) -> Result<VerificationRecord> {
-        let query = r#"SELECT
-            v.id,
-            v."name",
-            e.id as event_id,
-            e.pub_key,
-            e.created_at,
-            v.verified_at,
-            v.failed_at,
-            v.fail_count
-            FROM user_verification v
-            LEFT JOIN "event" e ON e.id = v.event_id
-            WHERE e.pub_key = $1
-            ORDER BY e.created_at DESC, v.verified_at DESC, v.failed_at DESC
-            LIMIT 1"#;
-        sqlx::query_as::<_, VerificationRecord>(query)
-            .bind(hex::decode(pub_key).ok())
-            .fetch_optional(&self.conn)
-            .await?
-            .ok_or(error::Error::SqlxError(RowNotFound))
-    }
-
-    async fn get_oldest_user_verification(&self, before: u64) -> Result<VerificationRecord> {
-        let query = r#"SELECT
-            v.id,
-            v."name",
-            e.id as event_id,
-            e.pub_key,
-            e.created_at,
-            v.verified_at,
-            v.failed_at,
-            v.fail_count
-            FROM user_verification v
-            LEFT JOIN "event" e ON e.id = v.event_id
-                WHERE (v.verified_at < $1 OR v.verified_at IS NULL)
-                AND (v.failed_at < $1 OR v.failed_at IS NULL)
-            ORDER BY v.verified_at ASC, v.failed_at ASC
-            LIMIT 1"#;
-        sqlx::query_as::<_, VerificationRecord>(query)
-            .bind(Utc.timestamp_opt(before as i64, 0).unwrap())
-            .fetch_optional(&self.conn)
-            .await?
-            .ok_or(error::Error::SqlxError(RowNotFound))
-    }
-
-    async fn create_account(&self, pub_key: &Keys) -> Result<bool> {
-        let pub_key = pub_key.public_key().to_string();
-        let mut tx = self.conn_write.begin().await?;
-
-        let result = sqlx::query("INSERT INTO account (pubkey, balance) VALUES ($1, 0);")
-            .bind(pub_key)
-            .execute(&mut tx)
-            .await;
-
-        let success = match result {
-            Ok(res) => {
-                tx.commit().await?;
-                res.rows_affected() == 1
-            }
-            Err(_err) => false,
-        };
-
-        Ok(success)
-    }
-
-    /// Admit account
-    async fn admit_account(&self, pub_key: &Keys, admission_cost: u64) -> Result<()> {
-        let pub_key = pub_key.public_key().to_string();
-        sqlx::query(
-            "UPDATE account SET is_admitted = TRUE, balance = balance - $1 WHERE pubkey = $2",
-        )
-        .bind(admission_cost as i64)
-        .bind(pub_key)
-        .execute(&self.conn_write)
-        .await?;
-        Ok(())
-    }
-
-    /// Gets if the account is admitted and balance
-    async fn get_account_balance(&self, pub_key: &Keys) -> Result<(bool, u64)> {
-        let pub_key = pub_key.public_key().to_string();
-        let query = r#"SELECT
-            is_admitted,
-            balance
-            FROM account
-            WHERE pubkey = $1
-            LIMIT 1"#;
-
-        let result = sqlx::query_as::<_, (bool, i64)>(query)
-            .bind(pub_key)
-            .fetch_optional(&self.conn_write)
-            .await?
-            .ok_or(error::Error::SqlxError(RowNotFound))?;
-
-        Ok((result.0, result.1 as u64))
-    }
-
-    /// Update account balance
-    async fn update_account_balance(
-        &self,
-        pub_key: &Keys,
-        positive: bool,
-        new_balance: u64,
-    ) -> Result<()> {
-        let pub_key = pub_key.public_key().to_string();
-        match positive {
-            true => {
-                sqlx::query("UPDATE account SET balance = balance + $1 WHERE pubkey = $2")
-                    .bind(new_balance as i64)
-                    .bind(pub_key)
-                    .execute(&self.conn_write)
-                    .await?
-            }
-            false => {
-                sqlx::query("UPDATE account SET balance = balance - $1 WHERE pubkey = $2")
-                    .bind(new_balance as i64)
-                    .bind(pub_key)
-                    .execute(&self.conn_write)
-                    .await?
-            }
-        };
-        Ok(())
-    }
-
-    /// Create invoice record
-    async fn create_invoice_record(&self, pub_key: &Keys, invoice_info: InvoiceInfo) -> Result<()> {
-        let pub_key = pub_key.public_key().to_string();
-        let mut tx = self.conn_write.begin().await?;
-
-        sqlx::query(
-            "INSERT INTO invoice (pubkey, payment_hash, amount, status, description, created_at, invoice) VALUES ($1, $2, $3, $4, $5, now(), $6)",
-        )
-        .bind(pub_key)
-        .bind(invoice_info.payment_hash)
-        .bind(invoice_info.amount as i64)
-        .bind(invoice_info.status)
-        .bind(invoice_info.memo)
-        .bind(invoice_info.bolt11)
-        .execute(&mut tx)
-        .await.unwrap();
-
-        debug!("Invoice added");
-
-        tx.commit().await?;
-        Ok(())
-    }
-
-    /// Update invoice record
-    async fn update_invoice(&self, payment_hash: &str, status: InvoiceStatus) -> Result<String> {
-        debug!("Payment Hash: {}", payment_hash);
-        let query = "SELECT pubkey, status, amount FROM invoice WHERE payment_hash=$1;";
-        let (pubkey, prev_invoice_status, amount) =
-            sqlx::query_as::<_, (String, InvoiceStatus, i64)>(query)
-                .bind(payment_hash)
-                .fetch_optional(&self.conn_write)
-                .await?
-                .ok_or(error::Error::SqlxError(RowNotFound))?;
-
-        // If the invoice is paid update the confirmed at timestamp
-        let query = if status.eq(&InvoiceStatus::Paid) {
-            "UPDATE invoice SET status=$1, confirmed_at = now() WHERE payment_hash=$2;"
-        } else {
-            "UPDATE invoice SET status=$1 WHERE payment_hash=$2;"
-        };
-
-        sqlx::query(query)
-            .bind(&status)
-            .bind(payment_hash)
-            .execute(&self.conn_write)
-            .await?;
-
-        if prev_invoice_status.eq(&InvoiceStatus::Unpaid) && status.eq(&InvoiceStatus::Paid) {
-            sqlx::query("UPDATE account SET balance = balance + $1 WHERE pubkey = $2")
-                .bind(amount)
-                .bind(&pubkey)
-                .execute(&self.conn_write)
-                .await?;
-        }
-
-        Ok(pubkey)
-    }
-
-    /// Get the most recent invoice for a given pubkey
-    /// invoice must be unpaid and not expired
-    async fn get_unpaid_invoice(&self, pubkey: &Keys) -> Result<Option<InvoiceInfo>> {
-        let query = r#"
-SELECT amount, payment_hash, description, invoice
-FROM invoice
-WHERE pubkey = $1
-ORDER BY created_at DESC
-LIMIT 1;
-        "#;
-        match sqlx::query_as::<_, (i64, String, String, String)>(query)
-            .bind(pubkey.public_key().to_string())
-            .fetch_optional(&self.conn_write)
-            .await
-            .unwrap()
-        {
-            Some((amount, payment_hash, description, invoice)) => Ok(Some(InvoiceInfo {
-                pubkey: pubkey.public_key().to_string(),
-                payment_hash,
-                bolt11: invoice,
-                amount: amount as u64,
-                status: InvoiceStatus::Unpaid,
-                memo: description,
-                confirmed_at: None,
-            })),
-            None => Ok(None),
-        }
-    }
-}
-
-/// Create a dynamic SQL query and params from a subscription filter.
-fn query_from_filter(f: &ReqFilter) -> Option<QueryBuilder<Postgres>> {
-    // if the filter is malformed, don't return anything.
-    if f.force_no_match {
-        return None;
-    }
-
-    let mut query = QueryBuilder::new("SELECT e.\"content\", e.created_at FROM \"event\" e WHERE ");
-
-    // This tracks whether we need to push a prefix AND before adding another clause
-    let mut push_and = false;
-    // Query for "authors", allowing prefix matches
-    if let Some(auth_vec) = &f.authors {
-        // filter out non-hex values
-        let auth_vec: Vec<&String> = auth_vec.iter().filter(|a| is_hex(a)).collect();
-
-        if !auth_vec.is_empty() {
-            query.push("(");
-
-            // shortcut authors into "IN" query
-            let any_is_range = auth_vec.iter().any(|pk| pk.len() != 64);
-            if !any_is_range {
-                query.push("e.pub_key in (");
-                let mut pk_sep = query.separated(", ");
-                for pk in auth_vec.iter() {
-                    pk_sep.push_bind(hex::decode(pk).ok());
-                }
-                query.push(") OR e.delegated_by in (");
-                let mut pk_delegated_sep = query.separated(", ");
-                for pk in auth_vec.iter() {
-                    pk_delegated_sep.push_bind(hex::decode(pk).ok());
-                }
-                query.push(")");
-                push_and = true;
-            } else {
-                let mut range_authors = query.separated(" OR ");
-                for auth in auth_vec {
-                    match hex_range(auth) {
-                        Some(HexSearch::Exact(ex)) => {
-                            range_authors
-                                .push("(e.pub_key = ")
-                                .push_bind_unseparated(ex.clone())
-                                .push_unseparated(" OR e.delegated_by = ")
-                                .push_bind_unseparated(ex)
-                                .push_unseparated(")");
-                        }
-                        Some(HexSearch::Range(lower, upper)) => {
-                            range_authors
-                                .push("((e.pub_key > ")
-                                .push_bind_unseparated(lower.clone())
-                                .push_unseparated(" AND e.pub_key < ")
-                                .push_bind_unseparated(upper.clone())
-                                .push_unseparated(") OR (e.delegated_by > ")
-                                .push_bind_unseparated(lower)
-                                .push_unseparated(" AND e.delegated_by < ")
-                                .push_bind_unseparated(upper)
-                                .push_unseparated("))");
-                        }
-                        Some(HexSearch::LowerOnly(lower)) => {
-                            range_authors
-                                .push("(e.pub_key > ")
-                                .push_bind_unseparated(lower.clone())
-                                .push_unseparated(" OR e.delegated_by > ")
-                                .push_bind_unseparated(lower)
-                                .push_unseparated(")");
-                        }
-                        None => {
-                            info!("Could not parse hex range from author {:?}", auth);
-                        }
-                    }
-                    push_and = true;
-                }
-            }
-            query.push(")");
-        }
-    }
-
-    // Query for Kind
-    if let Some(ks) = &f.kinds {
-        if !ks.is_empty() {
-            if push_and {
-                query.push(" AND ");
-            }
-            push_and = true;
-
-            query.push("e.kind in (");
-            let mut list_query = query.separated(", ");
-            for k in ks.iter() {
-                list_query.push_bind(*k as i64);
-            }
-            query.push(")");
-        }
-    }
-
-    // Query for event, allowing prefix matches
-    if let Some(id_vec) = &f.ids {
-        // filter out non-hex values
-        let id_vec: Vec<&String> = id_vec.iter().filter(|a| is_hex(a)).collect();
-
-        if !id_vec.is_empty() {
-            if push_and {
-                query.push(" AND (");
-            } else {
-                query.push("(");
-            }
-            push_and = true;
-
-            // shortcut ids into "IN" query
-            let any_is_range = id_vec.iter().any(|pk| pk.len() != 64);
-            if !any_is_range {
-                query.push("id in (");
-                let mut sep = query.separated(", ");
-                for id in id_vec.iter() {
-                    sep.push_bind(hex::decode(id).ok());
-                }
-                query.push(")");
-            } else {
-                // take each author and convert to a hex search
-                let mut id_query = query.separated(" OR ");
-                for id in id_vec {
-                    match hex_range(id) {
-                        Some(HexSearch::Exact(ex)) => {
-                            id_query
-                                .push("(id = ")
-                                .push_bind_unseparated(ex)
-                                .push_unseparated(")");
-                        }
-                        Some(HexSearch::Range(lower, upper)) => {
-                            id_query
-                                .push("(id > ")
-                                .push_bind_unseparated(lower)
-                                .push_unseparated(" AND id < ")
-                                .push_bind_unseparated(upper)
-                                .push_unseparated(")");
-                        }
-                        Some(HexSearch::LowerOnly(lower)) => {
-                            id_query
-                                .push("(id > ")
-                                .push_bind_unseparated(lower)
-                                .push_unseparated(")");
-                        }
-                        None => {
-                            info!("Could not parse hex range from id {:?}", id);
-                        }
-                    }
-                }
-            }
-
-            query.push(")");
-        }
-    }
-
-    // Query for tags
-    if let Some(map) = &f.tags {
-        if !map.is_empty() {
-            if push_and {
-                query.push(" AND ");
-            }
-            push_and = true;
-
-            for (key, val) in map.iter() {
-                query.push("e.id IN (SELECT ee.id FROM \"event\" ee LEFT JOIN tag t on ee.id = t.event_id WHERE ee.hidden != 1::bit(1) and (t.\"name\" = ")
-                    .push_bind(key.to_string())
-                    .push(" AND (value in (");
-
-                // plain value match first
-                let mut tag_query = query.separated(", ");
-                for v in val.iter() {
-                    if (v.len() % 2 != 0) && !is_lower_hex(v) {
-                        tag_query.push_bind(v.as_bytes());
-                    } else {
-                        tag_query.push_bind(hex::decode(v).ok());
-                    }
-                }
-                query.push("))))");
-            }
-        }
-    }
-
-    // Query for timestamp
-    if f.since.is_some() {
-        if push_and {
-            query.push(" AND ");
-        }
-        push_and = true;
-        query
-            .push("e.created_at >= ")
-            .push_bind(Utc.timestamp_opt(f.since.unwrap() as i64, 0).unwrap());
-    }
-
-    // Query for timestamp
-    if f.until.is_some() {
-        if push_and {
-            query.push(" AND ");
-        }
-        push_and = true;
-        query
-            .push("e.created_at <= ")
-            .push_bind(Utc.timestamp_opt(f.until.unwrap() as i64, 0).unwrap());
-    }
-
-    // never display hidden events
-    if push_and {
-        query.push(" AND e.hidden != 1::bit(1)");
-    } else {
-        query.push("e.hidden != 1::bit(1)");
-    }
-    // never display expired events
-    query
-        .push(" AND (e.expires_at IS NULL OR e.expires_at > ")
-        .push_bind(Utc.timestamp_opt(utils::unix_time() as i64, 0).unwrap())
-        .push(")");
-
-    // Apply per-filter limit to this query.
-    // The use of a LIMIT implies a DESC order, to capture only the most recent events.
-    if let Some(lim) = f.limit {
-        query.push(" ORDER BY e.created_at DESC LIMIT ");
-        query.push(lim.min(1000));
-    } else {
-        query.push(" ORDER BY e.created_at ASC LIMIT ");
-        query.push(1000);
-    }
-    Some(query)
-}
-
-impl FromRow<'_, PgRow> for VerificationRecord {
-    fn from_row(row: &'_ PgRow) -> std::result::Result<Self, Error> {
-        let name = Nip05Name::try_from(row.get::<'_, &str, &str>("name")).or(Err(RowNotFound))?;
-        Ok(VerificationRecord {
-            rowid: row.get::<'_, i64, &str>("id") as u64,
-            name,
-            address: hex::encode(row.get::<'_, Vec<u8>, &str>("pub_key")),
-            event: hex::encode(row.get::<'_, Vec<u8>, &str>("event_id")),
-            event_created: row.get::<'_, DateTime<Utc>, &str>("created_at").timestamp() as u64,
-            last_success: match row.try_get::<'_, DateTime<Utc>, &str>("verified_at") {
-                Ok(x) => Some(x.timestamp() as u64),
-                _ => None,
-            },
-            last_failure: match row.try_get::<'_, DateTime<Utc>, &str>("failed_at") {
-                Ok(x) => Some(x.timestamp() as u64),
-                _ => None,
-            },
-            failure_count: row.get::<'_, i32, &str>("fail_count") as u64,
-        })
-    }
-}
@@ -1,320 +0,0 @@
-use crate::repo::postgres::PostgresPool;
-use async_trait::async_trait;
-use sqlx::{Executor, Postgres, Transaction};
-
-#[async_trait]
-pub trait Migration {
-    fn serial_number(&self) -> i64;
-    async fn run(&self, tx: &mut Transaction<Postgres>);
-}
-
-struct SimpleSqlMigration {
-    pub serial_number: i64,
-    pub sql: Vec<&'static str>,
-}
-
-#[async_trait]
-impl Migration for SimpleSqlMigration {
-    fn serial_number(&self) -> i64 {
-        self.serial_number
-    }
-
-    async fn run(&self, tx: &mut Transaction<Postgres>) {
-        for sql in self.sql.iter() {
-            tx.execute(*sql).await.unwrap();
-        }
-    }
-}
-
-/// Execute all migrations on the database.
-pub async fn run_migrations(db: &PostgresPool) -> crate::error::Result<usize> {
-    prepare_migrations_table(db).await;
-    run_migration(m001::migration(), db).await;
-    let m002_result = run_migration(m002::migration(), db).await;
-    if m002_result == MigrationResult::Upgraded {
-        m002::rebuild_tags(db).await?;
-    }
-    run_migration(m003::migration(), db).await;
-    run_migration(m004::migration(), db).await;
-    run_migration(m005::migration(), db).await;
-    Ok(current_version(db).await as usize)
-}
-
-async fn current_version(db: &PostgresPool) -> i64 {
-    sqlx::query_scalar("SELECT max(serial_number) FROM migrations;")
-        .fetch_one(db)
-        .await
-        .unwrap()
-}
-
-async fn prepare_migrations_table(db: &PostgresPool) {
-    sqlx::query("CREATE TABLE IF NOT EXISTS migrations (serial_number bigint)")
-        .execute(db)
-        .await
-        .unwrap();
-}
-
-// Running a migration was either unnecessary, or completed
-#[derive(PartialEq, Eq, Debug, Clone)]
-enum MigrationResult {
-    Upgraded,
-    NotNeeded,
-}
-
-async fn run_migration(migration: impl Migration, db: &PostgresPool) -> MigrationResult {
-    let row: i64 =
-        sqlx::query_scalar("SELECT COUNT(*) AS count FROM migrations WHERE serial_number = $1")
-            .bind(migration.serial_number())
-            .fetch_one(db)
-            .await
-            .unwrap();
-
-    if row > 0 {
-        return MigrationResult::NotNeeded;
-    }
-
-    let mut transaction = db.begin().await.unwrap();
-    migration.run(&mut transaction).await;
-
-    sqlx::query("INSERT INTO migrations VALUES ($1)")
-        .bind(migration.serial_number())
-        .execute(&mut transaction)
-        .await
-        .unwrap();
-
-    transaction.commit().await.unwrap();
-    MigrationResult::Upgraded
-}
-
-mod m001 {
-    use crate::repo::postgres_migration::{Migration, SimpleSqlMigration};
-
-    pub const VERSION: i64 = 1;
-
-    pub fn migration() -> impl Migration {
-        SimpleSqlMigration {
-            serial_number: VERSION,
-            sql: vec![
-                r#"
-- Events table
-CREATE TABLE "event" (
-	id bytea NOT NULL,
-	pub_key bytea NOT NULL,
-	created_at timestamp with time zone NOT NULL,
-	kind integer NOT NULL,
-	"content" bytea NOT NULL,
-	hidden bit(1) NOT NULL DEFAULT 0::bit(1),
-	delegated_by bytea NULL,
-	first_seen timestamp with time zone NOT NULL DEFAULT now(),
-	CONSTRAINT event_pkey PRIMARY KEY (id)
-);
-CREATE INDEX event_created_at_idx ON "event" (created_at,kind);
-CREATE INDEX event_pub_key_idx ON "event" (pub_key);
-CREATE INDEX event_delegated_by_idx ON "event" (delegated_by);
-
-- Tags table
-CREATE TABLE "tag" (
-	id int8 NOT NULL GENERATED BY DEFAULT AS IDENTITY,
-	event_id bytea NOT NULL,
-	"name" varchar NOT NULL,
-	value bytea NOT NULL,
-	CONSTRAINT tag_fk FOREIGN KEY (event_id) REFERENCES "event"(id) ON DELETE CASCADE
-);
-CREATE INDEX tag_event_id_idx ON tag USING btree (event_id, name);
-CREATE INDEX tag_value_idx ON tag USING btree (value);
-
-- NIP-05 Verification table
-CREATE TABLE "user_verification" (
-	id int8 NOT NULL GENERATED BY DEFAULT AS IDENTITY,
-	event_id bytea NOT NULL,
-	"name" varchar NOT NULL,
-	verified_at timestamptz NULL,
-	failed_at timestamptz NULL,
-	fail_count int4 NULL DEFAULT 0,
-	CONSTRAINT user_verification_pk PRIMARY KEY (id),
-	CONSTRAINT user_verification_fk FOREIGN KEY (event_id) REFERENCES "event"(id) ON DELETE CASCADE
-);
-CREATE INDEX user_verification_event_id_idx ON user_verification USING btree (event_id);
-CREATE INDEX user_verification_name_idx ON user_verification USING btree (name);
-        "#,
-            ],
-        }
-    }
-}
-
-mod m002 {
-    use async_std::stream::StreamExt;
-    use indicatif::{ProgressBar, ProgressStyle};
-    use sqlx::Row;
-    use std::time::Instant;
-    use tracing::info;
-
-    use crate::event::{single_char_tagname, Event};
-    use crate::repo::postgres::PostgresPool;
-    use crate::repo::postgres_migration::{Migration, SimpleSqlMigration};
-    use crate::utils::is_lower_hex;
-
-    pub const VERSION: i64 = 2;
-
-    pub fn migration() -> impl Migration {
-        SimpleSqlMigration {
-            serial_number: VERSION,
-            sql: vec![
-                r#"
-- Add tag value column
-ALTER TABLE tag ADD COLUMN value_hex bytea;
-- Remove not-null constraint
-ALTER TABLE tag ALTER COLUMN value DROP NOT NULL;
-- Add value index
-CREATE INDEX tag_value_hex_idx ON tag USING btree (value_hex);
-        "#,
-            ],
-        }
-    }
-
-    pub async fn rebuild_tags(db: &PostgresPool) -> crate::error::Result<()> {
-        // Check how many events we have to process
-        let start = Instant::now();
-        let mut tx = db.begin().await.unwrap();
-        let mut update_tx = db.begin().await.unwrap();
-        // Clear out table
-        sqlx::query("DELETE FROM tag;")
-            .execute(&mut update_tx)
-            .await?;
-        {
-            let event_count: i64 = sqlx::query_scalar("SELECT COUNT(*) from event;")
-                .fetch_one(&mut tx)
-                .await
-                .unwrap();
-            let bar = ProgressBar::new(event_count.try_into().unwrap())
-                .with_message("rebuilding tags table");
-            bar.set_style(
-                ProgressStyle::with_template(
-                    "[{elapsed_precise}] {bar:40.white/blue} {pos:>7}/{len:7} [{percent}%] {msg}",
-                )
-                .unwrap(),
-            );
-            let mut events =
-                sqlx::query("SELECT id, content FROM event ORDER BY id;").fetch(&mut tx);
-            while let Some(row) = events.next().await {
-                bar.inc(1);
-                // get the row id and content
-                let row = row.unwrap();
-                let event_id: Vec<u8> = row.get(0);
-                let event_bytes: Vec<u8> = row.get(1);
-                let event: Event = serde_json::from_str(&String::from_utf8(event_bytes).unwrap())?;
-
-                for t in event.tags.iter().filter(|x| x.len() > 1) {
-                    let tagname = t.get(0).unwrap();
-                    let tagnamechar_opt = single_char_tagname(tagname);
-                    if tagnamechar_opt.is_none() {
-                        continue;
-                    }
-                    // safe because len was > 1
-                    let tagval = t.get(1).unwrap();
-                    // insert as BLOB if we can restore it losslessly.
-                    // this means it needs to be even length and lowercase.
-                    if (tagval.len() % 2 == 0) && is_lower_hex(tagval) {
-                        let q = "INSERT INTO tag (event_id, \"name\", value, value_hex) VALUES ($1, $2, NULL, $3) ON CONFLICT DO NOTHING;";
-                        sqlx::query(q)
-                            .bind(&event_id)
-                            .bind(tagname)
-                            .bind(hex::decode(tagval).ok())
-                            .execute(&mut update_tx)
-                            .await?;
-                    } else {
-                        let q = "INSERT INTO tag (event_id, \"name\", value, value_hex) VALUES ($1, $2, $3, NULL) ON CONFLICT DO NOTHING;";
-                        sqlx::query(q)
-                            .bind(&event_id)
-                            .bind(tagname)
-                            .bind(tagval.as_bytes())
-                            .execute(&mut update_tx)
-                            .await?;
-                    }
-                }
-            }
-            update_tx.commit().await?;
-            bar.finish();
-        }
-        info!("rebuilt tags in {:?}", start.elapsed());
-        Ok(())
-    }
-}
-
-mod m003 {
-    use crate::repo::postgres_migration::{Migration, SimpleSqlMigration};
-
-    pub const VERSION: i64 = 3;
-
-    pub fn migration() -> impl Migration {
-        SimpleSqlMigration {
-            serial_number: VERSION,
-            sql: vec![
-                r#"
-- Add unique constraint on tag
-ALTER TABLE tag ADD CONSTRAINT unique_constraint_name UNIQUE (event_id, "name", value, value_hex);
-        "#,
-            ],
-        }
-    }
-}
-
-mod m004 {
-    use crate::repo::postgres_migration::{Migration, SimpleSqlMigration};
-
-    pub const VERSION: i64 = 4;
-
-    pub fn migration() -> impl Migration {
-        SimpleSqlMigration {
-            serial_number: VERSION,
-            sql: vec![
-                r#"
-- Add expiration time for events
-ALTER TABLE event ADD COLUMN expires_at timestamp(0) with time zone;
-- Index expiration time
-CREATE INDEX event_expires_at_idx ON "event" (expires_at);
-        "#,
-            ],
-        }
-    }
-}
-
-mod m005 {
-    use crate::repo::postgres_migration::{Migration, SimpleSqlMigration};
-
-    pub const VERSION: i64 = 5;
-
-    pub fn migration() -> impl Migration {
-        SimpleSqlMigration {
-            serial_number: VERSION,
-            sql: vec![
-                r#"
-- Create account table
-CREATE TABLE "account" (
-    pubkey varchar NOT NULL,
-    is_admitted BOOLEAN NOT NULL DEFAULT FALSE,
-    balance BIGINT NOT NULL DEFAULT 0,
-    tos_accepted_at TIMESTAMP,
-    CONSTRAINT account_pkey PRIMARY KEY (pubkey)
-);
-
-CREATE TYPE status AS ENUM ('Paid', 'Unpaid', 'Expired');
-
-
-CREATE TABLE "invoice" (
-    payment_hash varchar NOT NULL,
-    pubkey varchar NOT NULL,
-    invoice varchar NOT NULL,
-    amount BIGINT NOT NULL,
-    status status NOT NULL DEFAULT 'Unpaid',
-    description varchar,
-    created_at timestamp,
-    confirmed_at timestamp,
-    CONSTRAINT invoice_payment_hash PRIMARY KEY (payment_hash),
-    CONSTRAINT invoice_pubkey_fkey FOREIGN KEY (pubkey) REFERENCES account (pubkey) ON DELETE CASCADE
-);
-        "#,
-            ],
-        }
-    }
-}
@@ -1,841 +0,0 @@
-//! Database schema and migrations
-use crate::db::PooledConnection;
-use crate::error::Result;
-use crate::event::{single_char_tagname, Event};
-use crate::utils::is_lower_hex;
-use const_format::formatcp;
-use indicatif::{ProgressBar, ProgressStyle};
-use rusqlite::limits::Limit;
-use rusqlite::params;
-use rusqlite::Connection;
-use std::cmp::Ordering;
-use std::time::Instant;
-use tracing::{debug, error, info};
-
-/// Startup DB Pragmas
-pub const STARTUP_SQL: &str = r##"
-PRAGMA main.synchronous = NORMAL;
-PRAGMA foreign_keys = ON;
-PRAGMA journal_size_limit = 32768;
-PRAGMA temp_store = 2; -- use memory, not temp files
-PRAGMA main.cache_size = 20000; -- 80MB max cache size per conn
-pragma mmap_size = 0; -- disable mmap (default)
-"##;
-
-/// Latest database version
-pub const DB_VERSION: usize = 18;
-
-/// Schema definition
-const INIT_SQL: &str = formatcp!(
-    r##"
-- Database settings
-PRAGMA encoding = "UTF-8";
-PRAGMA journal_mode = WAL;
-PRAGMA auto_vacuum = FULL;
-PRAGMA main.synchronous=NORMAL;
-PRAGMA foreign_keys = ON;
-PRAGMA application_id = 1654008667;
-PRAGMA user_version = {};
-
-- Event Table
-CREATE TABLE IF NOT EXISTS event (
-id INTEGER PRIMARY KEY,
-event_hash BLOB NOT NULL, -- 4-byte hash
-first_seen INTEGER NOT NULL, -- when the event was first seen (not authored!) (seconds since 1970)
-created_at INTEGER NOT NULL, -- when the event was authored
-expires_at INTEGER, -- when the event expires and may be deleted
-author BLOB NOT NULL, -- author pubkey
-delegated_by BLOB, -- delegator pubkey (NIP-26)
-kind INTEGER NOT NULL, -- event kind
-hidden INTEGER, -- relevant for queries
-content TEXT NOT NULL -- serialized json of event object
-);
-
-- Event Indexes
-CREATE UNIQUE INDEX IF NOT EXISTS event_hash_index ON event(event_hash);
-CREATE INDEX IF NOT EXISTS author_index ON event(author);
-CREATE INDEX IF NOT EXISTS kind_index ON event(kind);
-CREATE INDEX IF NOT EXISTS created_at_index ON event(created_at);
-CREATE INDEX IF NOT EXISTS delegated_by_index ON event(delegated_by);
-CREATE INDEX IF NOT EXISTS event_composite_index ON event(kind,created_at);
-CREATE INDEX IF NOT EXISTS kind_author_index ON event(kind,author);
-CREATE INDEX IF NOT EXISTS kind_created_at_index ON event(kind,created_at);
-CREATE INDEX IF NOT EXISTS author_created_at_index ON event(author,created_at);
-CREATE INDEX IF NOT EXISTS author_kind_index ON event(author,kind);
-CREATE INDEX IF NOT EXISTS event_expiration ON event(expires_at);
-
-- Tag Table
-- Tag values are stored as either a BLOB (if they come in as a
-- hex-string), or TEXT otherwise.
-- This means that searches need to select the appropriate column.
-- We duplicate the kind/created_at to make indexes much more efficient.
-CREATE TABLE IF NOT EXISTS tag (
-id INTEGER PRIMARY KEY,
-event_id INTEGER NOT NULL, -- an event ID that contains a tag.
-name TEXT, -- the tag name ("p", "e", whatever)
-value TEXT, -- the tag value, if not hex.
-value_hex BLOB, -- the tag value, if it can be interpreted as a lowercase hex string.
-created_at INTEGER NOT NULL, -- when the event was authored
-kind INTEGER NOT NULL, -- event kind
-FOREIGN KEY(event_id) REFERENCES event(id) ON UPDATE CASCADE ON DELETE CASCADE
-);
-CREATE INDEX IF NOT EXISTS tag_val_index ON tag(value);
-CREATE INDEX IF NOT EXISTS tag_composite_index ON tag(event_id,name,value);
-CREATE INDEX IF NOT EXISTS tag_name_eid_index ON tag(name,event_id,value);
-CREATE INDEX IF NOT EXISTS tag_covering_index ON tag(name,kind,value,created_at,event_id);
-
-- NIP-05 User Validation
-CREATE TABLE IF NOT EXISTS user_verification (
-id INTEGER PRIMARY KEY,
-metadata_event INTEGER NOT NULL, -- the metadata event used for this validation.
-name TEXT NOT NULL, -- the nip05 field value (user@domain).
-verified_at INTEGER, -- timestamp this author/nip05 was most recently verified.
-failed_at INTEGER, -- timestamp a verification attempt failed (host down).
-failure_count INTEGER DEFAULT 0, -- number of consecutive failures.
-FOREIGN KEY(metadata_event) REFERENCES event(id) ON UPDATE CASCADE ON DELETE CASCADE
-);
-CREATE INDEX IF NOT EXISTS user_verification_name_index ON user_verification(name);
-CREATE INDEX IF NOT EXISTS user_verification_event_index ON user_verification(metadata_event);
-
-- Create account table
-CREATE TABLE IF NOT EXISTS account (
-pubkey TEXT PRIMARY KEY,
-is_admitted INTEGER NOT NULL DEFAULT 0,
-balance INTEGER NOT NULL DEFAULT 0,
-tos_accepted_at INTEGER
-);
-
-- Create account index
-CREATE INDEX IF NOT EXISTS user_pubkey_index ON account(pubkey);
-
-- Invoice table
-CREATE TABLE IF NOT EXISTS invoice (
-payment_hash TEXT PRIMARY KEY,
-pubkey TEXT NOT NULL,
-invoice TEXT NOT NULL,
-amount INTEGER NOT NULL,
-status TEXT CHECK ( status IN ('Paid', 'Unpaid', 'Expired' ) ) NOT NUll DEFAULT 'Unpaid',
-description TEXT,
-created_at INTEGER NOT NULL,
-confirmed_at INTEGER,
-CONSTRAINT invoice_pubkey_fkey FOREIGN KEY (pubkey) REFERENCES account (pubkey) ON DELETE CASCADE
-);
-
-- Create invoice index
-CREATE INDEX IF NOT EXISTS invoice_pubkey_index ON invoice(pubkey);
-
-
-"##,
-    DB_VERSION
-);
-
-/// Determine the current application database schema version.
-pub fn curr_db_version(conn: &mut Connection) -> Result<usize> {
-    let query = "PRAGMA user_version;";
-    let curr_version = conn.query_row(query, [], |row| row.get(0))?;
-    Ok(curr_version)
-}
-
-/// Determine event count
-pub fn db_event_count(conn: &mut Connection) -> Result<usize> {
-    let query = "SELECT count(*) FROM event;";
-    let count = conn.query_row(query, [], |row| row.get(0))?;
-    Ok(count)
-}
-
-/// Determine tag count
-pub fn db_tag_count(conn: &mut Connection) -> Result<usize> {
-    let query = "SELECT count(*) FROM tag;";
-    let count = conn.query_row(query, [], |row| row.get(0))?;
-    Ok(count)
-}
-
-fn mig_init(conn: &mut PooledConnection) -> usize {
-    match conn.execute_batch(INIT_SQL) {
-        Ok(()) => {
-            info!(
-                "database pragma/schema initialized to v{}, and ready",
-                DB_VERSION
-            );
-        }
-        Err(err) => {
-            error!("update failed: {}", err);
-            panic!("database could not be initialized");
-        }
-    }
-    DB_VERSION
-}
-
-/// Upgrade DB to latest version, and execute pragma settings
-pub fn upgrade_db(conn: &mut PooledConnection) -> Result<usize> {
-    // check the version.
-    let mut curr_version = curr_db_version(conn)?;
-    info!("DB version = {:?}", curr_version);
-
-    debug!(
-        "SQLite max query parameters: {}",
-        conn.limit(Limit::SQLITE_LIMIT_VARIABLE_NUMBER)
-    );
-    debug!(
-        "SQLite max table/blob/text length: {} MB",
-        (f64::from(conn.limit(Limit::SQLITE_LIMIT_LENGTH)) / f64::from(1024 * 1024)).floor()
-    );
-    debug!(
-        "SQLite max SQL length: {} MB",
-        (f64::from(conn.limit(Limit::SQLITE_LIMIT_SQL_LENGTH)) / f64::from(1024 * 1024)).floor()
-    );
-
-    match curr_version.cmp(&DB_VERSION) {
-        // Database is new or not current
-        Ordering::Less => {
-            // initialize from scratch
-            if curr_version == 0 {
-                curr_version = mig_init(conn);
-            }
-            // for initialized but out-of-date schemas, proceed to
-            // upgrade sequentially until we are current.
-            if curr_version == 1 {
-                curr_version = mig_1_to_2(conn)?;
-            }
-            if curr_version == 2 {
-                curr_version = mig_2_to_3(conn)?;
-            }
-            if curr_version == 3 {
-                curr_version = mig_3_to_4(conn)?;
-            }
-            if curr_version == 4 {
-                curr_version = mig_4_to_5(conn)?;
-            }
-            if curr_version == 5 {
-                curr_version = mig_5_to_6(conn)?;
-            }
-            if curr_version == 6 {
-                curr_version = mig_6_to_7(conn)?;
-            }
-            if curr_version == 7 {
-                curr_version = mig_7_to_8(conn)?;
-            }
-            if curr_version == 8 {
-                curr_version = mig_8_to_9(conn)?;
-            }
-            if curr_version == 9 {
-                curr_version = mig_9_to_10(conn)?;
-            }
-            if curr_version == 10 {
-                curr_version = mig_10_to_11(conn)?;
-            }
-            if curr_version == 11 {
-                curr_version = mig_11_to_12(conn)?;
-            }
-            if curr_version == 12 {
-                curr_version = mig_12_to_13(conn)?;
-            }
-            if curr_version == 13 {
-                curr_version = mig_13_to_14(conn)?;
-            }
-            if curr_version == 14 {
-                curr_version = mig_14_to_15(conn)?;
-            }
-            if curr_version == 15 {
-                curr_version = mig_15_to_16(conn)?;
-            }
-            if curr_version == 16 {
-                curr_version = mig_16_to_17(conn)?;
-            }
-            if curr_version == 17 {
-                curr_version = mig_17_to_18(conn)?;
-            }
-
-            if curr_version == DB_VERSION {
-                info!(
-                    "All migration scripts completed successfully.  Welcome to v{}.",
-                    DB_VERSION
-                );
-            }
-        }
-        // Database is current, all is good
-        Ordering::Equal => {
-            debug!("Database version was already current (v{DB_VERSION})");
-        }
-        // Database is newer than what this code understands, abort
-        Ordering::Greater => {
-            panic!(
-                "Database version is newer than supported by this executable (v{curr_version} > v{DB_VERSION})",
-            );
-        }
-    }
-
-    // Setup PRAGMA
-    conn.execute_batch(STARTUP_SQL)?;
-    debug!("SQLite PRAGMA startup completed");
-    Ok(DB_VERSION)
-}
-
-pub fn rebuild_tags(conn: &mut PooledConnection) -> Result<()> {
-    // Check how many events we have to process
-    let count = db_event_count(conn)?;
-    let update_each_percent = 0.05;
-    let mut percent_done = 0.0;
-    let mut events_processed = 0;
-    let start = Instant::now();
-    let tx = conn.transaction()?;
-    {
-        // Clear out table
-        tx.execute("DELETE FROM tag;", [])?;
-        let mut stmt = tx.prepare("select id, content from event order by id;")?;
-        let mut tag_rows = stmt.query([])?;
-        while let Some(row) = tag_rows.next()? {
-            if (events_processed as f32) / (count as f32) > percent_done {
-                info!("Tag update {}% complete...", (100.0 * percent_done).round());
-                percent_done += update_each_percent;
-            }
-            // we want to capture the event_id that had the tag, the tag name, and the tag hex value.
-            let event_id: u64 = row.get(0)?;
-            let event_json: String = row.get(1)?;
-            let event: Event = serde_json::from_str(&event_json)?;
-            // look at each event, and each tag, creating new tag entries if appropriate.
-            for t in event.tags.iter().filter(|x| x.len() > 1) {
-                let tagname = t.get(0).unwrap();
-                let tagnamechar_opt = single_char_tagname(tagname);
-                if tagnamechar_opt.is_none() {
-                    continue;
-                }
-                // safe because len was > 1
-                let tagval = t.get(1).unwrap();
-                // insert as BLOB if we can restore it losslessly.
-                // this means it needs to be even length and lowercase.
-                if (tagval.len() % 2 == 0) && is_lower_hex(tagval) {
-                    tx.execute(
-                        "INSERT INTO tag (event_id, name, value_hex) VALUES (?1, ?2, ?3);",
-                        params![event_id, tagname, hex::decode(tagval).ok()],
-                    )?;
-                } else {
-                    // otherwise, insert as text
-                    tx.execute(
-                        "INSERT INTO tag (event_id, name, value) VALUES (?1, ?2, ?3);",
-                        params![event_id, tagname, &tagval],
-                    )?;
-                }
-            }
-            events_processed += 1;
-        }
-    }
-    tx.commit()?;
-    info!("rebuilt tags in {:?}", start.elapsed());
-    Ok(())
-}
-
-//// Migration Scripts
-
-fn mig_1_to_2(conn: &mut PooledConnection) -> Result<usize> {
-    // only change is adding a hidden column to events.
-    let upgrade_sql = r##"
-ALTER TABLE event ADD hidden INTEGER;
-UPDATE event SET hidden=FALSE;
-PRAGMA user_version = 2;
-"##;
-    match conn.execute_batch(upgrade_sql) {
-        Ok(()) => {
-            info!("database schema upgraded v1 -> v2");
-        }
-        Err(err) => {
-            error!("update failed: {}", err);
-            panic!("database could not be upgraded");
-        }
-    }
-    Ok(2)
-}
-
-fn mig_2_to_3(conn: &mut PooledConnection) -> Result<usize> {
-    // this version lacks the tag column
-    info!("database schema needs update from 2->3");
-    let upgrade_sql = r##"
-CREATE TABLE IF NOT EXISTS tag (
-id INTEGER PRIMARY KEY,
-event_id INTEGER NOT NULL, -- an event ID that contains a tag.
-name TEXT, -- the tag name ("p", "e", whatever)
-value TEXT, -- the tag value, if not hex.
-value_hex BLOB, -- the tag value, if it can be interpreted as a hex string.
-FOREIGN KEY(event_id) REFERENCES event(id) ON UPDATE CASCADE ON DELETE CASCADE
-);
-PRAGMA user_version = 3;
-"##;
-    // TODO: load existing refs into tag table
-    match conn.execute_batch(upgrade_sql) {
-        Ok(()) => {
-            info!("database schema upgraded v2 -> v3");
-        }
-        Err(err) => {
-            error!("update failed: {}", err);
-            panic!("database could not be upgraded");
-        }
-    }
-    // iterate over every event/pubkey tag
-    let tx = conn.transaction()?;
-    {
-        let mut stmt = tx.prepare("select event_id, \"e\", lower(hex(referenced_event)) from event_ref union select event_id, \"p\", lower(hex(referenced_pubkey)) from pubkey_ref;")?;
-        let mut tag_rows = stmt.query([])?;
-        while let Some(row) = tag_rows.next()? {
-            // we want to capture the event_id that had the tag, the tag name, and the tag hex value.
-            let event_id: u64 = row.get(0)?;
-            let tag_name: String = row.get(1)?;
-            let tag_value: String = row.get(2)?;
-            // this will leave behind p/e tags that were non-hex, but they are invalid anyways.
-            if is_lower_hex(&tag_value) {
-                tx.execute(
-                    "INSERT INTO tag (event_id, name, value_hex) VALUES (?1, ?2, ?3);",
-                    params![event_id, tag_name, hex::decode(&tag_value).ok()],
-                )?;
-            }
-        }
-    }
-    info!("Updated tag values");
-    tx.commit()?;
-    Ok(3)
-}
-
-fn mig_3_to_4(conn: &mut PooledConnection) -> Result<usize> {
-    info!("database schema needs update from 3->4");
-    let upgrade_sql = r##"
-- incoming metadata events with nip05
-CREATE TABLE IF NOT EXISTS user_verification (
-id INTEGER PRIMARY KEY,
-metadata_event INTEGER NOT NULL, -- the metadata event used for this validation.
-name TEXT NOT NULL, -- the nip05 field value (user@domain).
-verified_at INTEGER, -- timestamp this author/nip05 was most recently verified.
-failed_at INTEGER, -- timestamp a verification attempt failed (host down).
-failure_count INTEGER DEFAULT 0, -- number of consecutive failures.
-FOREIGN KEY(metadata_event) REFERENCES event(id) ON UPDATE CASCADE ON DELETE CASCADE
-);
-CREATE INDEX IF NOT EXISTS user_verification_name_index ON user_verification(name);
-CREATE INDEX IF NOT EXISTS user_verification_event_index ON user_verification(metadata_event);
-PRAGMA user_version = 4;
-"##;
-    match conn.execute_batch(upgrade_sql) {
-        Ok(()) => {
-            info!("database schema upgraded v3 -> v4");
-        }
-        Err(err) => {
-            error!("update failed: {}", err);
-            panic!("database could not be upgraded");
-        }
-    }
-    Ok(4)
-}
-
-fn mig_4_to_5(conn: &mut PooledConnection) -> Result<usize> {
-    info!("database schema needs update from 4->5");
-    let upgrade_sql = r##"
-DROP TABLE IF EXISTS event_ref;
-DROP TABLE IF EXISTS pubkey_ref;
-PRAGMA user_version=5;
-"##;
-    match conn.execute_batch(upgrade_sql) {
-        Ok(()) => {
-            info!("database schema upgraded v4 -> v5");
-        }
-        Err(err) => {
-            error!("update failed: {}", err);
-            panic!("database could not be upgraded");
-        }
-    }
-    Ok(5)
-}
-
-fn mig_5_to_6(conn: &mut PooledConnection) -> Result<usize> {
-    info!("database schema needs update from 5->6");
-    // We need to rebuild the tags table.  iterate through the
-    // event table.  build event from json, insert tags into a
-    // fresh tag table.  This was needed due to a logic error in
-    // how hex-like tags got indexed.
-    let start = Instant::now();
-    let tx = conn.transaction()?;
-    {
-        // Clear out table
-        tx.execute("DELETE FROM tag;", [])?;
-        let mut stmt = tx.prepare("select id, content from event order by id;")?;
-        let mut tag_rows = stmt.query([])?;
-        while let Some(row) = tag_rows.next()? {
-            let event_id: u64 = row.get(0)?;
-            let event_json: String = row.get(1)?;
-            let event: Event = serde_json::from_str(&event_json)?;
-            // look at each event, and each tag, creating new tag entries if appropriate.
-            for t in event.tags.iter().filter(|x| x.len() > 1) {
-                let tagname = t.get(0).unwrap();
-                let tagnamechar_opt = single_char_tagname(tagname);
-                if tagnamechar_opt.is_none() {
-                    continue;
-                }
-                // safe because len was > 1
-                let tagval = t.get(1).unwrap();
-                // insert as BLOB if we can restore it losslessly.
-                // this means it needs to be even length and lowercase.
-                if (tagval.len() % 2 == 0) && is_lower_hex(tagval) {
-                    tx.execute(
-                        "INSERT INTO tag (event_id, name, value_hex) VALUES (?1, ?2, ?3);",
-                        params![event_id, tagname, hex::decode(tagval).ok()],
-                    )?;
-                } else {
-                    // otherwise, insert as text
-                    tx.execute(
-                        "INSERT INTO tag (event_id, name, value) VALUES (?1, ?2, ?3);",
-                        params![event_id, tagname, &tagval],
-                    )?;
-                }
-            }
-        }
-        tx.execute("PRAGMA user_version = 6;", [])?;
-    }
-    tx.commit()?;
-    info!("database schema upgraded v5 -> v6 in {:?}", start.elapsed());
-    // vacuum after large table modification
-    let start = Instant::now();
-    conn.execute("VACUUM;", [])?;
-    info!("vacuumed DB after tags rebuild in {:?}", start.elapsed());
-    Ok(6)
-}
-
-fn mig_6_to_7(conn: &mut PooledConnection) -> Result<usize> {
-    info!("database schema needs update from 6->7");
-    let upgrade_sql = r##"
-ALTER TABLE event ADD delegated_by BLOB;
-CREATE INDEX IF NOT EXISTS delegated_by_index ON event(delegated_by);
-PRAGMA user_version = 7;
-"##;
-    match conn.execute_batch(upgrade_sql) {
-        Ok(()) => {
-            info!("database schema upgraded v6 -> v7");
-        }
-        Err(err) => {
-            error!("update failed: {}", err);
-            panic!("database could not be upgraded");
-        }
-    }
-    Ok(7)
-}
-
-fn mig_7_to_8(conn: &mut PooledConnection) -> Result<usize> {
-    info!("database schema needs update from 7->8");
-    // Remove redundant indexes, and add a better multi-column index.
-    let upgrade_sql = r##"
-DROP INDEX IF EXISTS created_at_index;
-DROP INDEX IF EXISTS kind_index;
-CREATE INDEX IF NOT EXISTS event_composite_index ON event(kind,created_at);
-PRAGMA user_version = 8;
-"##;
-    match conn.execute_batch(upgrade_sql) {
-        Ok(()) => {
-            info!("database schema upgraded v7 -> v8");
-        }
-        Err(err) => {
-            error!("update failed: {}", err);
-            panic!("database could not be upgraded");
-        }
-    }
-    Ok(8)
-}
-
-fn mig_8_to_9(conn: &mut PooledConnection) -> Result<usize> {
-    info!("database schema needs update from 8->9");
-    // Those old indexes were actually helpful...
-    let upgrade_sql = r##"
-CREATE INDEX IF NOT EXISTS created_at_index ON event(created_at);
-CREATE INDEX IF NOT EXISTS event_composite_index ON event(kind,created_at);
-PRAGMA user_version = 9;
-"##;
-    match conn.execute_batch(upgrade_sql) {
-        Ok(()) => {
-            info!("database schema upgraded v8 -> v9");
-        }
-        Err(err) => {
-            error!("update failed: {}", err);
-            panic!("database could not be upgraded");
-        }
-    }
-    Ok(9)
-}
-
-fn mig_9_to_10(conn: &mut PooledConnection) -> Result<usize> {
-    info!("database schema needs update from 9->10");
-    // Those old indexes were actually helpful...
-    let upgrade_sql = r##"
-CREATE INDEX IF NOT EXISTS tag_composite_index ON tag(event_id,name,value_hex,value);
-PRAGMA user_version = 10;
-"##;
-    match conn.execute_batch(upgrade_sql) {
-        Ok(()) => {
-            info!("database schema upgraded v9 -> v10");
-        }
-        Err(err) => {
-            error!("update failed: {}", err);
-            panic!("database could not be upgraded");
-        }
-    }
-    Ok(10)
-}
-
-fn mig_10_to_11(conn: &mut PooledConnection) -> Result<usize> {
-    info!("database schema needs update from 10->11");
-    // Those old indexes were actually helpful...
-    let upgrade_sql = r##"
-CREATE INDEX IF NOT EXISTS tag_name_eid_index ON tag(name,event_id,value_hex);
-reindex;
-pragma optimize;
-PRAGMA user_version = 11;
-"##;
-    match conn.execute_batch(upgrade_sql) {
-        Ok(()) => {
-            info!("database schema upgraded v10 -> v11");
-        }
-        Err(err) => {
-            error!("update failed: {}", err);
-            panic!("database could not be upgraded");
-        }
-    }
-    Ok(11)
-}
-
-fn mig_11_to_12(conn: &mut PooledConnection) -> Result<usize> {
-    info!("database schema needs update from 11->12");
-    let start = Instant::now();
-    let tx = conn.transaction()?;
-    {
-        // Lookup every replaceable event
-        let mut stmt = tx.prepare("select kind,author from event where kind in (0,3,41) or (kind>=10000 and kind<20000) order by id;")?;
-        let mut replaceable_rows = stmt.query([])?;
-        info!("updating replaceable events; this could take awhile...");
-        while let Some(row) = replaceable_rows.next()? {
-            // we want to capture the event_id that had the tag, the tag name, and the tag hex value.
-            let event_kind: u64 = row.get(0)?;
-            let event_author: Vec<u8> = row.get(1)?;
-            tx.execute(
-                "UPDATE event SET hidden=TRUE WHERE hidden!=TRUE and kind=? and author=? and id NOT IN (SELECT id FROM event WHERE kind=? AND author=? ORDER BY created_at DESC LIMIT 1)",
-                params![event_kind, event_author, event_kind, event_author],
-            )?;
-        }
-        tx.execute("PRAGMA user_version = 12;", [])?;
-    }
-    tx.commit()?;
-    info!(
-        "database schema upgraded v11 -> v12 in {:?}",
-        start.elapsed()
-    );
-    // vacuum after large table modification
-    let start = Instant::now();
-    conn.execute("VACUUM;", [])?;
-    info!(
-        "vacuumed DB after hidden event cleanup in {:?}",
-        start.elapsed()
-    );
-    Ok(12)
-}
-
-fn mig_12_to_13(conn: &mut PooledConnection) -> Result<usize> {
-    info!("database schema needs update from 12->13");
-    let upgrade_sql = r##"
-CREATE INDEX IF NOT EXISTS kind_author_index ON event(kind,author);
-reindex;
-pragma optimize;
-PRAGMA user_version = 13;
-"##;
-    match conn.execute_batch(upgrade_sql) {
-        Ok(()) => {
-            info!("database schema upgraded v12 -> v13");
-        }
-        Err(err) => {
-            error!("update failed: {}", err);
-            panic!("database could not be upgraded");
-        }
-    }
-    Ok(13)
-}
-
-fn mig_13_to_14(conn: &mut PooledConnection) -> Result<usize> {
-    info!("database schema needs update from 13->14");
-    let upgrade_sql = r##"
-CREATE INDEX IF NOT EXISTS kind_index ON event(kind);
-CREATE INDEX IF NOT EXISTS kind_created_at_index ON event(kind,created_at);
-pragma optimize;
-PRAGMA user_version = 14;
-"##;
-    match conn.execute_batch(upgrade_sql) {
-        Ok(()) => {
-            info!("database schema upgraded v13 -> v14");
-        }
-        Err(err) => {
-            error!("update failed: {}", err);
-            panic!("database could not be upgraded");
-        }
-    }
-    Ok(14)
-}
-
-fn mig_14_to_15(conn: &mut PooledConnection) -> Result<usize> {
-    info!("database schema needs update from 14->15");
-    let upgrade_sql = r##"
-CREATE INDEX IF NOT EXISTS author_created_at_index ON event(author,created_at);
-CREATE INDEX IF NOT EXISTS author_kind_index ON event(author,kind);
-PRAGMA user_version = 15;
-"##;
-    match conn.execute_batch(upgrade_sql) {
-        Ok(()) => {
-            info!("database schema upgraded v14 -> v15");
-        }
-        Err(err) => {
-            error!("update failed: {}", err);
-            panic!("database could not be upgraded");
-        }
-    }
-    // clear out hidden events
-    let clear_hidden_sql = r##"DELETE FROM event WHERE HIDDEN=true;"##;
-    info!("removing hidden events; this may take awhile...");
-    match conn.execute_batch(clear_hidden_sql) {
-        Ok(()) => {
-            info!("all hidden events removed");
-        }
-        Err(err) => {
-            error!("delete failed: {}", err);
-            panic!("could not remove hidden events");
-        }
-    }
-    Ok(15)
-}
-
-fn mig_15_to_16(conn: &mut PooledConnection) -> Result<usize> {
-    let count = db_event_count(conn)?;
-    info!("database schema needs update from 15->16 (this may take a few minutes)");
-    let upgrade_sql = r##"
-DROP TABLE tag;
-CREATE TABLE tag (
-id INTEGER PRIMARY KEY,
-event_id INTEGER NOT NULL, -- an event ID that contains a tag.
-name TEXT, -- the tag name ("p", "e", whatever)
-value TEXT, -- the tag value, if not hex.
-created_at INTEGER NOT NULL, -- when the event was authored
-kind INTEGER NOT NULL, -- event kind
-FOREIGN KEY(event_id) REFERENCES event(id) ON UPDATE CASCADE ON DELETE CASCADE
-);
-CREATE INDEX IF NOT EXISTS tag_val_index ON tag(value);
-CREATE INDEX IF NOT EXISTS tag_composite_index ON tag(event_id,name,value);
-CREATE INDEX IF NOT EXISTS tag_name_eid_index ON tag(name,event_id,value);
-CREATE INDEX IF NOT EXISTS tag_covering_index ON tag(name,kind,value,created_at,event_id);
-"##;
-
-    let start = Instant::now();
-    let tx = conn.transaction()?;
-
-    let bar = ProgressBar::new(count.try_into().unwrap()).with_message("rebuilding tags table");
-    bar.set_style(
-        ProgressStyle::with_template(
-            "[{elapsed_precise}] {bar:40.white/blue} {pos:>7}/{len:7} [{percent}%] {msg}",
-        )
-        .unwrap(),
-    );
-    {
-        tx.execute_batch(upgrade_sql)?;
-        let mut stmt =
-            tx.prepare("select id, kind, created_at, content from event order by id;")?;
-        let mut tag_rows = stmt.query([])?;
-        let mut count = 0;
-        while let Some(row) = tag_rows.next()? {
-            count += 1;
-            if count % 10 == 0 {
-                bar.inc(10);
-            }
-            let event_id: u64 = row.get(0)?;
-            let kind: u64 = row.get(1)?;
-            let created_at: u64 = row.get(2)?;
-            let event_json: String = row.get(3)?;
-            let event: Event = serde_json::from_str(&event_json)?;
-            // look at each event, and each tag, creating new tag entries if appropriate.
-            for t in event.tags.iter().filter(|x| x.len() > 1) {
-                let tagname = t.get(0).unwrap();
-                let tagnamechar_opt = single_char_tagname(tagname);
-                if tagnamechar_opt.is_none() {
-                    continue;
-                }
-                // safe because len was > 1
-                let tagval = t.get(1).unwrap();
-                // otherwise, insert as text
-                tx.execute(
-                    "INSERT INTO tag (event_id, name, value, kind, created_at) VALUES (?1, ?2, ?3, ?4, ?5);",
-                    params![event_id, tagname, &tagval, kind, created_at],
-                )?;
-            }
-        }
-        tx.execute("PRAGMA user_version = 16;", [])?;
-    }
-    bar.finish();
-    tx.commit()?;
-    info!(
-        "database schema upgraded v15 -> v16 in {:?}",
-        start.elapsed()
-    );
-    Ok(16)
-}
-
-fn mig_16_to_17(conn: &mut PooledConnection) -> Result<usize> {
-    info!("database schema needs update from 16->17");
-    let upgrade_sql = r##"
-ALTER TABLE event ADD COLUMN expires_at INTEGER;
-CREATE INDEX IF NOT EXISTS event_expiration ON event(expires_at);
-PRAGMA user_version = 17;
-"##;
-    match conn.execute_batch(upgrade_sql) {
-        Ok(()) => {
-            info!("database schema upgraded v16 -> v17");
-        }
-        Err(err) => {
-            error!("update failed: {}", err);
-            panic!("database could not be upgraded");
-        }
-    }
-    Ok(17)
-}
-
-fn mig_17_to_18(conn: &mut PooledConnection) -> Result<usize> {
-    info!("database schema needs update from 17->18");
-    let upgrade_sql = r##"
-- Create invoices table
-CREATE TABLE IF NOT EXISTS invoice (
-payment_hash TEXT PRIMARY KEY,
-pubkey TEXT NOT NULL,
-invoice TEXT NOT NULL,
-amount INTEGER NOT NULL,
-status TEXT CHECK ( status IN ('Paid', 'Unpaid', 'Expired' ) ) NOT NUll DEFAULT 'Unpaid',
-description TEXT,
-created_at INTEGER NOT NULL,
-confirmed_at INTEGER,
-CONSTRAINT invoice_pubkey_fkey FOREIGN KEY (pubkey) REFERENCES account (pubkey) ON DELETE CASCADE
-);
-
-- Create invoice index
-CREATE INDEX IF NOT EXISTS invoice_pubkey_index ON invoice(pubkey);
-
-- Create account table
-
-CREATE TABLE IF NOT EXISTS account (
-pubkey TEXT PRIMARY KEY,
-is_admitted INTEGER NOT NULL DEFAULT 0,
-balance INTEGER NOT NULL DEFAULT 0,
-tos_accepted_at INTEGER
-);
-
-- Create account index
-CREATE INDEX IF NOT EXISTS account_pubkey_index ON account(pubkey);
-
-
-pragma optimize;
-PRAGMA user_version = 18;
-"##;
-    match conn.execute_batch(upgrade_sql) {
-        Ok(()) => {
-            info!("database schema upgraded v17 -> v18");
-        }
-        Err(err) => {
-            error!("update failed: {}", err);
-            panic!("database could not be upgraded");
-        }
-    }
-    Ok(18)
-}
@@ -0,0 +1,375 @@
+//! Database schema and migrations
+use crate::db::PooledConnection;
+use crate::error::Result;
+use crate::event::{single_char_tagname, Event};
+use crate::utils::is_lower_hex;
+use const_format::formatcp;
+use rusqlite::limits::Limit;
+use rusqlite::params;
+use rusqlite::Connection;
+use std::cmp::Ordering;
+use std::time::Instant;
+use tracing::{debug, error, info};
+
+/// Startup DB Pragmas
+pub const STARTUP_SQL: &str = r##"
+PRAGMA main.synchronous=NORMAL;
+PRAGMA foreign_keys = ON;
+PRAGMA journal_size_limit=32768;
+pragma mmap_size = 536870912; -- 512MB of mmap
+"##;
+
+/// Latest database version
+pub const DB_VERSION: usize = 7;
+
+/// Schema definition
+const INIT_SQL: &str = formatcp!(
+    r##"
+-- Database settings
+PRAGMA encoding = "UTF-8";
+PRAGMA journal_mode=WAL;
+PRAGMA main.synchronous=NORMAL;
+PRAGMA foreign_keys = ON;
+PRAGMA application_id = 1654008667;
+PRAGMA user_version = {};
+
+-- Event Table
+CREATE TABLE IF NOT EXISTS event (
+id INTEGER PRIMARY KEY,
+event_hash BLOB NOT NULL, -- 4-byte hash
+first_seen INTEGER NOT NULL, -- when the event was first seen (not authored!) (seconds since 1970)
+created_at INTEGER NOT NULL, -- when the event was authored
+author BLOB NOT NULL, -- author pubkey
+delegated_by BLOB, -- delegator pubkey (NIP-26)
+kind INTEGER NOT NULL, -- event kind
+hidden INTEGER, -- relevant for queries
+content TEXT NOT NULL -- serialized json of event object
+);
+
+-- Event Indexes
+CREATE UNIQUE INDEX IF NOT EXISTS event_hash_index ON event(event_hash);
+CREATE INDEX IF NOT EXISTS created_at_index ON event(created_at);
+CREATE INDEX IF NOT EXISTS author_index ON event(author);
+CREATE INDEX IF NOT EXISTS delegated_by_index ON event(delegated_by);
+CREATE INDEX IF NOT EXISTS kind_index ON event(kind);
+
+-- Tag Table
+-- Tag values are stored as either a BLOB (if they come in as a
+-- hex-string), or TEXT otherwise.
+-- This means that searches need to select the appropriate column.
+CREATE TABLE IF NOT EXISTS tag (
+id INTEGER PRIMARY KEY,
+event_id INTEGER NOT NULL, -- an event ID that contains a tag.
+name TEXT, -- the tag name ("p", "e", whatever)
+value TEXT, -- the tag value, if not hex.
+value_hex BLOB, -- the tag value, if it can be interpreted as a lowercase hex string.
+FOREIGN KEY(event_id) REFERENCES event(id) ON UPDATE CASCADE ON DELETE CASCADE
+);
+CREATE INDEX IF NOT EXISTS tag_val_index ON tag(value);
+CREATE INDEX IF NOT EXISTS tag_val_hex_index ON tag(value_hex);
+
+-- NIP-05 User Validation
+CREATE TABLE IF NOT EXISTS user_verification (
+id INTEGER PRIMARY KEY,
+metadata_event INTEGER NOT NULL, -- the metadata event used for this validation.
+name TEXT NOT NULL, -- the nip05 field value (user@domain).
+verified_at INTEGER, -- timestamp this author/nip05 was most recently verified.
+failed_at INTEGER, -- timestamp a verification attempt failed (host down).
+failure_count INTEGER DEFAULT 0, -- number of consecutive failures.
+FOREIGN KEY(metadata_event) REFERENCES event(id) ON UPDATE CASCADE ON DELETE CASCADE
+);
+CREATE INDEX IF NOT EXISTS user_verification_name_index ON user_verification(name);
+CREATE INDEX IF NOT EXISTS user_verification_event_index ON user_verification(metadata_event);
+"##,
+    DB_VERSION
+);
+
+/// Determine the current application database schema version.
+pub fn curr_db_version(conn: &mut Connection) -> Result<usize> {
+    let query = "PRAGMA user_version;";
+    let curr_version = conn.query_row(query, [], |row| row.get(0))?;
+    Ok(curr_version)
+}
+
+fn mig_init(conn: &mut PooledConnection) -> Result<usize> {
+    match conn.execute_batch(INIT_SQL) {
+        Ok(()) => {
+            info!(
+                "database pragma/schema initialized to v{}, and ready",
+                DB_VERSION
+            );
+        }
+        Err(err) => {
+            error!("update failed: {}", err);
+            panic!("database could not be initialized");
+        }
+    }
+    Ok(DB_VERSION)
+}
+
+/// Upgrade DB to latest version, and execute pragma settings
+pub fn upgrade_db(conn: &mut PooledConnection) -> Result<()> {
+    // check the version.
+    let mut curr_version = curr_db_version(conn)?;
+    info!("DB version = {:?}", curr_version);
+
+    debug!(
+        "SQLite max query parameters: {}",
+        conn.limit(Limit::SQLITE_LIMIT_VARIABLE_NUMBER)
+    );
+    debug!(
+        "SQLite max table/blob/text length: {} MB",
+        (conn.limit(Limit::SQLITE_LIMIT_LENGTH) as f64 / (1024 * 1024) as f64).floor()
+    );
+    debug!(
+        "SQLite max SQL length: {} MB",
+        (conn.limit(Limit::SQLITE_LIMIT_SQL_LENGTH) as f64 / (1024 * 1024) as f64).floor()
+    );
+
+    match curr_version.cmp(&DB_VERSION) {
+        // Database is new or not current
+        Ordering::Less => {
+            // initialize from scratch
+            if curr_version == 0 {
+                curr_version = mig_init(conn)?;
+            }
+            // for initialized but out-of-date schemas, proceed to
+            // upgrade sequentially until we are current.
+            if curr_version == 1 {
+                curr_version = mig_1_to_2(conn)?;
+            }
+
+            if curr_version == 2 {
+                curr_version = mig_2_to_3(conn)?;
+            }
+
+            if curr_version == 3 {
+                curr_version = mig_3_to_4(conn)?;
+            }
+
+            if curr_version == 4 {
+                curr_version = mig_4_to_5(conn)?;
+            }
+
+            if curr_version == 5 {
+                curr_version = mig_5_to_6(conn)?;
+            }
+            if curr_version == 6 {
+                curr_version = mig_6_to_7(conn)?;
+            }
+            if curr_version == DB_VERSION {
+                info!(
+                    "All migration scripts completed successfully.  Welcome to v{}.",
+                    DB_VERSION
+                );
+            }
+        }
+        // Database is current, all is good
+        Ordering::Equal => {
+            debug!("Database version was already current (v{})", DB_VERSION);
+        }
+        // Database is newer than what this code understands, abort
+        Ordering::Greater => {
+            panic!(
+                "Database version is newer than supported by this executable (v{} > v{})",
+                curr_version, DB_VERSION
+            );
+        }
+    }
+
+    // Setup PRAGMA
+    conn.execute_batch(STARTUP_SQL)?;
+    debug!("SQLite PRAGMA startup completed");
+    Ok(())
+}
+
+//// Migration Scripts
+
+fn mig_1_to_2(conn: &mut PooledConnection) -> Result<usize> {
+    // only change is adding a hidden column to events.
+    let upgrade_sql = r##"
+ALTER TABLE event ADD hidden INTEGER;
+UPDATE event SET hidden=FALSE;
+PRAGMA user_version = 2;
+"##;
+    match conn.execute_batch(upgrade_sql) {
+        Ok(()) => {
+            info!("database schema upgraded v1 -> v2");
+        }
+        Err(err) => {
+            error!("update failed: {}", err);
+            panic!("database could not be upgraded");
+        }
+    }
+    Ok(2)
+}
+
+fn mig_2_to_3(conn: &mut PooledConnection) -> Result<usize> {
+    // this version lacks the tag column
+    info!("database schema needs update from 2->3");
+    let upgrade_sql = r##"
+CREATE TABLE IF NOT EXISTS tag (
+id INTEGER PRIMARY KEY,
+event_id INTEGER NOT NULL, -- an event ID that contains a tag.
+name TEXT, -- the tag name ("p", "e", whatever)
+value TEXT, -- the tag value, if not hex.
+value_hex BLOB, -- the tag value, if it can be interpreted as a hex string.
+FOREIGN KEY(event_id) REFERENCES event(id) ON UPDATE CASCADE ON DELETE CASCADE
+);
+PRAGMA user_version = 3;
+"##;
+    // TODO: load existing refs into tag table
+    match conn.execute_batch(upgrade_sql) {
+        Ok(()) => {
+            info!("database schema upgraded v2 -> v3");
+        }
+        Err(err) => {
+            error!("update failed: {}", err);
+            panic!("database could not be upgraded");
+        }
+    }
+    // iterate over every event/pubkey tag
+    let tx = conn.transaction()?;
+    {
+        let mut stmt = tx.prepare("select event_id, \"e\", lower(hex(referenced_event)) from event_ref union select event_id, \"p\", lower(hex(referenced_pubkey)) from pubkey_ref;")?;
+        let mut tag_rows = stmt.query([])?;
+        while let Some(row) = tag_rows.next()? {
+            // we want to capture the event_id that had the tag, the tag name, and the tag hex value.
+            let event_id: u64 = row.get(0)?;
+            let tag_name: String = row.get(1)?;
+            let tag_value: String = row.get(2)?;
+            // this will leave behind p/e tags that were non-hex, but they are invalid anyways.
+            if is_lower_hex(&tag_value) {
+                tx.execute(
+                    "INSERT INTO tag (event_id, name, value_hex) VALUES (?1, ?2, ?3);",
+                    params![event_id, tag_name, hex::decode(&tag_value).ok()],
+                )?;
+            }
+        }
+    }
+    info!("Updated tag values");
+    tx.commit()?;
+    Ok(3)
+}
+
+fn mig_3_to_4(conn: &mut PooledConnection) -> Result<usize> {
+    info!("database schema needs update from 3->4");
+    let upgrade_sql = r##"
+-- incoming metadata events with nip05
+CREATE TABLE IF NOT EXISTS user_verification (
+id INTEGER PRIMARY KEY,
+metadata_event INTEGER NOT NULL, -- the metadata event used for this validation.
+name TEXT NOT NULL, -- the nip05 field value (user@domain).
+verified_at INTEGER, -- timestamp this author/nip05 was most recently verified.
+failed_at INTEGER, -- timestamp a verification attempt failed (host down).
+failure_count INTEGER DEFAULT 0, -- number of consecutive failures.
+FOREIGN KEY(metadata_event) REFERENCES event(id) ON UPDATE CASCADE ON DELETE CASCADE
+);
+CREATE INDEX IF NOT EXISTS user_verification_name_index ON user_verification(name);
+CREATE INDEX IF NOT EXISTS user_verification_event_index ON user_verification(metadata_event);
+PRAGMA user_version = 4;
+"##;
+    match conn.execute_batch(upgrade_sql) {
+        Ok(()) => {
+            info!("database schema upgraded v3 -> v4");
+        }
+        Err(err) => {
+            error!("update failed: {}", err);
+            panic!("database could not be upgraded");
+        }
+    }
+    Ok(4)
+}
+
+fn mig_4_to_5(conn: &mut PooledConnection) -> Result<usize> {
+    info!("database schema needs update from 4->5");
+    let upgrade_sql = r##"
+DROP TABLE IF EXISTS event_ref;
+DROP TABLE IF EXISTS pubkey_ref;
+PRAGMA user_version=5;
+"##;
+    match conn.execute_batch(upgrade_sql) {
+        Ok(()) => {
+            info!("database schema upgraded v4 -> v5");
+        }
+        Err(err) => {
+            error!("update failed: {}", err);
+            panic!("database could not be upgraded");
+        }
+    }
+    Ok(5)
+}
+
+fn mig_5_to_6(conn: &mut PooledConnection) -> Result<usize> {
+    info!("database schema needs update from 5->6");
+    // We need to rebuild the tags table.  iterate through the
+    // event table.  build event from json, insert tags into a
+    // fresh tag table.  This was needed due to a logic error in
+    // how hex-like tags got indexed.
+    let start = Instant::now();
+    let tx = conn.transaction()?;
+    {
+        // Clear out table
+        tx.execute("DELETE FROM tag;", [])?;
+        let mut stmt = tx.prepare("select id, content from event order by id;")?;
+        let mut tag_rows = stmt.query([])?;
+        while let Some(row) = tag_rows.next()? {
+            // we want to capture the event_id that had the tag, the tag name, and the tag hex value.
+            let event_id: u64 = row.get(0)?;
+            let event_json: String = row.get(1)?;
+            let event: Event = serde_json::from_str(&event_json)?;
+            // look at each event, and each tag, creating new tag entries if appropriate.
+            for t in event.tags.iter().filter(|x| x.len() > 1) {
+                let tagname = t.get(0).unwrap();
+                let tagnamechar_opt = single_char_tagname(tagname);
+                if tagnamechar_opt.is_none() {
+                    continue;
+                }
+                // safe because len was > 1
+                let tagval = t.get(1).unwrap();
+                // insert as BLOB if we can restore it losslessly.
+                // this means it needs to be even length and lowercase.
+                if (tagval.len() % 2 == 0) && is_lower_hex(tagval) {
+                    tx.execute(
+                        "INSERT INTO tag (event_id, name, value_hex) VALUES (?1, ?2, ?3);",
+                        params![event_id, tagname, hex::decode(tagval).ok()],
+                    )?;
+                } else {
+                    // otherwise, insert as text
+                    tx.execute(
+                        "INSERT INTO tag (event_id, name, value) VALUES (?1, ?2, ?3);",
+                        params![event_id, tagname, &tagval],
+                    )?;
+                }
+            }
+        }
+        tx.execute("PRAGMA user_version = 6;", [])?;
+    }
+    tx.commit()?;
+    info!("database schema upgraded v5 -> v6 in {:?}", start.elapsed());
+    // vacuum after large table modification
+    let start = Instant::now();
+    conn.execute("VACUUM;", [])?;
+    info!("vacuumed DB after tags rebuild in {:?}", start.elapsed());
+    Ok(6)
+}
+
+fn mig_6_to_7(conn: &mut PooledConnection) -> Result<usize> {
+    info!("database schema needs update from 6->7");
+    // only change is adding a hidden column to events.
+    let upgrade_sql = r##"
+ALTER TABLE event ADD delegated_by BLOB;
+CREATE INDEX IF NOT EXISTS delegated_by_index ON event(delegated_by);
+PRAGMA user_version = 7;
+"##;
+    match conn.execute_batch(upgrade_sql) {
+        Ok(()) => {
+            info!("database schema upgraded v6 -> v7");
+        }
+        Err(err) => {
+            error!("update failed: {}", err);
+            panic!("database could not be upgraded");
+        }
+    }
+    Ok(7)
+}
@@ -2,8 +2,7 @@
 use crate::error::Result;
 use crate::event::Event;
 use serde::de::Unexpected;
-use serde::ser::SerializeMap;
-use serde::{Deserialize, Deserializer, Serialize, Serializer};
+use serde::{Deserialize, Deserializer, Serialize};
 use serde_json::Value;
 use std::collections::HashMap;
 use std::collections::HashSet;
@@ -20,7 +19,7 @@ pub struct Subscription {
 /// Corresponds to client-provided subscription request elements.  Any
 /// element can be present if it should be used in filtering, or
 /// absent ([`None`]) if it should be ignored.
-#[derive(PartialEq, Eq, Debug, Clone)]
+#[derive(Serialize, PartialEq, Eq, Debug, Clone)]
 pub struct ReqFilter {
    /// Event hashes
    pub ids: Option<Vec<String>>,
@@ -35,49 +34,12 @@ pub struct ReqFilter {
    /// Limit number of results
    pub limit: Option<u64>,
    /// Set of tags
+    #[serde(skip)]
    pub tags: Option<HashMap<char, HashSet<String>>>,
    /// Force no matches due to malformed data
-    // we can't represent it in the req filter, so we don't want to
-    // erroneously match.  This basically indicates the req tried to
-    // do something invalid.
    pub force_no_match: bool,
 }

-impl Serialize for ReqFilter {
-    fn serialize<S>(&self, serializer: S) -> Result<S::Ok, S::Error>
-    where
-        S: Serializer,
-    {
-        let mut map = serializer.serialize_map(None)?;
-        if let Some(ids) = &self.ids {
-            map.serialize_entry("ids", &ids)?;
-        }
-        if let Some(kinds) = &self.kinds {
-            map.serialize_entry("kinds", &kinds)?;
-        }
-        if let Some(until) = &self.until {
-            map.serialize_entry("until", until)?;
-        }
-        if let Some(since) = &self.since {
-            map.serialize_entry("since", since)?;
-        }
-        if let Some(limit) = &self.limit {
-            map.serialize_entry("limit", limit)?;
-        }
-        if let Some(authors) = &self.authors {
-            map.serialize_entry("authors", &authors)?;
-        }
-        // serialize tags
-        if let Some(tags) = &self.tags {
-            for (k, v) in tags {
-                let vals: Vec<&String> = v.iter().collect();
-                map.serialize_entry(&format!("#{k}"), &vals)?;
-            }
-        }
-        map.end()
-    }
-}
-
 impl<'de> Deserialize<'de> for ReqFilter {
    fn deserialize<D>(deserializer: D) -> Result<ReqFilter, D::Error>
    where
@@ -100,22 +62,12 @@ impl<'de> Deserialize<'de> for ReqFilter {
            tags: None,
            force_no_match: false,
        };
-        let empty_string = "".into();
        let mut ts = None;
        // iterate through each key, and assign values that exist
-        for (key, val) in filter {
+        for (key, val) in filter.into_iter() {
            // ids
            if key == "ids" {
-                let raw_ids: Option<Vec<String>> = Deserialize::deserialize(val).ok();
-                if let Some(a) = raw_ids.as_ref() {
-                    if a.contains(&empty_string) {
-                        return Err(serde::de::Error::invalid_type(
-                            Unexpected::Other("prefix matches must not be empty strings"),
-                            &"a json object",
-                        ));
-                    }
-                }
-                rf.ids = raw_ids;
+                rf.ids = Deserialize::deserialize(val).ok();
            } else if key == "kinds" {
                rf.kinds = Deserialize::deserialize(val).ok();
            } else if key == "since" {
@@ -125,16 +77,7 @@ impl<'de> Deserialize<'de> for ReqFilter {
            } else if key == "limit" {
                rf.limit = Deserialize::deserialize(val).ok();
            } else if key == "authors" {
-                let raw_authors: Option<Vec<String>> = Deserialize::deserialize(val).ok();
-                if let Some(a) = raw_authors.as_ref() {
-                    if a.contains(&empty_string) {
-                        return Err(serde::de::Error::invalid_type(
-                            Unexpected::Other("prefix matches must not be empty strings"),
-                            &"a json object",
-                        ));
-                    }
-                }
-                rf.authors = raw_authors;
+                rf.authors = Deserialize::deserialize(val).ok();
            } else if key.starts_with('#') && key.len() > 1 && val.is_array() {
                if let Some(tag_search) = tag_search_char_from_filter(key) {
                    if ts.is_none() {
@@ -144,7 +87,7 @@ impl<'de> Deserialize<'de> for ReqFilter {
                    if let Some(m) = ts.as_mut() {
                        let tag_vals: Option<Vec<String>> = Deserialize::deserialize(val).ok();
                        if let Some(v) = tag_vals {
-                            let hs = v.into_iter().collect::<HashSet<_>>();
+                            let hs = HashSet::from_iter(v.into_iter());
                            m.insert(tag_search.to_owned(), hs);
                        }
                    };
@@ -188,7 +131,7 @@ impl<'de> Deserialize<'de> for Subscription {
        D: Deserializer<'de>,
    {
        let mut v: Value = Deserialize::deserialize(deserializer)?;
-        // this should be a 3-or-more element array.
+        // this shoud be a 3-or-more element array.
        // verify the first element is a String, REQ
        // get the subscription from the second element.
        // convert each of the remaining objects into filters
@@ -225,7 +168,6 @@ impl<'de> Deserialize<'de> for Subscription {
            // create indexes
            filters.push(f);
        }
-        filters.dedup();
        Ok(Subscription {
            id: sub_id.to_owned(),
            filters,
@@ -235,23 +177,13 @@ impl<'de> Deserialize<'de> for Subscription {

 impl Subscription {
    /// Get a copy of the subscription identifier.
-    #[must_use]
    pub fn get_id(&self) -> String {
        self.id.clone()
    }
-
-    /// Determine if any filter is requesting historical (database)
-    /// queries.  If every filter has limit:0, we do not need to query the DB.
-    #[must_use]
-    pub fn needs_historical_events(&self) -> bool {
-        self.filters.iter().any(|f| f.limit != Some(0))
-    }
-
    /// Determine if this subscription matches a given [`Event`].  Any
    /// individual filter match is sufficient.
-    #[must_use]
    pub fn interested_in_event(&self, event: &Event) -> bool {
-        for f in &self.filters {
+        for f in self.filters.iter() {
            if f.interested_in_event(event) {
                return true;
            }
@@ -274,20 +206,23 @@ impl ReqFilter {
    fn ids_match(&self, event: &Event) -> bool {
        self.ids
            .as_ref()
-            .map_or(true, |vs| prefix_match(vs, &event.id))
+            .map(|vs| prefix_match(vs, &event.id))
+            .unwrap_or(true)
    }

    fn authors_match(&self, event: &Event) -> bool {
        self.authors
            .as_ref()
-            .map_or(true, |vs| prefix_match(vs, &event.pubkey))
+            .map(|vs| prefix_match(vs, &event.pubkey))
+            .unwrap_or(true)
    }

    fn delegated_authors_match(&self, event: &Event) -> bool {
        if let Some(delegated_pubkey) = &event.delegated_by {
            self.authors
                .as_ref()
-                .map_or(true, |vs| prefix_match(vs, delegated_pubkey))
+                .map(|vs| prefix_match(vs, delegated_pubkey))
+                .unwrap_or(true)
        } else {
            false
        }
@@ -311,16 +246,18 @@ impl ReqFilter {

    /// Check if this filter either matches, or does not care about the kind.
    fn kind_match(&self, kind: u64) -> bool {
-        self.kinds.as_ref().map_or(true, |ks| ks.contains(&kind))
+        self.kinds
+            .as_ref()
+            .map(|ks| ks.contains(&kind))
+            .unwrap_or(true)
    }

    /// Determine if all populated fields in this filter match the provided event.
-    #[must_use]
    pub fn interested_in_event(&self, event: &Event) -> bool {
        //        self.id.as_ref().map(|v| v == &event.id).unwrap_or(true)
        self.ids_match(event)
-            && self.since.map_or(true, |t| event.created_at >= t)
-            && self.until.map_or(true, |t| event.created_at <= t)
+            && self.since.map(|t| event.created_at > t).unwrap_or(true)
+            && self.until.map(|t| event.created_at < t).unwrap_or(true)
            && self.kind_match(event.kind)
            && (self.authors_match(event) || self.delegated_authors_match(event))
            && self.tag_match(event)
@@ -354,24 +291,6 @@ mod tests {
        assert!(serde_json::from_str::<Subscription>(raw_json).is_err());
    }

-    #[test]
-    fn req_empty_authors_prefix() {
-        let raw_json = "[\"REQ\",\"some-id\",{\"authors\": [\"\"]}]";
-        assert!(serde_json::from_str::<Subscription>(raw_json).is_err());
-    }
-
-    #[test]
-    fn req_empty_ids_prefix() {
-        let raw_json = "[\"REQ\",\"some-id\",{\"ids\": [\"\"]}]";
-        assert!(serde_json::from_str::<Subscription>(raw_json).is_err());
-    }
-
-    #[test]
-    fn req_empty_ids_prefix_mixed() {
-        let raw_json = "[\"REQ\",\"some-id\",{\"ids\": [\"\",\"aaa\"]}]";
-        assert!(serde_json::from_str::<Subscription>(raw_json).is_err());
-    }
-
    #[test]
    fn legacy_filter() {
        // legacy field in filter
@@ -379,23 +298,6 @@ mod tests {
        assert!(serde_json::from_str::<Subscription>(raw_json).is_ok());
    }

-    #[test]
-    fn dupe_filter() -> Result<()> {
-        let raw_json = r#"["REQ","some-id",{"kinds": [1984]}, {"kinds": [1984]}]"#;
-        let s: Subscription = serde_json::from_str(raw_json)?;
-        assert_eq!(s.filters.len(), 1);
-        Ok(())
-    }
-
-    #[test]
-    fn dupe_filter_many() -> Result<()> {
-        // duplicate filters in different order
-        let raw_json = r#"["REQ","some-id",{"kinds":[1984]},{"kinds":[1984]},{"kinds":[1984]},{"kinds":[1984]}]"#;
-        let s: Subscription = serde_json::from_str(raw_json)?;
-        assert_eq!(s.filters.len(), 1);
-        Ok(())
-    }
-
    #[test]
    fn author_filter() -> Result<()> {
        let raw_json = r#"["REQ","some-id",{"authors": ["test-author-id"]}]"#;
@@ -627,24 +529,4 @@ mod tests {
        assert!(!s.interested_in_event(&e));
        Ok(())
    }
-
-    #[test]
-    fn serialize_filter() -> Result<()> {
-        let s: Subscription = serde_json::from_str(
-            r##"["REQ","xyz",{"authors":["abc", "bcd"], "since": 10, "until": 20, "limit":100, "#e": ["foo", "bar"], "#d": ["test"]}]"##,
-        )?;
-        let f = s.filters.get(0);
-        let serialized = serde_json::to_string(&f)?;
-        let serialized_wrapped = format!(r##"["REQ", "xyz",{}]"##, serialized);
-        let parsed: Subscription = serde_json::from_str(&serialized_wrapped)?;
-        let parsed_filter = parsed.filters.get(0);
-        if let Some(pf) = parsed_filter {
-            assert_eq!(pf.since, Some(10));
-            assert_eq!(pf.until, Some(20));
-            assert_eq!(pf.limit, Some(100));
-        } else {
-            assert!(false, "filter could not be parsed");
-        }
-        Ok(())
-    }
 }
@@ -1,10 +1,7 @@
 //! Common utility functions
-use bech32::FromBase32;
 use std::time::SystemTime;
-use url::Url;

 /// Seconds since 1970.
-#[must_use]
 pub fn unix_time() -> u64 {
    SystemTime::now()
        .duration_since(SystemTime::UNIX_EPOCH)
@@ -13,36 +10,17 @@ pub fn unix_time() -> u64 {
 }

 /// Check if a string contains only hex characters.
-#[must_use]
 pub fn is_hex(s: &str) -> bool {
    s.chars().all(|x| char::is_ascii_hexdigit(&x))
 }

-/// Check if string is a nip19 string
-pub fn is_nip19(s: &str) -> bool {
-    s.starts_with("npub") || s.starts_with("note")
-}
-
-pub fn nip19_to_hex(s: &str) -> Result<String, bech32::Error> {
-    let (_hrp, data, _checksum) = bech32::decode(s)?;
-    let data = Vec::<u8>::from_base32(&data)?;
-    Ok(hex::encode(data))
-}
-
 /// Check if a string contains only lower-case hex chars.
-#[must_use]
 pub fn is_lower_hex(s: &str) -> bool {
    s.chars().all(|x| {
        (char::is_ascii_lowercase(&x) || char::is_ascii_digit(&x)) && char::is_ascii_hexdigit(&x)
    })
 }

-pub fn host_str(url: &str) -> Option<String> {
-    Url::parse(url)
-        .ok()
-        .and_then(|u| u.host_str().map(|s| s.to_string()))
-}
-
 #[cfg(test)]
 mod tests {
    use super::*;
@@ -50,23 +28,6 @@ mod tests {
    #[test]
    fn lower_hex() {
        let hexstr = "abcd0123";
-        assert!(is_lower_hex(hexstr));
-    }
-
-    #[test]
-    fn nip19() {
-        let hexkey = "3bf0c63fcb93463407af97a5e5ee64fa883d107ef9e558472c4eb9aaaefa459d";
-        let nip19key = "npub180cvv07tjdrrgpa0j7j7tmnyl2yr6yr7l8j4s3evf6u64th6gkwsyjh6w6";
-        assert!(!is_nip19(hexkey));
-        assert!(is_nip19(nip19key));
-    }
-
-    #[test]
-    fn nip19_hex() {
-        let nip19key = "npub180cvv07tjdrrgpa0j7j7tmnyl2yr6yr7l8j4s3evf6u64th6gkwsyjh6w6";
-        let expected = "3bf0c63fcb93463407af97a5e5ee64fa883d107ef9e558472c4eb9aaaefa459d";
-        let got = nip19_to_hex(nip19key).unwrap();
-
-        assert_eq!(expected, got);
+        assert_eq!(is_lower_hex(hexstr), true);
    }
 }
@@ -1,10 +0,0 @@
-#[cfg(test)]
-mod tests {
-    use nostr_rs_relay::cli::CLIArgs;
-
-    #[test]
-    fn cli_tests() {
-        use clap::CommandFactory;
-        CLIArgs::command().debug_assert();
-    }
-}
@@ -36,9 +36,9 @@ pub fn start_relay() -> Result<Relay> {
    settings.database.min_conn = 4;
    settings.database.max_conn = 8;
    let (shutdown_tx, shutdown_rx): (MpscSender<()>, MpscReceiver<()>) = syncmpsc::channel();
-    let handle = thread::spawn(move || {
+    let handle = thread::spawn(|| {
        // server will block the thread it is run on.
-        let _ = start_server(&settings, shutdown_rx);
+        let _ = start_server(settings, shutdown_rx);
    });
    // how do we know the relay has finished starting up?
    Ok(Relay {
@@ -1,356 +0,0 @@
-#[cfg(test)]
-mod tests {
-    use bitcoin_hashes::hex::ToHex;
-    use bitcoin_hashes::sha256;
-    use bitcoin_hashes::Hash;
-    use secp256k1::rand;
-    use secp256k1::{KeyPair, Secp256k1, XOnlyPublicKey};
-
-    use nostr_rs_relay::conn::ClientConn;
-    use nostr_rs_relay::error::Error;
-    use nostr_rs_relay::event::Event;
-    use nostr_rs_relay::utils::unix_time;
-
-    const RELAY: &str = "wss://nostr.example.com/";
-
-    #[test]
-    fn test_generate_auth_challenge() {
-        let mut client_conn = ClientConn::new("127.0.0.1".into());
-
-        assert_eq!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        client_conn.generate_auth_challenge();
-
-        assert_ne!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        let last_auth_challenge = client_conn.auth_challenge().cloned();
-
-        client_conn.generate_auth_challenge();
-
-        assert_ne!(client_conn.auth_challenge(), None);
-        assert_ne!(
-            client_conn.auth_challenge().unwrap(),
-            &last_auth_challenge.unwrap()
-        );
-        assert_eq!(client_conn.auth_pubkey(), None);
-    }
-
-    #[test]
-    fn test_authenticate_with_valid_event() {
-        let mut client_conn = ClientConn::new("127.0.0.1".into());
-
-        assert_eq!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        client_conn.generate_auth_challenge();
-
-        assert_ne!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        let challenge = client_conn.auth_challenge().unwrap();
-        let event = auth_event(challenge);
-
-        let result = client_conn.authenticate(&event, RELAY.into());
-
-        assert!(matches!(result, Ok(())));
-        assert_eq!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), Some(&event.pubkey));
-    }
-
-    #[test]
-    fn test_fail_to_authenticate_in_invalid_state() {
-        let mut client_conn = ClientConn::new("127.0.0.1".into());
-
-        assert_eq!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        let event = auth_event(&"challenge".into());
-        let result = client_conn.authenticate(&event, RELAY.into());
-
-        assert!(matches!(result, Err(Error::AuthFailure)));
-    }
-
-    #[test]
-    fn test_authenticate_when_already_authenticated() {
-        let mut client_conn = ClientConn::new("127.0.0.1".into());
-
-        assert_eq!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        client_conn.generate_auth_challenge();
-
-        assert_ne!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        let challenge = client_conn.auth_challenge().unwrap().clone();
-
-        let event = auth_event(&challenge);
-        let result = client_conn.authenticate(&event, RELAY.into());
-
-        assert!(matches!(result, Ok(())));
-        assert_eq!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), Some(&event.pubkey));
-
-        let event1 = auth_event(&challenge);
-        let result1 = client_conn.authenticate(&event1, RELAY.into());
-
-        assert!(matches!(result1, Ok(())));
-        assert_eq!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), Some(&event.pubkey));
-        assert_ne!(client_conn.auth_pubkey(), Some(&event1.pubkey));
-    }
-
-    #[test]
-    fn test_fail_to_authenticate_with_invalid_event() {
-        let mut client_conn = ClientConn::new("127.0.0.1".into());
-
-        assert_eq!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        client_conn.generate_auth_challenge();
-
-        assert_ne!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        let challenge = client_conn.auth_challenge().unwrap();
-        let mut event = auth_event(challenge);
-        event.sig = event.sig.chars().rev().collect::<String>();
-
-        let result = client_conn.authenticate(&event, RELAY.into());
-
-        assert!(matches!(result, Err(Error::AuthFailure)));
-    }
-
-    #[test]
-    fn test_fail_to_authenticate_with_invalid_event_kind() {
-        let mut client_conn = ClientConn::new("127.0.0.1".into());
-
-        assert_eq!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        client_conn.generate_auth_challenge();
-
-        assert_ne!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        let challenge = client_conn.auth_challenge().unwrap();
-        let event = auth_event_with_kind(challenge, 9999999999999999);
-
-        let result = client_conn.authenticate(&event, RELAY.into());
-
-        assert!(matches!(result, Err(Error::AuthFailure)));
-    }
-
-    #[test]
-    fn test_fail_to_authenticate_with_expired_timestamp() {
-        let mut client_conn = ClientConn::new("127.0.0.1".into());
-
-        assert_eq!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        client_conn.generate_auth_challenge();
-
-        assert_ne!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        let challenge = client_conn.auth_challenge().unwrap();
-        let event = auth_event_with_created_at(challenge, unix_time() - 1200); // 20 minutes
-
-        let result = client_conn.authenticate(&event, RELAY.into());
-
-        assert!(matches!(result, Err(Error::AuthFailure)));
-    }
-
-    #[test]
-    fn test_fail_to_authenticate_with_future_timestamp() {
-        let mut client_conn = ClientConn::new("127.0.0.1".into());
-
-        assert_eq!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        client_conn.generate_auth_challenge();
-
-        assert_ne!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        let challenge = client_conn.auth_challenge().unwrap();
-        let event = auth_event_with_created_at(challenge, unix_time() + 1200); // 20 minutes
-
-        let result = client_conn.authenticate(&event, RELAY.into());
-
-        assert!(matches!(result, Err(Error::AuthFailure)));
-    }
-
-    #[test]
-    fn test_fail_to_authenticate_without_tags() {
-        let mut client_conn = ClientConn::new("127.0.0.1".into());
-
-        assert_eq!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        client_conn.generate_auth_challenge();
-
-        assert_ne!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        let event = auth_event_without_tags();
-
-        let result = client_conn.authenticate(&event, RELAY.into());
-
-        assert!(matches!(result, Err(Error::AuthFailure)));
-    }
-
-    #[test]
-    fn test_fail_to_authenticate_without_challenge() {
-        let mut client_conn = ClientConn::new("127.0.0.1".into());
-
-        assert_eq!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        client_conn.generate_auth_challenge();
-
-        assert_ne!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        let event = auth_event_without_challenge();
-
-        let result = client_conn.authenticate(&event, RELAY.into());
-
-        assert!(matches!(result, Err(Error::AuthFailure)));
-    }
-
-    #[test]
-    fn test_fail_to_authenticate_without_relay() {
-        let mut client_conn = ClientConn::new("127.0.0.1".into());
-
-        assert_eq!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        client_conn.generate_auth_challenge();
-
-        assert_ne!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        let challenge = client_conn.auth_challenge().unwrap();
-        let event = auth_event_without_relay(challenge);
-
-        let result = client_conn.authenticate(&event, RELAY.into());
-
-        assert!(matches!(result, Err(Error::AuthFailure)));
-    }
-
-    #[test]
-    fn test_fail_to_authenticate_with_invalid_challenge() {
-        let mut client_conn = ClientConn::new("127.0.0.1".into());
-
-        assert_eq!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        client_conn.generate_auth_challenge();
-
-        assert_ne!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        let event = auth_event(&"invalid challenge".into());
-
-        let result = client_conn.authenticate(&event, RELAY.into());
-
-        assert!(matches!(result, Err(Error::AuthFailure)));
-    }
-
-    #[test]
-    fn test_fail_to_authenticate_with_invalid_relay() {
-        let mut client_conn = ClientConn::new("127.0.0.1".into());
-
-        assert_eq!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        client_conn.generate_auth_challenge();
-
-        assert_ne!(client_conn.auth_challenge(), None);
-        assert_eq!(client_conn.auth_pubkey(), None);
-
-        let challenge = client_conn.auth_challenge().unwrap();
-        let event = auth_event_with_relay(challenge, &"xyz".into());
-
-        let result = client_conn.authenticate(&event, RELAY.into());
-
-        assert!(matches!(result, Err(Error::AuthFailure)));
-    }
-
-    fn auth_event(challenge: &String) -> Event {
-        create_auth_event(Some(challenge), Some(&RELAY.into()), 22242, unix_time())
-    }
-
-    fn auth_event_with_kind(challenge: &String, kind: u64) -> Event {
-        create_auth_event(Some(challenge), Some(&RELAY.into()), kind, unix_time())
-    }
-
-    fn auth_event_with_created_at(challenge: &String, created_at: u64) -> Event {
-        create_auth_event(Some(challenge), Some(&RELAY.into()), 22242, created_at)
-    }
-
-    fn auth_event_without_challenge() -> Event {
-        create_auth_event(None, Some(&RELAY.into()), 22242, unix_time())
-    }
-
-    fn auth_event_without_relay(challenge: &String) -> Event {
-        create_auth_event(Some(challenge), None, 22242, unix_time())
-    }
-
-    fn auth_event_without_tags() -> Event {
-        create_auth_event(None, None, 22242, unix_time())
-    }
-
-    fn auth_event_with_relay(challenge: &String, relay: &String) -> Event {
-        create_auth_event(Some(challenge), Some(relay), 22242, unix_time())
-    }
-
-    fn create_auth_event(
-        challenge: Option<&String>,
-        relay: Option<&String>,
-        kind: u64,
-        created_at: u64,
-    ) -> Event {
-        let secp = Secp256k1::new();
-        let key_pair = KeyPair::new(&secp, &mut rand::thread_rng());
-        let public_key = XOnlyPublicKey::from_keypair(&key_pair);
-
-        let mut tags: Vec<Vec<String>> = vec![];
-
-        if let Some(c) = challenge {
-            let tag = vec!["challenge".into(), c.into()];
-            tags.push(tag);
-        }
-
-        if let Some(r) = relay {
-            let tag = vec!["relay".into(), r.into()];
-            tags.push(tag);
-        }
-
-        let mut event = Event {
-            id: "0".to_owned(),
-            pubkey: public_key.to_hex(),
-            delegated_by: None,
-            created_at,
-            kind,
-            tags,
-            content: "".to_owned(),
-            sig: "0".to_owned(),
-            tagidx: None,
-        };
-
-        let c = event.to_canonical().unwrap();
-        let digest: sha256::Hash = sha256::Hash::hash(c.as_bytes());
-
-        let msg = secp256k1::Message::from_slice(digest.as_ref()).unwrap();
-        let sig = secp.sign_schnorr(&msg, &key_pair);
-
-        event.id = format!("{digest:x}");
-        event.sig = sig.to_hex();
-
-        event
-    }
-}
@@ -1,10 +1,8 @@
 use anyhow::Result;
-use futures::SinkExt;
-use futures::StreamExt;
+
 use std::thread;
 use std::time::Duration;
-use tokio_tungstenite::connect_async;
-use tracing::info;
+
 mod common;

 #[tokio::test]
@@ -47,33 +45,3 @@ async fn relay_home_page() -> Result<()> {
    let _res = relay.shutdown_tx.send(());
    Ok(())
 }
-
-//#[tokio::test]
-// Still inwork
-async fn publish_test() -> Result<()> {
-    // get a relay and wait for startup
-    let relay = common::start_relay()?;
-    common::wait_for_healthy_relay(&relay).await?;
-    // open a non-secure websocket connection.
-    let (mut ws, _res) = connect_async(format!("ws://localhost:{}", relay.port)).await?;
-    // send a simple pre-made message
-    let simple_event = r#"["EVENT", {"content": "hello world","created_at": 1691239763,
-      "id":"f3ce6798d70e358213ebbeba4886bbdfacf1ecfd4f65ee5323ef5f404de32b86",
-      "kind": 1,
-      "pubkey": "79be667ef9dcbbac55a06295ce870b07029bfcdb2dce28d959f2815b16f81798",
-      "sig": "30ca29e8581eeee75bf838171dec818af5e6de2b74f5337de940f5cc91186534c0b20d6cf7ad1043a2c51dbd60b979447720a471d346322103c83f6cb66e4e98",
-      "tags": []}]"#;
-    ws.send(simple_event.into()).await?;
-    // get response from server, confirm it is an array with first element "OK"
-    let event_confirm = ws.next().await;
-    ws.close(None).await?;
-    info!("event confirmed: {:?}", event_confirm);
-    // open a new connection, and wait for some time to get the event.
-    let (mut sub_ws, _res) = connect_async(format!("ws://localhost:{}", relay.port)).await?;
-    let event_sub = r#"["REQ", "simple", {}]"#;
-    sub_ws.send(event_sub.into()).await?;
-    // read from subscription
-    let _ws_next = sub_ws.next().await;
-    let _res = relay.shutdown_tx.send(());
-    Ok(())
-}