I agree. I think it's a good option to add for 3.4. I also agree with Niko that it should be a higher, generic feature across the board—the same as all other config options. There is no particular reason to limit it to .json only, I can clearly see the case where you want to read string or boolean values from files named by the variables.
J. On Fri, Sep 11, 2026 at 9:04 PM Niko Oliveira <[email protected]> wrote: > 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 > > >
