Comment 23 for bug 1324592

Revision history for this message
Brant Knudson (blk-u) wrote : Re: Trust scope can be circumvented by chaining trusts

A couple of comments on the patch in comment 21 --

1) in test_v3_oauth1.py at line 556, a file is created. The file should be deleted with self.addCleanup(os.remove, self.tmpfilename)
2) same file, at line 559 is extra-indented (8 chars rather than 4 as expected)

Other than that looks good to me.