subprocess.popen throws "File name too long" on Mac

[jseabold]

If I pass a longer query through, I get an OSError: [Errno 63] File name too long: <redacted bcp command>. Adding a shell=True to subprocess.run fixes things, but I don't recall all the details of subprocess to remember if this is a general solution.

[jseabold]

Using the MSSQL driver.

[mikealfare]

I'll have to research that one more. My understanding was that setting shell=True was a potential vulnerability, but don't hold me to that.

[mikealfare]

It looks like it can open security vulnerabilities, so I'd rather not introduce that. Is it possible to move the default directory to a higher level directory? When I write out to %USERPROFILE% (the default) in Windows, it doesn't seem to have a problem.

[jseabold]

There's an alternative syntax that relies on using a list of strings that I've used in the past without the shell option, but I don't remember the details without reading for a while.

…

On Fri, Apr 3, 2020, 1:30 PM Mike ***@***.***> wrote: It looks like it can open security vulnerabilities, so I'd rather not introduce that. Is it possible to move the default directory to a higher level directory? When I write out to %USERPROFILE% (the default) in Windows, it doesn't seem to have a problem. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#18 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AACIJZETMLTXMXSVJRYRBNLRKYTMXANCNFSM4L4J2Q2Q> .