Skip to content

Remove Proxy and Email Notifications#22

Merged
zupzup merged 1 commit intomasterfrom
cleanup-notif-proxy
Feb 23, 2026
Merged

Remove Proxy and Email Notifications#22
zupzup merged 1 commit intomasterfrom
cleanup-notif-proxy

Conversation

@zupzup
Copy link
Contributor

@zupzup zupzup commented Feb 20, 2026

  • Cleanup Static File Server, Proxy and Email Notifications in bcr-relay
  • In preparation for moving this repo to Wildcat-Aux

@zupzup zupzup self-assigned this Feb 20, 2026
Copilot AI review requested due to automatic review settings February 20, 2026 09:25
@codecov
Copy link

codecov bot commented Feb 20, 2026

Codecov Report

❌ Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 23.52%. Comparing base (7e5bca9) to head (9dfda0b).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
src/main.rs 0.00% 4 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master      #22       +/-   ##
===========================================
+ Coverage   10.23%   23.52%   +13.29%     
===========================================
  Files          13        6        -7     
  Lines        1465      340     -1125     
===========================================
- Hits          150       80       -70     
+ Misses       1315      260     -1055     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes the proxy and email notification subsystems (including their routes, templates, and DB tables) and simplifies the runtime/container setup accordingly, as part of general cleanup ahead of moving the repo.

Changes:

  • Removed proxy and email notification modules, endpoints, and associated utilities/templates.
  • Removed notification-related Postgres schema initialization.
  • Simplified runtime packaging by dropping static file serving/copying and trimming reported relay features.

Reviewed changes

Copilot reviewed 15 out of 17 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
static/logo.png Removed static asset (static file serving no longer exposed).
src/util.rs Removed shared helpers that were only used by proxy/notifications.
src/proxy/mod.rs Removed proxy endpoint implementation and supporting code.
src/notification/mod.rs Removed notification HTTP API and HTML rendering logic.
src/notification/* Removed notification preferences/store/email implementation and templates.
src/relay.rs Removed email-related config fields; relay remains focused on Nostr + rate limiting.
src/rate_limit.rs Removed HTTP-facing rate limiter/RealIp extraction; retained sliding-window limiter for relay policies.
src/main.rs Removed static serving and proxy/notification routes; updated advertised relay features and app state.
src/db/mod.rs Removed creation of notification-specific tables.
Dockerfile Stopped copying static assets into the runtime image.
Cargo.toml Removed several deps, but still includes now-unused direct deps (see comments).
Cargo.lock Reflected dependency graph changes and patch-level updates.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@zupzup zupzup force-pushed the cleanup-notif-proxy branch from 8ed21f1 to 9dfda0b Compare February 20, 2026 09:35
@zupzup zupzup merged commit 58b9a7b into master Feb 23, 2026
7 of 8 checks passed
@zupzup zupzup deleted the cleanup-notif-proxy branch February 23, 2026 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants