Add more info to errors

[lorensr]

For instance, all I see from this line

meteor-slingshot/lib/upload.js

Line 163 in 73055a7

return new Meteor.Error(xhr.statusText + " - " + xhr.status,

is

 Error: Failed to upload file to cloud storage [Bad Request - 400]
    at getError (upload.js:163)
    at XMLHttpRequest.<anonymous> (upload.js:176)

and I have no idea what went wrong. But the xhr does have more informative fields:

Thanks for this great package!

[gsuess]

This could be tricky as each storage service has a different way of structuring their error documents and the client library has no way of know how to handle it.

What can be done however is to attach the parsed raw document that the server responds with to the error object.

[MichaelJCole]

Error: Failed to upload file to cloud storage [ - 0] - seems to indicate a CORS issue

[wiserweb]

Consider this adhoc documentation

Error: Failed to upload file to cloud storage [ - 0] may also indicate an incorrect Metadata Key that is set by specifying a key via the X-Account-Meta-Temp-Url-Key header.

Note that the X-Account-Meta-Temp-Url-Key is set at the Project level only and not for each individual container.

Any class 200 status code indicates success.

Here's a one liner to set the metadata key using curl.

curl -I -X POST -H 'X-Auth-Token: your-auth-token' -H 'X-Account-Meta-Temp-Url-Key: your-key' https://storage.clouddrive.com/v1/AUTH_your-tenantid-goes-here

and don't forget to set CORS - and don't use * origin in prod..

curl -I -X POST -H 'X-Auth-Token: your-auth-token' -H 'X-Container-Meta-Access-Control-Allow-Origin: *' -H 'X-Container-Meta-Access-Expose-Headers: etag location x-timestamp x-trans-id Access-Control-Allow-Origin' https://storage.clouddrive.com/v1/AUTH_your-tenantid-goes-here

These two curl commands should resolve the error message.

[Floriferous]

It's a shame that it's really hard to override individual functions in this package. the getError method for example is deeply nested in a closure. Cloning the package in your own repo appears to be the simplest fix for this.