[GitHub] [airflow] jtommi commented on issue #27328: SFTPOperator throws object of type 'PlainXComArg' has no len() when using with Taskflow API

[GitBox <gi...@apache.org>]

jtommi commented on issue #27328:
URL: https://github.com/apache/airflow/issues/27328#issuecomment-1294988085

   @uranusjr Thanks for the initial direction to fixing this.
   
   I drafted #27346.
   I think the initial bug is fixed, but my test that checks for compatibility is failing.
   https://github.com/apache/airflow/blob/c84831151a1e2be018f4280180e3002bcf3153f4/tests/providers/sftp/operators/test_sftp.py#L413
   
   The reason is `expected str, bytes or os.PathLike object, not PlainXComArg` on https://github.com/apache/airflow/blob/c84831151a1e2be018f4280180e3002bcf3153f4/airflow/providers/sftp/operators/sftp.py#L173
   
   The thing is that I don't understand why the code worked with version 2 of the provider package (I actually didn't check with version 4, but I didn't get import errors).
   The only change from previous versions of the code is that before it did `os.path.dirname(self.local_filepath)` and now  `os.path.dirname(local_filepath)`, but I couldn't find any code that converted a PlainXComArg into a string.


-- 
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: commits-unsubscribe@airflow.apache.org

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