My favorites | Sign in
Logo
          
New issue | Search
for
| Advanced search | Search tips
Issue 721: Diff Viewer SCMError - callback_get_login required
1 person starred this issue and may be notified of changes. Back to list
Status:  Fixed
Owner:  ----
Closed:  Feb 2009
Type-Defect
Priority-Medium
Component-SCMTools


Sign in to add a comment
 
Reported by python.technofreak, Oct 31, 2008
*NOTE: Do not post confidential information in this bug report.*

What's the URL of the page containing the problem?

http://example.com/r/5/diff/#index_header

What steps will reproduce the problem?
1. Add new review requests
2. Add a diff file (svn diff) 
3. Click on View Diff in Review Requests page.

What is the expected output? What do you see instead?

I get the following error,

callback_get_login required

Traceback (most recent call last):
  File "/home/parthan/reviewboard/diffviewer/views.py", line 86, in view_diff
    files = get_diff_files(diffset, None, interdiffset, highlighting)
  File "/home/parthan/reviewboard/diffviewer/diffutils.py", line 654, in
get_diff_files
    enable_syntax_highlighting)
  File "/home/parthan/reviewboard/diffviewer/diffutils.py", line 573, in
generate_files
    large_data=True)
  File "/home/parthan/reviewboard/djblets/util/misc.py", line 142, in
cache_memoize
    data = lookup_callable()
  File "/home/parthan/reviewboard/diffviewer/diffutils.py", line 572, in
<lambda>
    enable_syntax_highlighting),
  File "/home/parthan/reviewboard/diffviewer/diffutils.py", line 338, in
get_chunks
    old = get_original_file(filediff)
  File "/home/parthan/reviewboard/diffviewer/diffutils.py", line 245, in
get_original_file
    large_data=True)[0]
  File "/home/parthan/reviewboard/djblets/util/misc.py", line 142, in
cache_memoize
    data = lookup_callable()
  File "/home/parthan/reviewboard/diffviewer/diffutils.py", line 244, in
<lambda>
    data = cache_memoize(key, lambda: [fetch_file(file, revision)],
  File "/home/parthan/reviewboard/diffviewer/diffutils.py", line 224, in
fetch_file
    data = tool.get_file(file, revision)
  File "/home/parthan/reviewboard/scmtools/svn.py", line 115, in get_file
    raise SCMError(e)
SCMError: callback_get_login required


What operating system are you using? What browser?

Linux, Firefox3.0

Please provide any additional information below.

Am attaching the diff file I use, taken from the root directory of my svn
repository

printproposal.diff
38.7 KB   View   Download
Comment 1 by chipx86, Nov 25, 2008
It sounds like the server is expecting login info in order to view anything. Do you
have this configured in your repository entry in the database?
Summary: Diff Viewer SCMError - callback_get_login required
Status: NeedInfo
Labels: Component-SCMTools
Comment 2 by trowbrds, Dec 27, 2008
Assuming that this was the problem, since no additional information was provided. If
this is not the case, please add a comment and we'll reopen the issue.
Status: Incomplete
Comment 3 by dgfitch, Jan 20, 2009
This error also occurs if you HAVE entered login information, and then it starts
failing for some reason (expired password, in my case) -- the callback_get_login is
pretty opaque to me as an idiot administrator, so maybe it just needs a better message.
Comment 4 by chipx86, Jan 20, 2009
The error is definitely opaque, and not sent by us. Do you know what version of PySVN
you have?
Status: NeedInfo
Comment 5 by trowbrds, Feb 09, 2009
Fixed in SVN r1771.
Status: Fixed
Sign in to add a comment

Powered by Google Project Hosting