Skip to content

fix(app_config) :: test the documented connection timeout defaults - #1436

Open
81reap wants to merge 1 commit into
sqlpage:mainfrom
81reap:mutation/12-1-connection-timeout-defaults
Open

fix(app_config) :: test the documented connection timeout defaults#1436
81reap wants to merge 1 commit into
sqlpage:mainfrom
81reap:mutation/12-1-connection-timeout-defaults

Conversation

@81reap

@81reap 81reap commented Sep 7, 2026

Copy link
Copy Markdown
Collaborator

configuration.md documents that (1) the idle timeout defaults to 30 minutes, (2) the maximum lifetime to an hour, (3) that SQLite doesn't have those limits, and (4) that 0 disables the timeout. Adds tests for these cases.

discovered by #1396

SQLPage/configuration.md

Lines 18 to 19 in 6d32b40

| `database_connection_idle_timeout_seconds` | SQLite: None<BR> All other: 30 minutes | Automatically close database connections after this period of inactivity. Set to 0 to disable. |
| `database_connection_max_lifetime_seconds` | SQLite: None<BR> All other: 60 minutes | Always close database connections after this amount of time. Set to 0 to disable. |

`configuration.md` documents that (1) the idle timeout defaults to 30 minutes, (2) the maximum lifetime to an hour, (3) that SQLite doesn't have those limits, and (4) that `0` disables the timeout. Adds tests for these cases.
@81reap
81reap marked this pull request as ready for review September 7, 2026 05:28
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.

1 participant