potiuk commented on PR #35694:
URL: https://github.com/apache/airflow/pull/35694#issuecomment-1821919107
> @VladaZakharova @potiuk please correct me if i'm wrong but i don't see how
you can specify a list of strings when configuring a connection through the
Airflow UI for a connection field that is a string. this change allows the user
to specify a comma-separated list of strings in the Airflow UI for the
`impersonation_chain` argument, and upon instantiating the hook we detect a
comma-separated string for that argument and convert it to a list of strings.
Correct me if I am wrong - but I think you cannot do it with your change
either? The impersonation_chain can only be passed by Hook / Operator init
parameters, not through connection extra (which is what you are suggesting) ?
And it is defined like this everywhere in those:
```
impersonation_chain: str | Sequence[str] | None = None,
```
But.maybe I am missing something ??
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
