Merge lp:~jml/canonical-identity-provider/clean-up-temp into lp:canonical-identity-provider/release
Proposed by
Jonathan Lange
Status: | Merged |
---|---|
Approved by: | Jonathan Lange |
Approved revision: | no longer in the source branch. |
Merged at revision: | 616 |
Proposed branch: | lp:~jml/canonical-identity-provider/clean-up-temp |
Merge into: | lp:canonical-identity-provider/release |
Diff against target: |
70 lines (+41/-0) 2 files modified
scripts/run-acceptance-tests (+20/-0) tarmac_tests.sh (+21/-0) |
To merge this branch: | bzr merge lp:~jml/canonical-identity-provider/clean-up-temp |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Natalia Bidart (community) | Approve | ||
James Westby (community) | Approve | ||
Review via email: mp+143537@code.launchpad.net |
Commit message
Control the creation and deletion of /tmp/djopenid_
Description of the change
Changes the tarmac_tests.sh and run-acceptance-
the creation and deletion of /tmp/djopenid_
by webui for storing openid consumer information.
The problem is that it gets created implicitly when webui gets a startOpenID
or finishOpenID request, and when it does, it is owned by whichever user
does this.
Thus, this changes the test scripts at least to make sure it's owned by the
'tarmac' group and that anyone in the group can at least clear out the
contents.
To post a comment you must log in.
14:31 < nessita> jml: I'm looking at https:/ /code.launchpad .net/~jml/ canonical- identity- provider/ clean-up- temp/+merge/ 143537, I was wondering if we can somehow avoid the duplication
of the code