Skip to content

🐛 Accept standard-table form for proxy config#162

Merged
hayleigh-dot-dev merged 1 commit into
lustre-labs:mainfrom
jtdowney:fix-proxy-standard-table
Apr 21, 2026
Merged

🐛 Accept standard-table form for proxy config#162
hayleigh-dot-dev merged 1 commit into
lustre-labs:mainfrom
jtdowney:fix-proxy-standard-table

Conversation

@jtdowney
Copy link
Copy Markdown
Contributor

Previously, only the inline-table form was recognized, e.g.

[tools.lustre.dev]
proxy = { from = "/api", to = "http://localhost:3000/api" }

The equivalent standard-table form failed with ProxyInvalidConfig:

[tools.lustre.dev.proxy]
from = "/api"
to = "http://localhost:3000/api"

Accept both tom.InlineTable and tom.Table in get_proxies_from_config, for the single-proxy case and entries inside an array of proxies.

Previously, only the inline-table form was recognized, e.g.

    [tools.lustre.dev]
    proxy = { from = "/api", to = "http://localhost:3000/api" }

The equivalent standard-table form failed with ProxyInvalidConfig:

    [tools.lustre.dev.proxy]
    from = "/api"
    to = "http://localhost:3000/api"

Accept both tom.InlineTable and tom.Table in get_proxies_from_config,
for the single-proxy case and entries inside an array of proxies.
@hayleigh-dot-dev
Copy link
Copy Markdown
Contributor

Oh cool, i wonder why tom even exposes them as different things. Thanks!

@hayleigh-dot-dev hayleigh-dot-dev merged commit 8a2b8c0 into lustre-labs:main Apr 21, 2026
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.

2 participants