No, please go ahead. DatasetAccessors was next on my hit list for adding auth support but if you want to go ahead and refactor a bit to make this easier to do please feel free
Rob On 6/28/13 11:10 AM, "Andy Seaborne" <[email protected]> wrote: >Rob, > >I was looking at cleaning up DatasetAccessors for HTTP to use HttpOp. > >In looking at HttpOp, the handling of results via a >Map<String, HttpResponseHandler> is getting in the way of leaving the >dispatch on content type to the RIOT registries. > >Instead, I'd like to have one HttpResponseHandler, and if theer is a >need to switch on content type, then it's a HttpResponseHandler that >dispatches to other HttpResponseHandlers. > >The interface HttpResponseHandler will change - remove passing in the >content type (does not work properly for multipart). > >This shouldn't get in the way of your authentication changes (that's on >the way out, HttpResponseHandler is on the way back in again). > >As it's in the same sort of area, I thought I'd explicitly mention it. > >If it's a bit close to changes you have in progress, I can hold back and >wait. Just shout. > > Andy
