[Opensim-dev] Error detection when storing an asset

[Oren Hurvitz]

My fix has two parts.

The first is that the Store() operation needs to understand failures
correctly. There has been a consensus that it should, so I'll add that to
Git.

The second is that MakeRequest() should propagate exceptions, instead of
just returning null (which is what it does now). So far there have been 3
votes for this (me, Mike Chase, and Justin) and 2 against (Melanie, Diva).
That's very close; does anyone else want to make their position known?

Next, I see that there's confusion in this discussion about what happens in
Store() if MakeRequest() throws an exception. And the answer is, nothing
will be different, because Store() already correctly catches exceptions.
That is precisely how it should work: the low-level communications system
reports when it has failed, and higher levels (that know the business value
of the call) decide how to handle it. However, MakeRequest() is called from
other places as well and they might need to be changed to handle exceptions
better.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://opensimulator.org/pipermail/opensim-dev/attachments/20140419/f98d0519/attachment-0001.html>