Hey Shivansh,

Thanks for the proposal!

Supporting _file seems like an interesting and fair request. I don't think
we should limit it to just getjson though as your PR currently does. The
feature should live at the higher level that _cmd and _secret live with all
the same checks and allowlisting that those abide by.

Cheers,
Niko

On Wed, Sep 9, 2026 at 6:24 AM singa ser <[email protected]> wrote:

> Hi everyone,
>
> I recently opened PR #72355 to support *_file and *_cmd suffixes in
> AirflowConfigParser.getjson().
>
> Right now, getjson() doesn't check for file or command suffixes when
> resolving configs (unlike get(), getboolean(), etc.), so setting JSON
> configs via secrets files or commands doesn't work out of the box.
>
> Jarek suggested bringing this to the dev list to see if anyone has feedback
> or concerns before we merge it.
>
> PR link: https://github.com/apache/airflow/pull/72355
>
> Thanks,
> Shivansh
>

Reply via email to