geopython / pywps

PyWPS is an implementation of the Web Processing Service standard from the Open Geospatial Consortium. PyWPS is written in Python.
https://pywps.org
MIT License
177 stars 117 forks source link

pywps sets 444 on output file #37

Closed landam closed 8 years ago

landam commented 9 years ago

Hi,

PyWPS 3.x changes permission on referenced output file, see [1]. In the result the user who runs the process (usually www-data) cannot delete this file.

 -r--r--r-- 2 www-data www-data 57590190 Aug 25 14:22 /tmp/basin_20252/basin.zip

Why PyWPS behaves like that? Thanks, Martin

[1] https://github.com/geopython/PyWPS/blob/master/pywps/Wps/Execute/__init__.py#L1135

cehbrecht commented 9 years ago

probably that goes on my account ... not sure why i'm explicitly setting the permissions but probably there were reasons ...

I would change the permissions to 644. I've no commit rights so i can i only send a pull request ...

Cheers, Carsten

landam commented 9 years ago

Thanks for quick feedback!

tomkralidis commented 8 years ago

Can we close this issue?

cehbrecht commented 8 years ago

this is fixed with patch #38 which is already merged.