refactor!: serve the static frontend trough the backend #187

New Issue

AtHeartEngineer · 2025-07-08T08:41:52-04:00

AtHeartEngineer commented

2025-07-08 08:41:52 -04:00

Originally created by @stonith404 on 5/9/2025

Fixes: https://github.com/pocket-id/pocket-id/issues/536

TODO

Cache static assets
Setup GitHub action to build binaries and attach them to new releases

Breaking changes

The following breaking changes should be documented in version 1.0.0:

Port in the "ports" section and in the health check in the docker-compose.yml has to be changed to 1411.
PUBLIC_UI_CONFIG_DISABLED has been renamed to UI_CONFIG_DISABLED
CADDY_DISABLED has been removed. If you've disabled Caddy, you probably have path mappings to /.well-known and /api in your reverse proxy configuration. These mappings aren't necessary anymore, your reverse proxy should now solely point to Pocket ID on port 1411 (or the port set with the PORT variable).
CADDY_PORT has been removed and replaced by PORT
BACKEND_PORT has been removed and replaced by PORT
The behavior of PORT has been changed. This variable now doesn't change the port of the frontend anymore, but for the whole application.

*Originally created by @stonith404 on 5/9/2025* Fixes: https://github.com/pocket-id/pocket-id/issues/536 ## TODO - [x] [Cache static assets](https://github.com/pocket-id/pocket-id/pull/520#discussion_r2081971318) - [x] Setup GitHub action to build binaries and attach them to new releases ## Breaking changes The following breaking changes should be documented in version `1.0.0`: - Port in the "ports" section and in the health check in the `docker-compose.yml` has to be changed to `1411`. - `PUBLIC_UI_CONFIG_DISABLED` has been renamed to `UI_CONFIG_DISABLED` - `CADDY_DISABLED` has been removed. If you've disabled Caddy, you probably have path mappings to `/.well-known` and `/api` in your reverse proxy configuration. These mappings aren't necessary anymore, your reverse proxy should now solely point to Pocket ID on port `1411` (or the port set with the `PORT` variable). - `CADDY_PORT` has been removed and replaced by `PORT` - `BACKEND_PORT` has been removed and replaced by `PORT` - The behavior of `PORT` has been changed. This variable now doesn't change the port of the frontend anymore, but for the whole application.

AtHeartEngineer closed this issue

2025-07-08 08:41:53 -04:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github/pocket-id#187