Allow unknown config options in client#218
Conversation
We purposely don't parse the entire the config in the client, only the things releveant to the client. When we added the denys in e5eccc1 we now will fail with a normal config because our cli will throw an error for the config options we don't parse in the client.
Exercise the CLI against the server-generated config file so e2e tests cover client-side parsing of daemon config sections. Rebuild the e2e CLI binary when the client crate changes so this path cannot use stale code.
|
👋 Thanks for assigning @tankyleo as a reviewer! |
Ah my bad, I was still in the old world where we have So I thought clients just had a separate configuration file called |
Bug accidentally introduced in #216, now tests will catch
We purposely don't parse the entire the config in the client, only the things releveant to the client. When we added the denys in e5eccc1 we now will fail with a normal config because our cli will throw an error for the config options we don't parse in the client.