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 as trusted
 root somewhere?
 Check you're using the right port for your AD configuration (perhaps 636 or
 3269 depending on whether you're trying to talk to the GC or not).

According to tcpdump Review Board uses port 389 (ldap) to connect to
the AD if TLS is enabled.

I verified using another LDAP client that the AD server supports
StartTLS on port 389 and ldaps on port 636.

AFAICS, there is no way to change the port from the RB admin
interface. Is there anything obvious that I am missing?

Thanks,
Thilo

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en


Numbering for review comments.

2012-01-18 Thread N.S.S.R.Murthy
Is there way to number the review comments. It helps in referring to
previous or next comments more specifically. At present we are using
line numbers. But having comments numbers is more helpful.

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en


Issue 2453 in reviewboard: bugs

2012-01-18 Thread reviewboard

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

New issue 2453 by tjs...@gmail.com: bugs
http://code.google.com/p/reviewboard/issues/detail?id=2453

invalid test

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1896 in reviewboard: Omitting the git repo path throws 500 error

2012-01-18 Thread reviewboard


Comment #11 on issue 1896 by chip...@gmail.com: Omitting the git repo path  
throws 500 error

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

It should be the origin. Same thing you would clone.

post-review will do a 'git remote show origin' and try to find a matching  
repository (comparing Path) on Review Board.


--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 2453 in reviewboard: bugs

2012-01-18 Thread reviewboard

Updates:
Status: Invalid

Comment #1 on issue 2453 by chip...@gmail.com: bugs
http://code.google.com/p/reviewboard/issues/detail?id=2453

(No comment was entered for this change.)

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 386 in reviewboard: Files without a trailing newline break the diff parser.

2012-01-18 Thread reviewboard


Comment #10 on issue 386 by sharadoj...@gmail.com: Files without a trailing  
newline break the diff parser.

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

I am facing similar issue too. Could you please provide us the patched file  
which fixed the problem.


--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 386 in reviewboard: Files without a trailing newline break the diff parser.

2012-01-18 Thread reviewboard


Comment #11 on issue 386 by chip...@gmail.com: Files without a trailing  
newline break the diff parser.

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

This was fixed 4 years ago. You're likely hitting a brand new issue, and  
will need to open a new bug. We don't have any patched file that fixes your  
problem, or it would be part of the codebase.


--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1038 in reviewboard: Add time zone to settings page.

2012-01-18 Thread reviewboard


Comment #11 on issue 1038 by trowb...@gmail.com: Add time zone to settings  
page.

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

Yes, we have plans for this (and some experimental code). We don't need a  
separate issue.


--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 2452 in reviewboard: Allow for batched operations on RBS

2012-01-18 Thread reviewboard

Updates:
Status: Duplicate
Mergedinto: 579

Comment #1 on issue 2452 by trowb...@gmail.com: Allow for batched  
operations on RBS

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

(No comment was entered for this change.)

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 579 in reviewboard: batch operations (especially delete), would be very useful

2012-01-18 Thread reviewboard


Comment #11 on issue 579 by trowb...@gmail.com: batch operations  
(especially delete), would be very useful

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

Issue 2452 has been merged into this issue.

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 2445 in reviewboard: Unable to 'View Diff' when using with RhodeCode for mercurial repositories

2012-01-18 Thread reviewboard


Comment #9 on issue 2445 by trowb...@gmail.com: Unable to 'View Diff' when  
using with RhodeCode for mercurial repositories

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

Your raw file url is wrong. You need to figure out how to get the  
contents of files at specific revisions from RhodeCode.


--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 2451 in reviewboard: Add RB number next to each RB summary in My Outgoing Reviews list.

2012-01-18 Thread reviewboard

Updates:
Status: UserError

Comment #1 on issue 2451 by trowb...@gmail.com: Add RB number next to each  
RB summary in My Outgoing Reviews list.

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

You can add columns to dashboard views using the pencil icon at the right.

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.