Re: Active Directory Authentication with TLS

2012-01-18 Thread Thilo-Alexander Ginkel
On Tue, Jan 17, 2012 at 21:09, Brad Hards br...@frogmouth.net wrote: On Wednesday 18 January 2012 02:08:50 Thilo-Alexander Ginkel wrote: When disabling TLS, everything works like expected. Are you just enabling / disabling TLS? Yes. Any ideas? Do I need to register the AD DC's CA certificate

Active Directory Authentication with TLS

2012-01-17 Thread Thilo-Alexander Ginkel
Hi there, we are currently trying to switch Review Board from the Standard Registration mechanism to Active Directory Authentication. Unfortunately, this only seems to work as long as TLS is not enabled. The Domain Controller seems to support TLS and according to tcpdump I can see some data being

Re: Unable to upload screenshots

2011-02-16 Thread Thilo-Alexander Ginkel
On Mon, Feb 14, 2011 at 23:01, Christian Hammond chip...@chipx86.com wrote: I don't believe they're stored temporarily anywhere. If so, it's internal to Django and I don't know it off-hand. The only thing I've ever seen permission-wise is when there's a directory within media/uploaded that

Re: How to submit review for merging branch to trunk?

2010-10-19 Thread Thilo-Alexander Ginkel
On Tue, Oct 19, 2010 at 18:55, s...@pobox.com wrote: (Using Subversion...) I have some code checked in on a branch.  Before merging back to trunk I would like to create a review request for the merge.  It's not clear at all to me how I would do that with post-review (or even using the form

Re: Review performance issue

2010-10-15 Thread Thilo-Alexander Ginkel
On Fri, Oct 15, 2010 at 09:22, Mohan mohan...@gmail.com wrote: Only RB is running on the server. when i did 'ps' it shows the following apache instances. [r...@rboard log]# ps -ef | grep httpd apache    1279 22272  0 01:51 ?        00:00:03 /usr/sbin/httpd apache   12843 22272  0 07:29 ?    

Re: Review performance issue

2010-10-14 Thread Thilo-Alexander Ginkel
On Thu, Oct 14, 2010 at 07:53, Mohan mohan...@gmail.com wrote: I have noticed that high memory usage on this machine. [r...@rboard ~]# free                   total        used               free     shared buffers     cached Mem:       5760760    5726016      34744          0        220

Re: RB 1.0.9 - 1.5 upgrade woes

2010-10-14 Thread Thilo-Alexander Ginkel
Ginkel th...@ginkel.com wrote: On Mon, Oct 11, 2010 at 19:02, Thilo-Alexander Ginkel th...@ginkel.com wrote: ACK. I have set up a test installation based on production data and will play around with it a little over the next days to pinpoint what exactly went wrong. After some further

Re: Review performance issue

2010-10-14 Thread Thilo-Alexander Ginkel
On Thu, Oct 14, 2010 at 15:29, mohan kumar mohan...@gmail.com wrote: Thanks for your response. apache (httpd) uses more cpu and memory:  PID USER      PR  NI  VIRT  RES  SHR S %CPU %MEM    TIME+  COMMAND 14071 apache    25   0  449m 116m 6124 R 89.0  2.1   0:34.34 httpd Hm, just had a

Re: RB 1.0.9 - 1.5 upgrade woes

2010-10-12 Thread Thilo-Alexander Ginkel
On Mon, Oct 11, 2010 at 19:02, Thilo-Alexander Ginkel th...@ginkel.com wrote: ACK. I have set up a test installation based on production data and will play around with it a little over the next days to pinpoint what exactly went wrong. After some further investigation it seems that mod_python

Re: Active Directory

2010-10-11 Thread Thilo-Alexander Ginkel
On Mon, Oct 11, 2010 at 15:18, Al West a...@tsew.net wrote: Okay so at least I know it works!  I did try using LDAP settings but it's only my second time using LDAP on Linux so I was getting the parameters wrong and locked myself out a few times.  How does one reset the authentication settings

Re: RB 1.0.9 - 1.5 upgrade woes

2010-10-11 Thread Thilo-Alexander Ginkel
Hi Christian, On Sat, Oct 9, 2010 at 08:15, Christian Hammond chip...@chipx86.com wrote: We don't have any permission enforcement for submitting review requests, so I can't imagine what would be causing that problem. Nor can I imagine why everything would be using your admin user. These are

