From: Brian Warner Date: Thu, 6 Sep 2007 00:12:27 +0000 (-0700) Subject: webish: implement 'PUT /uri' (to create anonymous files) X-Git-Url: https://git.rkrishnan.org/somewhere?a=commitdiff_plain;h=db3f11648cdfdb57542e5e2c75cd8aca81d29dfc;p=tahoe-lafs%2Ftahoe-lafs.git webish: implement 'PUT /uri' (to create anonymous files) --- diff --git a/src/allmydata/test/test_web.py b/src/allmydata/test/test_web.py index d9e61384..c691a682 100644 --- a/src/allmydata/test/test_web.py +++ b/src/allmydata/test/test_web.py @@ -62,8 +62,9 @@ def make_newuri(data): class MyUploader(service.Service): implements(interfaces.IUploader) name = "uploader" - def __init__(self, files): + def __init__(self, files, nodes): self.files = files + self.nodes = nodes def upload(self, uploadable): d = uploadable.get_size() @@ -72,7 +73,9 @@ class MyUploader(service.Service): def _got_data(data): newuri = make_newuri(data) self.files[newuri] = data + self.nodes[newuri] = MyFileNode(newuri, self.parent) uploadable.close() + return newuri d.addCallback(_got_data) return d @@ -173,7 +176,7 @@ class WebMixin(object): dl = MyDownloader(self.files) dl.setServiceParent(self.s) - ul = MyUploader(self.files) + ul = MyUploader(self.files, self.nodes) ul.setServiceParent(self.s) v.public_root = self.makedir() @@ -1389,6 +1392,20 @@ class Web(WebMixin, unittest.TestCase): "to not replace it") return d + def test_PUT_NEWFILE_URI(self): + file_contents = "New file contents here\n" + d = self.PUT("/uri", file_contents) + def _check(uri): + self.failUnless(uri in self.files) + self.failUnless(uri in self.nodes) + self.failUnlessEqual(self.files[uri], file_contents) + return self.GET("/uri/%s" % uri.replace("/","!")) + d.addCallback(_check) + def _check2(res): + self.failUnlessEqual(res, file_contents) + d.addCallback(_check2) + return d + def test_XMLRPC(self): raise unittest.SkipTest("not yet") pass diff --git a/src/allmydata/webish.py b/src/allmydata/webish.py index 2801270d..6d1fa382 100644 --- a/src/allmydata/webish.py +++ b/src/allmydata/webish.py @@ -1019,6 +1019,36 @@ class VDrive(rend.Page): d.addErrback(_trap_KeyError) return d +class URIPUTHandler(rend.Page): + def renderHTTP(self, ctx): + req = inevow.IRequest(ctx) + assert req.method == "PUT" + + t = "" + if "t" in req.args: + t = req.args["t"][0] + + if t == "": + # "PUT /uri", to create an unlinked file. This is like PUT but + # without the associated set_uri. + uploadable = upload.FileHandle(req.content) + uploader = IClient(ctx).getServiceNamed("uploader") + d = uploader.upload(uploadable) + # that fires with the URI of the new file + return d + + if t == "mkdir": + # "PUT /uri?t=mkdir", to create an unlinked directory. We use the + # public vdriveserver to create the dirnode. + vdrive = IClient(ctx).getServiceNamed("vdrive") + d = vdrive.create_directory() + d.addCallback(lambda dirnode: dirnode.get_uri()) + return d + + req.setResponseCode(http.BAD_REQUEST) + req.setHeader("content-type", "text/plain") + return "/uri only accepts PUT" + class Root(rend.Page): @@ -1049,6 +1079,11 @@ class Root(rend.Page): there = there.clear("uri") there = there.child("uri").child(uri) return there, () + if len(segments) == 1 and req.method == "PUT": + # /uri + # either "PUT /uri" to create an unlinked file, or + # "PUT /uri?t=mkdir" to create an unlinked directory + return URIPUTHandler(), () if len(segments) < 2: return rend.NotFound uri = segments[1].replace("!", "/")