Hrm... maybe having two bugs open wasn't such a great idea. Also maybe this is why smaller changes are better?
So what we *need* is to be able to take some Kilo+ sha, and Al's latest patch from lp bug #1453948 + some patch here that closes all of our issues (#1 PUT tempurl can not create a DLO; #2 cross container DLO 401's with container tempurl).
I found
+ req = Request(req.environ.copy())
in patch number #56 - but it seems like everyone else is building on the hack in #57 that doesn't do sam's info_cache thing (i.e. #60, #67 and #69) and instead just calls that _set_info method again - but was mysteriously missing the environ.copy() line in my original version (i blame a git fail on my part)?
So, if everyone is cool with that general idea of patch #57 - i'll try to wade through the follow on patches and post here with what we need.
Hrm... maybe having two bugs open wasn't such a great idea. Also maybe this is why smaller changes are better?
So what we *need* is to be able to take some Kilo+ sha, and Al's latest patch from lp bug #1453948 + some patch here that closes all of our issues (#1 PUT tempurl can not create a DLO; #2 cross container DLO 401's with container tempurl).
I found
+ req = Request( req.environ. copy())
in patch number #56 - but it seems like everyone else is building on the hack in #57 that doesn't do sam's info_cache thing (i.e. #60, #67 and #69) and instead just calls that _set_info method again - but was mysteriously missing the environ.copy() line in my original version (i blame a git fail on my part)?
So, if everyone is cool with that general idea of patch #57 - i'll try to wade through the follow on patches and post here with what we need.