Skip to content

fix(ui): improve access-log filters and Caddy config readability - #101

Merged
RentnerKev merged 3 commits into
mainfrom
fix/access-log-filters-caddy-config
Sep 13, 2026
Merged

RentnerKev merged 3 commits into
mainfrom
fix/access-log-filters-caddy-config

Conversation

@RentnerKev

@RentnerKev RentnerKev commented Sep 13, 2026

Copy link
Copy Markdown
Owner

Summary

Access logs now offer a searchable host dropdown, a status dropdown and debounced case-insensitive search. Filter actions align on desktop, and client IPs stay on one line.

Language settings and access-log country flags use country-flag-icons. The old language SVGs, flag-icons, geoip-country and its dependency override are removed. Web responses preserve stored country codes; the web service does not run GeoIP lookups.

Global Caddy Active config is indented and syntax-highlighted. Generated defaults, Preview and Reload are removed from the global editor; save/reset and background-fetch guards remain.

Outstanding work

The requested persistent country enrichment using only sapics/ip-location-db user-country.mmdb is not integrated yet. This PR must not be treated as complete for that requirement. No MaxMind database, accounts, credentials, updater container, external GeoIP API or Cloudflare country-header lookup is included.

Related issue

Fixes #100

Type of change

  • Bug fix
  • New feature
  • Refactoring
  • Documentation
  • Dependency update

Testing

  • The initial implementation passed the full bun run check suite.
  • The flag migration and cleanup passed 22 focused tests, TypeScript checks, lint and the web production build.
  • Initial browser inspection with synthetic fixtures verified filters, inline controls and configuration highlighting.
  • GitHub CI runs again for this head; persistent MMDB enrichment remains outstanding.

Checklist

  • Relevant completed changes tested.
  • Tests updated where appropriate.
  • No secrets, credentials, production data or private keys included.
  • Documentation updated.
  • Related issue linked.
  • Persistent country enrichment completed and verified.

@RentnerKev RentnerKev added this to the v1.0.0-beta.1 milestone Sep 13, 2026
@coderabbitai

coderabbitai Bot commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: eb8e0776-dbc8-4603-a66f-76ff6bf1e744


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added documentation Improvements or additions to documentation area: proxy Proxy host configuration and request forwarding. area: runtime OpenResty and privileged controller runtime behavior. area: ui Web interface components, routes, styling, and localization. labels Sep 13, 2026
@github-actions

Copy link
Copy Markdown

🐳 RentnerProxy PR Preview

A test-only image for this pull request has been published successfully.

Images

Type Image
Moving preview for this PR ghcr.io/rentnerkev/rentnerproxy:pr-101
Exact tested build ghcr.io/rentnerkev/rentnerproxy:pr-101-e0e61198bce6

Image digest: sha256:8a298d2300f0b3c09ecc72702f3c5d1bfd3480d57119e4ffafb73cc082fb0e01

Tested and built merge commit: e0e61198bce6cc732d652c69042a4966953dabd9

Pull request head commit: 6f94dfae09873dc8da7e37bb43be5e287f4a7fb9

For reproducible testing and bug reports, prefer pr-101-e0e61198bce6. The pr-101 tag moves to the newest successful preview for this PR.


⚠️ Unreviewed development preview

This image contains unreviewed pull-request code. It may contain bugs, breaking changes, incomplete or incompatible migrations, configuration errors, and changes that damage data or runtime state.

Do not use this PR preview image with production data.

Before testing:

  1. Back up PostgreSQL.
  2. Back up the complete RentnerProxy controller/runtime state in /var/lib/rentnerproxy, including certificates, private keys, ACME state, active configuration, last-known-good state, and trusted CA material.
  3. Prefer a separate test database and separate Docker volumes.
  4. Use an isolated Docker Compose project and avoid production port bindings.

Docker

docker pull ghcr.io/rentnerkev/rentnerproxy:pr-101-e0e61198bce6

Docker Compose

In a separate test copy of docker-compose.yml, replace the service image with:

image: ghcr.io/rentnerkev/rentnerproxy:pr-101-e0e61198bce6

Then use an isolated Compose project:

docker compose --project-name rentnerproxy-pr-101 pull
docker compose --project-name rentnerproxy-pr-101 up -d

The repository Compose file binds ports 80, 81, and 443; change those bindings or stop the production stack before testing.

This preview is intended only for testing this pull request and is not a production release.

@RentnerKev
RentnerKev merged commit 2d80036 into main Sep 13, 2026
20 of 22 checks passed
@RentnerKev
RentnerKev deleted the fix/access-log-filters-caddy-config branch September 13, 2026 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: proxy Proxy host configuration and request forwarding. area: runtime OpenResty and privileged controller runtime behavior. area: ui Web interface components, routes, styling, and localization. documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(ui): improve access-log filters and global Caddy config readability

1 participant