Plumb problems with p4 repos through the webapi.
Review Request #2454 — Created July 7, 2011 and submitted
Plumb problems with p4 repos through the webapi. Right now, if someone tries to post-review against a p4 server with a broken setup (wrong auth, wrong P4PORT, etc), they'll get a generic 500 error and a traceback will be sent to the server admins. We can do better by translating P4Errors into SCMErrors, and then translating SCMErrors into webapi error codes. This also adds validation of the perforce repository to the creation form, so hopefully these errors will be caught sooner.
Newly added unit tests.
| Description | From | Last Updated |
|---|---|---|
|
I'd either change the order, or update sshutils.py to be more specific about the key, because it assumes the key … |
chipx86
|
Review request changed
- Change Summary:
-
Fix AuthenticationError args, add validation at the admin form step.
- Description:
-
Plumb problems with p4 repos through the webapi.
Right now, if someone tries to post-review against a p4 server with a broken
setup (wrong auth, wrong P4PORT, etc), they'll get a generic 500 error and a traceback will be sent to the server admins. We can do better by translating P4Errors into SCMErrors, and then translating SCMErrors into webapi error codes. + + This also adds validation of the perforce repository to the creation form, so
+ hopefully these errors will be caught sooner.