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

use of logging is imperfect #28

Closed doutriaux1 closed 8 years ago

doutriaux1 commented 9 years ago

logging is made via

logging.error("bla")

I would suggest adding the capability to use "loggers" rather than logging directly.

I'm trying to use pywps with multiple instance of pywps on the server and it gets confised

I would recommend replacing the logFile in the init with a logger keyword that would the be used to log, allowing to log simultaneously to multiple logs.

A string with the logger name is actually sufficient you can retrieve the logger via

logging.getLogger(name)

Let me know if this makes sense. Thanks for the great work!

cehbrecht commented 9 years ago

... when starting with PyWPS i also noticed that pywps logging with self.log() didn't work for async processes. As a workaround i was using my own logging for my processes which also uses concurrent-logging:

https://github.com/bird-house/malleefowl/blob/master/malleefowl/wpslogging.py

But it would be better if we improve logging in PyWPS itself.

jachym commented 9 years ago

Hi,

nobody is working on this. We can implement logger in pywps4 and ask someone to work on this in pywps-3 as for example GSoC project ...

J

Wed Jan 28 2015 at 18:44:04 odesílatel MacPingu notifications@github.com napsal:

... when starting with PyWPS i also noticed that pywps logging with self.log() didn't work for async processes. As a workaround i was using my own logging for my processes which also uses concurrent-logging:

https://github.com/bird-house/malleefowl/blob/master/malleefowl/wpslogging.py

But it would be better if we improve logging in PyWPS itself.

Reply to this email directly or view it on GitHub https://github.com/geopython/PyWPS/issues/28#issuecomment-71879919.

tomkralidis commented 8 years ago

@doutriaux1: fixed in master. Feel free to reopen this issue if there are problems. Thanks!