TheProjecter / sardine

Automatically exported from code.google.com/p/sardine
0 stars 0 forks source link

HTTP errors should be stated in exception message #47

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
currently the HTTP error code is not mentioned and one has to have a look
at the server side logs (if one is able to). This is really bad for
debugging and monitoring :-(

e.g.

satisfying: "Caused by: com.googlecode.sardine.util.SardineException: The
server has returned an HTTP error: 404"

better: "Caused by: com.googlecode.sardine.util.SardineException: The
server has returned an HTTP error: 404: page not found"

instead of: "Caused by: com.googlecode.sardine.util.SardineException: The
server has returned an HTTP error"

a detailed error example:
Caused by: com.googlecode.sardine.util.SardineException: The server has
returned an HTTP error
    at com.googlecode.sardine.SardineImpl.createDirectory(SardineImpl.java:332)
    at
aspicon.obs2conf.model.InfrastructureManager.createCustomerInfrastructure(Infras
tructureManager.java:79)
    ... 52 more

other conditions could be that on directory creation a dir may already
exist etc.

Original issue reported on code.google.com by and.diet...@gmail.com on 7 Apr 2010 at 9:49

GoogleCodeExporter commented 9 years ago
fixed in r117

Original comment by latch...@gmail.com on 7 Apr 2010 at 6:40

GoogleCodeExporter commented 9 years ago

Original comment by latch...@gmail.com on 7 Apr 2010 at 11:24