Compute keep_open_until in /allowed + 9 bugfixes #8

2026-06-17T13:54:15Z

mykhailo commented

2026-06-17 13:54:15 +00:00

First-time contributor

Closes #1

A. Hold feature: /allowed returns keep_open_until from the newest approved hold in doorlog; /longpoll skips hold events to avoid spurious open pulses.

B. 9 bugfixes: assert→raise SECRET check, text() on 403, removed dead /logs code, flattened auth decorator, by_slackid None fallback, load kube config once, guard missing slack command, backoff on PyMongoError, mongo→mongosh.

Files: app/doorboy-proxy.py, app/kube.py, app/slack.py, mongo-init.sh.

Closes #1 **A. Hold feature:** /allowed returns `keep_open_until` from the newest approved hold in `doorlog`; /longpoll skips hold events to avoid spurious open pulses. **B. 9 bugfixes:** assert→raise SECRET check, text() on 403, removed dead /logs code, flattened auth decorator, by_slackid None fallback, load kube config once, guard missing slack command, backoff on PyMongoError, mongo→mongosh. Files: app/doorboy-proxy.py, app/kube.py, app/slack.py, mongo-init.sh.

mykhailo added 1 commit 2026-06-17 13:54:15 +00:00

Add keep_open_until to /allowed for hold-door; fix 9 bugs e6fc5cb85f

/allowed returns keep_open_until from the newest approved hold in
doorlog; /longpoll skips hold events to avoid spurious open pulses.

Fixes: assert->raise for SECRET check, text() on 403, remove dead
/logs code, flatten auth decorator, by_slackid None fallback, load
kube config once, guard missing slack command, backoff on PyMongoError,
mongo->mongosh.

You are not authorized to merge this pull request.

This pull request can be merged automatically.

This branch is out-of-date with the base branch

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u https://git.k-space.ee/mykhailo/doorboy-proxy issue-1-keep-open:mykhailo-issue-1-keep-open

git checkout mykhailo-issue-1-keep-open

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: k-space/doorboy-proxy#8

Allow edits from maintainers