SFTP Provider: Fix default folder permissions #26593
Conversation
|
Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contribution Guide (https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst)
|
|
I don’t think this is a good fix. Instead we should change the default to |
|
@uranusjr I can change the default value to |
|
Change them both to use octal. Using decimal is a bug in the first place. (We can keep the |
0d6cc20 to
f8041bc
Compare
86446b8 to
f6db51f
Compare
f6db51f to
bf20f85
Compare
|
cool ! |
Relates #26226
Fixing default mode when creating a directory with paramiko and convert decimal mode to octal mode (paramiko documention)