tempfile._TemporaryFileWrapper not available under Google Appengine

[cdman]

Google appengine replaces the tempfile module with a custom one (since you can't write to the FS under "traditional" GAE): https://code.google.com/p/googleappengine/source/browse/trunk/python/google/appengine/dist/tempfile.py

However this module lacks the _TemporaryFileWrapper on which utils.DownloadedFile is based: https://github.com/core-api/python-client/blob/master/coreapi/utils.py#L119

Given that _TemporaryFileWrapper is an undocumented API (class) IMHO the dependency on it should be removed.

[tomchristie]

👍 Good call.

[tomchristie]

Worth noting that even when we do resolve this, the DownloadCodec still wouldn't work under GAE, but that's much less of a concern. (Ie. at least everything else will work.)

[canthonyscott]

Is there any workaround for this? I am trying to deploy an app to the Google App Engine standard python environment and this is causing Django to crash on load.

For the record everything works fine in the GAE flexible environment, but not in the standard environment.

[cdman]

It seems to me that rather than using _TemporaryFileWrapper, we should use NamedTemporaryFile - which is a function, not a class. This would "work" on GAE in the sense that the module would load (but calling it would still raise a NotImplementedError).

[tomchristie]

Hoping to look into resolving this sometime over the next few days.
Pull requests welcome in the meantime.

[tomchristie]

Okay, that should be resolved now. If anyone wants to confirm that'd be fab.