Re: KeyError at /admin/db/scmtools/repository/add/

2010-10-09 Thread Thilo-Alexander Ginkel
Hi Alex, On Sat, Oct 9, 2010 at 16:47, Alex atrof...@gmail.com wrote: Exception Type: KeyError at /admin/db/scmtools/repository/add/ Exception Value: 'tool' I guess your scmtools on the database is empty. I have seen this once on a test upgrade - I guess something went wrong during the

Re: Review Board 1.5 Released!

2010-10-01 Thread Thilo-Alexander Ginkel
On Fri, Oct 1, 2010 at 09:15, Christian Hammond chip...@chipx86.com wrote: We just released Review Board 1.5! A lot went into this release, and I think it turned out really well. Congratulations and thanks a lot for the continued effort of all contributors! Regards, Thilo -- Want to help the

Re: RB integration with Redmine bug tracker

2010-09-02 Thread Thilo-Alexander Ginkel
On Thursday 02 September 2010 21:09:59 Christian Hammond wrote: I want to first make sure we're both clear on what you're trying to do and what you can do. Review Board can't post to bug trackers today. It's a way to link to existing bug trackers, so that if someone references an existing

Re: User Account Management through REST API

2010-06-30 Thread Thilo-Alexander Ginkel
On Wed, Jun 30, 2010 at 11:19, Christian Hammond chip...@chipx86.com wrote: I know what other people have done is written custom auth backends to talk to a central database for users, so instead of creating/updating users in Review Board based on redmine's data, you'd just have Review Board use

Re: User Account Management through REST API

2010-06-30 Thread Thilo-Alexander Ginkel
On Wednesday 30 June 2010 20:34:31 Christian Hammond wrote: You actually can add a user in LDAP who is not in Review Board as a reviewer. They won't appear in the autocomplete, but if you type the name it will work. Review Board checks the auth backend for any unrecognized names that are

Re: Marking a review as currently being reviewed

2010-05-28 Thread Thilo-Alexander Ginkel
On Wed, May 26, 2010 at 21:47, David Trowbridge trowb...@gmail.com wrote: There aren't any better ways to do it, since this is kind of contrary to the use cases we had when building RB. I personally like it when more than one person looks at a review--more eyes means more bugs are caught early

Re: RBTools UnicodeDecodeError of fresh VS2K8 project from fresh Win7 x64 + Py26 + RBTools0.2RC1

2010-02-25 Thread Thilo-Alexander Ginkel
On Thursday 25 February 2010 01:55:42 Pv wrote: I am pretty sure this aborts in RBTools itself before it ever gets to the server. Again, a manual upload of the diff file to the server works fine. Could http://reviews.reviewboard.org/r/1298/ have caused this? Regards, Thilo -- Want to help

Re: ReviewBoard Error while posting review for GIT

2010-01-27 Thread Thilo-Alexander Ginkel
On Tuesday 26 January 2010 18:55:20 Kunjal wrote: Here is my config looks like: [core] repositoryformatversion = 0 filemode = true bare = false logallrefupdates = true [remote origin] url =

Re: What is the status of ClearCase integration on Windows?

2010-01-15 Thread Thilo-Alexander Ginkel
On Friday 15 January 2010 23:20:32 Sassan wrote: I am also thinking it might be a good idea to add a repository independent base functionality to the post-review script where it is handed the root directory of two source trees, before and after the change and it will then just compare the

Re: Enable search in RB 1.0.5.

2010-01-10 Thread Thilo-Alexander Ginkel
Hi, IIRC the root cause for this issue is that your Lucene version is too new. Using the latest 2.x version should do the trick. Regards, Thilo On 10 Jan 2010 03:56, Kunjal kunjal.par...@gmail.com wrote: Hello, We want to enable search in RB. We are having issues. I've finally built

Re: One or more fields had errors: fields: 'path': 'This field is required.'

2009-12-15 Thread Thilo-Alexander Ginkel
On Mon, Dec 14, 2009 at 23:11, Christian Hammond chip...@chipx86.com wrote: It should just be the field constraints from those FileFields. We don't do any custom validation checks in those forms that I can see. It could potentially fail if the diff itself is empty, even though the name is

