Re: Issue 1874 in reviewboard: post-review does not properly parse diffs between commits in a branch on a git-svn repository

2013-08-10 Thread reviewboard

Updates:
Status: Fixed

Comment #8 on issue 1874 by trowb...@gmail.com: post-review does not  
properly parse diffs between commits in a branch on a git-svn repository

http://code.google.com/p/reviewboard/issues/detail?id=1874

Fixed in rbtools master (dc0b739). Thanks!

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


Re: Issue 2322 in reviewboard: Unable to post reviews with post-review (0.3.4): can not log in into Reviewboard (1.6.1)

2013-08-10 Thread reviewboard

Updates:
Status: SetupIssue

Comment #23 on issue 2322 by trowb...@gmail.com: Unable to post reviews  
with post-review (0.3.4): can not log in into Reviewboard (1.6.1)

http://code.google.com/p/reviewboard/issues/detail?id=2322

Digging through the mailing list, this seems to be when combining ldap +  
fastcgi. Since there's really nothing we can do about the fastcgi  
interface, I'm going to change the docs to more heavily recommend mod_wsgi  
and cite this issue specifically.


--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.




Re: Issue 2632 in reviewboard: Cannot upload a diff created via svn

2013-08-10 Thread reviewboard

Updates:
Status: Fixed

Comment #9 on issue 2632 by trowb...@gmail.com: Cannot upload a diff  
created via svn

http://code.google.com/p/reviewboard/issues/detail?id=2632

Something about your svn binary is pretty weird (We've only seen "(revision  
)" from IntelliJ IDEA before). That said, I think we can work around it.  
I've pushed a fix for this to master (95cf9c1). It will ship in 1.8.0.


--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


Re: Issue 2661 in reviewboard: Hitting enter in a text field causes the page to scroll down

2013-08-10 Thread reviewboard

Updates:
Status: UnableToReproduce

Comment #5 on issue 2661 by trowb...@gmail.com: Hitting enter in a text  
field causes the page to scroll down

http://code.google.com/p/reviewboard/issues/detail?id=2661

(No comment was entered for this change.)

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.




Re: Issue 2829 in reviewboard: Diff generation hangs when trying to "Edit Review"

2013-08-10 Thread reviewboard

Updates:
Status: Fixed

Comment #6 on issue 2829 by trowb...@gmail.com: Diff generation hangs when  
trying to "Edit Review"

http://code.google.com/p/reviewboard/issues/detail?id=2829

This was fixed in djblets (1b79a0c1).

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.




Issue 3052 in reviewboard: security vulnerability: python injection

2013-08-10 Thread reviewboard

Status: New
Owner: 
Labels: Type-Defect Priority-Medium

New issue 3052 by yuzi...@gmail.com: security vulnerability: python  
injection

http://code.google.com/p/reviewboard/issues/detail?id=3052

*** For customer support, please post to reviewbo...@googlegroups.com
*** If you have a patch, please submit it to
http://reviews.reviewboard.org/
***
*** Do not post confidential information in this bug report!

What version are you running?
1.7.11


What steps will reproduce the problem?

import rbtools.api.client
r = rbtools.api.client.RBClient('http://my-reviewboard',  
username='my_username', password='my_password')

r.get_root().get_user(username='quit()')



What is the expected output?

{u'stat': u'fail', u'err': {u'msg': u'Object does not exist', u'code': 100}}
'{"stat": "fail", "err": {"msg": "Object does not exist", "code": 100}}'
Traceback (most recent call last):
  File "", line 1, in 
  File "/Library/Python/2.7/site-packages/rbtools/api/resource.py", line  
441, in 

self._get_template_request(url, **kwargs)))
  File "/Library/Python/2.7/site-packages/rbtools/api/decorators.py", line  
24, in request_method

*args, **kwargs)
  File "/Library/Python/2.7/site-packages/rbtools/api/transport/sync.py",  
line 54, in execute_request_method

return self._execute_request(request)
  File "/Library/Python/2.7/site-packages/rbtools/api/transport/sync.py",  
line 63, in _execute_request

rsp = self.server.make_request(request)
  File "/Library/Python/2.7/site-packages/rbtools/api/request.py", line  
