potiuk commented on issue #18111: URL: https://github.com/apache/airflow/issues/18111#issuecomment-937704490
> IMO, It's not reasonable for us to swallow the permission error. Fully agree with @xinbinhuang. The 404 is explicit "I have permissions, but the file does not exist". This is entirely different than "The file exist but I have no permissions". IMHO Replace=True does exactly what it should do - replaces stuff regardless if the file is there or whether you even have permission to list them. Replace = false should fail if you cannot check if the file is there - even if you have no list permission - generally if you have not enough permissions to check if the file is there, and if you want to use "replace = False", I'd expect to get exactly what you get - permission error (precisely because you have no permissions to check it). So for me this works as intended and expected. I will convert it into discussion if you want to further discuss it and close the PR, but I think it's just wrong to add 403 here. -- 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]
