Merge lp:~cmiller/desktopcouch/couchdb-breaks-basic-auth into lp:desktopcouch
| Status: | Merged |
|---|---|
| Approved by: | Chad Miller on 2010-07-20 |
| Approved revision: | 166 |
| Merged at revision: | 166 |
| Proposed branch: | lp:~cmiller/desktopcouch/couchdb-breaks-basic-auth |
| Merge into: | lp:desktopcouch |
| Diff against target: |
11 lines (+1/-0) 1 file modified
desktopcouch/local_files.py (+1/-0) |
| To merge this branch: | bzr merge lp:~cmiller/desktopcouch/couchdb-breaks-basic-auth |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Manuel de la Peña (community) | 2010-07-13 | Needs Information on 2010-07-14 | |
|
Review via email:
|
|||
Commit Message
Support new Basic auth for HTTP that our bookmark file requires. (LP: #599745)
| Chad Miller (cmiller) wrote : | # |
No reason, Manuel. It has to be something. I don't think the choice is very important, since the user should never see a password dialog that mentions it.
| Elliot Murphy (statik) wrote : | # |
On Wed, Jul 14, 2010 at 10:01 AM, Chad Miller <email address hidden> wrote:
> No reason, Manuel. It has to be something. I don't think the choice is very important, since the user should never see a password dialog that mentions it.
How about 'unused' instead of 'administrator' in order to preemptively
answer this question for future readers of the code?
--
Elliot Murphy | https:/
- 166. By Chad Miller on 2010-07-14
-
Use a more descriptive name for the HTTP Basic auth realm.

Looks good to me, just one question, why are you setting the authentication realm to be administrator?