Commit ea155b4
committed
fix(server): provide fresh ConfigProvider per HttpApi listener
Effect's `ConfigProvider.fromEnv()` spreads `process.env` into a frozen
trie at construction time, and the default `ConfigProvider` is cached on
a module-singleton `Context.Reference` after the first read. Without an
explicit override, every later `Server.listen()` keeps observing the
env snapshot from the first config read in the process.
Install a fresh `ConfigProvider` inside `buildLayer` so each listener's
`Config.string(...)` resolves against the current `process.env`. This
keeps `ServerAuth.Config` (and any other env-backed config) consistent
with `Flag.*` reads in production and across the test suite.
Parameterize the no-auth listen test over both backends so the
effect-httpapi path is exercised; previously it was hardcoded to Hono.1 parent 2c819f2 commit ea155b4
2 files changed
Lines changed: 25 additions & 17 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
8 | | - | |
| 8 | + | |
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
| |||
259 | 259 | | |
260 | 260 | | |
261 | 261 | | |
| 262 | + | |
| 263 | + | |
| 264 | + | |
| 265 | + | |
| 266 | + | |
| 267 | + | |
262 | 268 | | |
263 | 269 | | |
264 | 270 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
40 | 40 | | |
41 | 41 | | |
42 | 42 | | |
43 | | - | |
44 | | - | |
| 43 | + | |
| 44 | + | |
45 | 45 | | |
46 | 46 | | |
47 | 47 | | |
| |||
300 | 300 | | |
301 | 301 | | |
302 | 302 | | |
303 | | - | |
304 | | - | |
305 | | - | |
306 | | - | |
307 | | - | |
308 | | - | |
309 | | - | |
310 | | - | |
311 | | - | |
312 | | - | |
313 | | - | |
314 | | - | |
315 | | - | |
316 | | - | |
| 303 | + | |
| 304 | + | |
| 305 | + | |
| 306 | + | |
| 307 | + | |
| 308 | + | |
| 309 | + | |
| 310 | + | |
| 311 | + | |
| 312 | + | |
| 313 | + | |
| 314 | + | |
| 315 | + | |
| 316 | + | |
| 317 | + | |
| 318 | + | |
317 | 319 | | |
0 commit comments