Re: One or more fields had errors: fields: 'path': 'This field is required.'

2009-12-15 Thread Thilo-Alexander Ginkel
On Tue, Dec 15, 2009 at 15:28, Thilo-Alexander Ginkel th...@ginkel.com wrote: The question is, how can I calculate the length of this string in bytes, not characters, taking the wire encoding into account? A patch is available at: http://reviews.reviewboard.org/r/1298/ Regards, Thilo -- Want

Re: RB and Browser Compatibilty.

2009-12-01 Thread Thilo-Alexander Ginkel
On Wednesday 02 December 2009 02:18:17 Kunjal wrote: We are on verge of rolling out RB 1.0.5 to 300+ developers. Some of our developers use IE and some of them use FireFox. Does RB 1.0.5 work with both browser seamless? During my testing I did not find the difference but I might have missed

Review Board diff issues: table.rows[endRow] is undefined

2009-11-19 Thread Thilo-Alexander Ginkel
Hello everyone, I just stumbled across a problem while using Review Board 1.0.5.1 that seems to be related to the retrieval of diffs while the diff viewer is loading. The retrieval of the third file in my diff list seems to trigger a Javascript error, which I can see in my browser's error console

Re: Error uploading diff: One or more fields had errors (105)

2009-11-16 Thread Thilo-Alexander Ginkel
On Monday 16 November 2009 11:50:36 developer 236 wrote: i installed p4 tool also @ the server m/c, but still getting the same error. The p4 is also in path environment variable. Is the PATH modification also visible to the Apache process hosting Review Board? I don't know which OS you are

Re: Integrated authentication with Active Directory

2009-11-13 Thread Thilo-Alexander Ginkel
On Fri, Nov 13, 2009 at 03:37, Akhilesh akhileshjo...@gmail.com wrote: I'm having some trouble setting up integrated authentication with AD. Here is what I have done- 1. joined VM hosting ReviewBoard site to company.com 2. From Admin menu, set Authentication method = Active Directory 3. Set

Re: Integrated authentication with Active Directory

2009-11-13 Thread Thilo-Alexander Ginkel
On Friday 13 November 2009 17:36:17 Akhilesh wrote: Thanks Thilo for reply. I see that Find DC from DNS remains disabled. How do I enable it? IIRC you need to install pydns (e.g., using easy_install). Regards, Thilo --~--~-~--~~~---~--~~ Want to help the

Re: Review Board 1.0.5 released

2009-10-22 Thread Thilo-Alexander Ginkel
On Thu, Oct 22, 2009 at 14:12, Paolo Borelli paolo.bore...@gmail.com wrote:        I upgraded to 1.0.5 and I am having problems... Creating new review requests from the web interface fails: after selecting the file, when you press the Create Review Request simply nothing happens, no error

Installing PyLucene on Windows

2009-10-14 Thread Thilo-Alexander Ginkel
Hello everyone, I was thinking about adding search support to our Review Board installation, which would require the installation of PyLucene. Has somebody succeeded in getting this working on a Windows-based platform? Thanks, Thilo --~--~-~--~~~---~--~~ Want to

Re: Expand broken since 1.0.3?

2009-09-28 Thread Thilo-Alexander Ginkel
On Mon, Sep 28, 2009 at 12:20, Thilo-Alexander Ginkel th...@ginkel.com wrote: On Fri, Sep 25, 2009 at 22:21, Christian Hammond chip...@chipx86.com wrote: That's odd.. I'll try to reproduce that tonight or this weekend. I just tried reproducing the issue, but was not successful. So

Expand broken since 1.0.3?

2009-09-25 Thread Thilo-Alexander Ginkel
Hello everyone, can somebody confirm my observation that clicking on any of the Expand hyperlinks (to expand collapsed source code sections) in the RB diff viewer no longer works with 1.0.3? Thanks, Thilo --~--~-~--~~~---~--~~ You received this message because

Re: Expand broken since 1.0.3?

2009-09-25 Thread Thilo-Alexander Ginkel
On Fri, Sep 25, 2009 at 17:31, Thilo-Alexander Ginkel th...@ginkel.comwrote: can somebody confirm my observation that clicking on any of the Expand hyperlinks (to expand collapsed source code sections) in the RB diff viewer no longer works with 1.0.3? Just a brief update: The root cause

