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

Unit mismatch in FileSizeExceeded error_message #571

Closed huard closed 3 years ago

huard commented 3 years ago

Description

The file property of the URLHandler object checks if the data size is larger than the maximum data size. When it exceeds the maximum value, it raises an error: https://github.com/geopython/pywps/blob/20e1e254a3f7914e555fa89f363d1f6eb5f3895c/pywps/inout/basic.py#L458

The value is in units of bytes, but the message says megabytes.

Also, it would be useful if the message included the actual size of the data.

Environment

Steps to Reproduce

Additional Information

huard commented 3 years ago

Also, why is the FileSizeExceeded exception in the try statement replaced by NoApplicableCode?

cehbrecht commented 3 years ago

I run into the same issue: https://github.com/roocs/rook/issues/139#issuecomment-797554832

cehbrecht commented 3 years ago

should be fixed by PRs #580 and #581.