3313: HTTP authentication is required, but cannot be used with --diff-filename=-

weber*****@gmai***** (Google Code) (Is this you? Claim this profile.)
david
 david
Oct. 30, 2014 
Using rbt 0.6, I'm trying to use "rbt post" by piping in a svn diff. I'm getting this error: "HTTP authentication is required, but cannot be used with --diff-filename=-". My RB server doesn't require HTTP authentication, so I'm not sure why this happens. The same request works with rbt 0.5.1.


What version are you running? 0.5.3 - 0.6

What's the URL of the page containing the problem? N/A

What steps will reproduce the problem?
1. Run the following command:
svn diff [...] | rbt post --repository-url=https://svn-repos-url --username=rbuser --password=rbpass --submit-as=other-user --server=http://reviewboard-url --diff-filename=- --basedir=/basedir --summary='test' --description='test' -d

What is the expected output? What do you see instead?
I see the following:
>>> RBTools 0.6
>>> Python 2.7.3 (default, Mar 11 2013, 22:38:13) 
[GCC 4.4.7 20120313 (Red Hat 4.4.7-3)]
>>> Running on Linux-2.6.32-358.18.1.el6.x86_64-x86_64-with-redhat-6.4-Santiago
>>> Home = /home/myuser
>>> Current directory = /home/myuser
>>> Checking for a Subversion repository...
>>> Running: svn info https://svn-repos-url --non-interactive
>>> Running: diff --version
>>> repository info: Path: https://svn-repos-url, Base path: /, Supports changesets: False
>>> Making HTTP GET request to http://reviewboard-url/api/
HTTP authentication is required, but cannot be used with --diff-filename=-

What operating system are you using? What browser?
RHEL 6.4.

Please provide any additional information below.
david
#1 david 
This should only happen if there's some sort of authentication failure. If your cookie or credentials on the command line are correct, it should work fine.
  • +NotABug
#2 weber*****@gmai***** (Google Code) (Is this you? Claim this profile.) 
I haven't had a chance to re-test it in the latest version, but I don't think that was the case.

I originally reported the issue here:
https://groups.google.com/forum/#!topic/reviewboard/-m1fhSLYspg
and the response from Christian Hammond was:

"This error is a bit unclear, but it's referring to the web API's HTTP authentication.

Some of this code changed, and it looks like the problem is that we're doing the check regarding --diff-filename=- too early, preventing the provided username/password from working."
david
#3 david 
Aha, thanks for the link.
  • -NotABug
    +New
david
#4 david
  • -New
    +PendingReview
  • +Project-RBTools
    +Component-RBTools
  • +david
david
#5 david 
Fixed in release-0.6.x (77da95b). Thanks!
  • -PendingReview
    +Fixed