Re: Expand broken since 1.0.3?

2009-09-25 Thread Thilo-Alexander Ginkel
On Friday 25 September 2009 21:38:35 Christian Hammond wrote: This is a standard diff, not an interdiff? Yes, it's a standard diff. One interesting thing is that for requests created prior to 1.0.3 the fragment expansion still seems to work (I just tried a couple of requests, though). Only

Re: Expand broken since 1.0.3?

2009-09-25 Thread Thilo-Alexander Ginkel
On Friday 25 September 2009 22:11:01 Christian Hammond wrote: Were you trying to expand a diff that was still attached to a draft of a review request? Or was it actually a published diff visible to everyone? It was a published diff visible to everyone (actually a colleague pointed me to the

Best practices for supporting multiple projects with a RB installation

2009-08-20 Thread Thilo-Alexander Ginkel
Hello everybody, I was wondering whether there are any best practices or experiences on how to support multiple projects (within the same organization) with a Review Board installation (the developers working on each project are typically different people). Would you try to fit everything

Re: Windows post-review .exe

2009-08-19 Thread Thilo-Alexander Ginkel
On Wednesday 19 August 2009 23:03:33 Dana Lacoste wrote: So, I have everything working (from trunk, with a patch that I've uploaded for review :) to get my perforce users working with reviewboard. That is, I can, from Windows, run post-review and create a review with a diff. WooHoo! The

Re: Trouble posting reviews since 1.0.1 upgrade

2009-08-13 Thread Thilo-Alexander Ginkel
On Wed, Aug 12, 2009 at 23:53, Christian Hammondchip...@chipx86.com wrote: It sounds like it's actually using the ClearCase SCM instead of your own. Are you sure your repository entry is still mapping to your custom SCM? In 1.0, we accidentally left out the database entry for the ClearCase

Trouble posting reviews since 1.0.1 upgrade

2009-08-12 Thread Thilo-Alexander Ginkel
Hello everybody, since upgrading from 1.0rc2 to 1.0.1 I have trouble posting reviews to RB using a custom post-review implementation. The problem is that the JSON response does not contain too many error details, just: {fields: {path: [substring not found]}, stat: fail, err: {msg: One or more

Re: Trouble posting reviews since 1.0.1 upgrade

2009-08-12 Thread Thilo-Alexander Ginkel
Hello Christian, On Wednesday 12 August 2009 22:25:03 Christian Hammond wrote: There's another thread on this problem as well. Are you using ClearCase? no, this happens with a proprietary SCM, for which I added support to RB and post-review. So, I cannot rule out that my scm implementation is

Re: X.509 Certificate Authentication

2009-08-11 Thread Thilo-Alexander Ginkel
On Mon, Aug 3, 2009 at 06:44, Nathan Heijermans nheijerm...@gmail.com wrote: This basically sound like what I intend to implement. Would you mind sharing some patches? I just uploaded my patch to http://reviews.review-board.org/r/938/. Thanks! I will try your changes once I have updated my

Re: X.509 Certificate Authentication

2009-07-31 Thread Thilo-Alexander Ginkel
On Fri, Jul 31, 2009 at 13:27, rupert.thurnerrupert.thur...@gmail.com wrote: just as a side note, edgewall trac supports it by just taking the webservers authentication, see:  * http://trac.edgewall.org/browser/trunk/trac/web/auth.py.  *

Integrating a custom SCM / non-numeric change IDs

2009-05-11 Thread Thilo-Alexander Ginkel
Hello everybody, while attempting to add some basic support for a proprietary SCM to Review Board / post-review I stumbled over one issue that I'm currently uncertain how to solve: The SCM I'm integrating uses change identifiers, which are not numeric. Instead arbitrary strings are used, which

Re: Integrating a custom SCM / non-numeric change IDs

2009-05-11 Thread Thilo-Alexander Ginkel
On May 11, 9:55 pm, Christian Hammond chip...@chipx86.com wrote: With this SCM, is the change identifier a server-stored ID that contains the description and other information for Review Board to parse? Or is it more like an atomic ID representing that change that gets pushed to the server