You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/03/31 00:40:43 UTC

[GitHub] [airflow] mik-laj edited a comment on pull request #15013: Enable Connection creation from Vault parameters

mik-laj edited a comment on pull request #15013:
URL: https://github.com/apache/airflow/pull/15013#issuecomment-810669033


   > Is there a good reason why extra must be encoded JSON? Seems like a nice solution would be to allow for any string there.
   
   We expect the extra field to be a  dictionary in Web UI. The use of primitive types also causes a problem with extending a given connection ie. it is not future-proff. We also have a design misunderstood here. This field should contain extra **fields**, not additional data. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org