umputun / cronn / 19485067761

75%

Build Type

push

github

Committed by web-flow

Commit Message

Add base URL support for reverse proxy deployments (#53)

* feat: add base URL support for reverse proxy deployments

add --web.base-url flag to enable running web dashboard behind reverse proxy at subpath (e.g., /cronn). implements url() template helper for dynamic URL generation, cookiePath() helper for cookie scoping, and http.StripPrefix wrapper for proper routing.

- add --web.base-url flag with validation (must start with /, trailing slash removed)
- implement url() and cookiePath() template helpers
- update all templates (base, login, dashboard, jobs) to use url helper
- update app.js to use window.BASE_URL for dynamic API calls
- add cookie path scoping to match base URL
- add comprehensive tests (9 subtests covering all scenarios)
- update README with reverse proxy configuration examples

backward compatible - empty base URL defaults to root path

* test: add validateBaseURL tests and improve nginx docs

- add table-driven test for validateBaseURL covering edge cases
- fix nginx config to handle both /cronn and /cronn/ paths
- add redirect for consistency in reverse proxy setup

* fix(web): correct base URL redirect without trailing slash

add handler() method to properly redirect base URL requests
without trailing slash to the correct path. previously,
/cronn would incorrectly redirect to / instead of /cronn/.

the fix extracts base URL wrapping logic into a dedicated
handler() method that both production and tests can use,
preventing logic duplication.

related #52

* test(web): fix flaky cleanup test race condition

the Eventually check was succeeding as soon as 3 executions existed, which
could happen after processing only the first 3 events. in slower CI
environments, the remaining events would process after the check, leaving 4
executions instead of the expected 3.

now the check waits for the newest execution (output 4) to exist, ensuring
all 5 events are fully processed before verifying the final count.

Run Details

45 of 51 new or added lines in 4 files covered. (88.24%)

2 existing lines in 1 file now uncovered.

2530 of 3397 relevant lines covered (74.48%)

37.9 hits per line