414, in make_request

self.process_error(e.code, e.read())
  File "/Library/Python/2.7/site-packages/rbtools/api/request.py", line  
387, in process_error

rsp['err']['msg'])
rbtools.api.errors.APIError: Object does not exist (HTTP 404, API Error 100)


What do you see instead?

'\n\n\n\n  http-equiv="content-type" content="text/html; charset=utf-8">\n   
Page not found at /api/users/quit()/\n  content="NONE,NOARCHIVE">\n  \nhtml * {  
padding:0; margin:0; }\nbody * { padding:10px 20px; }\nbody * * {  
padding:0; }\nbody { font:small sans-serif; background:#eee; }\n 
body>div { border-bottom:1px solid #ddd; }\nh1 { font-weight:normal;  
margin-bottom:.4em; }\nh1 span { font-size:60%; color:#666;  
font-weight:normal; }\ntable { border:none; border-collapse: collapse;  
width:100%; }\ntd, th { vertical-align:top; padding:2px 3px; }\nth  
{ width:12em; text-align:right; color:#666; padding-right:.5em; }\n 
#info { background:#f6f6f6; }\n#info ol { margin: 0.5em 4em; }\n 
#info ol li { font-family: monospace; }\n#summary { background: #ffc;  
}\n#explanation { background:#eee; border-bottom: 0px none; }\n   
\n\n\n  \nPage not found  
(404)\n\n  \n 
Request Method:\nGET\n  \n   
\nRequest URL:\n   
http://purpleslam-reviewboard.apple.com/api/users/quit()/\n   
\n\n  \n  \n\n  \n   
Using the URLconf defined in djblets.util.rooturl,\n   
Django tried these URL patterns, in this order:\n  \n   
\n\n  \n\n 
^\n\n\n 
^admin/extensions/\n\n\n   
\n\n  \n\n 
^\n\n\n^admin/\n 
\n\n  \n\n  \n 
\n^\n\n\n 
^static\\/(?P.*)$\n\n\n   
\n\n  \n\n 
^\n\n\n 
^media\\/(?P.*)$\n\n\n   
\n\n  \n\n 
^\n\n\n 
^js-tests/$\n\n\n  \n 
\n  \n\n^\n 
\n\n^account/\n\n 
\n  \n\n  \n\n 
^\n\n\n 
^s/(?P[A-Za-z0-9\\-_.]+)/\n 
\n\n  \n\n  \n 
\n^\n\n\n 
^$\n[name=\'root\']\n\n  \n 
\n  \n\n^\n 
\n\n^api/\n\n 
\n^$\n[name=\'root-resource\']\n 
\n  \n\n  \n\n 
^\n\n\n^api/\n 
\n\n^default-re

Re: Issue 3052 in reviewboard: security vulnerability: python injection


Updates:
Status: NeedInfo

Comment #1 on issue 3052 by chip...@gmail.com: security vulnerability:  
python injection

http://code.google.com/p/reviewboard/issues/detail?id=3052

Thanks for reporting this.

I'm not able to reproduce it. I set up an environment with 1.7.11 and tried  
the repro case you provided. I haven't been able to cause this.


Looking at your log output, the 404 result you provided shows a result from  
Django. The Django 404 happens because the URLs registered don't allow  
parens, so we never get to a point where we reach any API handler  
for "quit()" that can throw an API version of a 404, instead throwing only  
a standard Django 404.


So all that looks correct. Well, "correct." We should probably have some  
generic thing on /api/* that throws a 404 if nothing else matches.


Now, you end up with an Operation Timed Out. That's very strange. What  
happens when connecting from a web browser?


Also, what version of RBTools are you using?

Does that query for get_review_requests work before doing the  
get_user('quit()') ?


--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.




Re: Issue 3052 in reviewboard: Useless 404 payload for unhandled URLs


Updates:
Summary: Useless 404 payload for unhandled URLs
Labels: Component-API

Comment #2 on issue 3052 by chip...@gmail.com: Useless 404 payload for  
unhandled URLs

http://code.google.com/p/reviewboard/issues/detail?id=3052

(No comment was entered for this change.)

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.