Closed
Bug 12924
Opened 25 years ago
Closed 25 years ago
[DOGFOOD][FEATURE] support password authenticated news servers
Categories
(MailNews Core :: Backend, defect, P3)
MailNews Core
Backend
Tracking
(Not tracked)
VERIFIED
FIXED
M14
People
(Reporter: sspitzer, Assigned: sspitzer)
References
Details
(Whiteboard: [PDT-])
and leverage mscott's GetPassword() work in nsMsgIncomingServer().
Assignee | ||
Updated•25 years ago
|
Assignee | ||
Comment 1•25 years ago
|
||
accepting, adding a dependency, marking m11
Comment 2•25 years ago
|
||
Triage to M15
Comment 3•25 years ago
|
||
Shaver's taking a look at implementing this for us today. Thank you shaver!
Assignee | ||
Updated•25 years ago
|
Summary: [FEATURE] support password authenticated news servers → [DOGFOOD][FEATURE] support password authenticated news servers
Assignee | ||
Comment 4•25 years ago
|
||
marking dogfood.
We don't think is is needed for CPD dogfood. Feel free to educate PDT on why
it's needed?
Assignee: sspitzer → shaver
Status: ASSIGNED → NEW
Target Milestone: M15 → M11
I need it for _my_ dogfood, along with @-in-username, so I'm doing it. I also
think that solving NNTP auth will make it easier for Alec to define a good
auth-data mechanism that can be shared by all of mailnews, but that's a separate
argument. I'm told that piles of ISPs use authentication to protect their news
servers as well, but don't have personal experience with more than one or two.
Anyway, it's pretty much fait accompli: I'll send patches to seth/alec for
review when I fix the RDF-URI-escaping bug.
Will this make it into M11? If not, pls change the target milestone to M12 so
this bug doesn't show up on the M11 stopper list. Thanks
Updated•25 years ago
|
Target Milestone: M11 → M12
Comment 8•25 years ago
|
||
m12
Status: NEW → ASSIGNED
I'm going to restart work on this soon, now that alecf has tweaked the generic
authentication code.
No, really. (I'll even leave it M12, to add to the excitement!)
Target Milestone: M12 → M13
I suck. =(
Comment 11•25 years ago
|
||
*** Bug 22087 has been marked as a duplicate of this bug. ***
OK, I've started to reconstitute my patches, now that alecf has made the world a
happier place for authentication, but it'll be close for M13. alecf/seth, if
you can answer questions and stuff in IRC, it'll make life easier.
Target Milestone: M13 → M14
Missed it. M14 should be cake, but then, M13 should have been cake. =/
Assignee | ||
Updated•25 years ago
|
Assignee: shaver → sspitzer
Status: ASSIGNED → NEW
Assignee | ||
Comment 14•25 years ago
|
||
taking this from mike, he's overloaded.
I got the fix in my tree. checking in early m14.
Assignee | ||
Comment 15•25 years ago
|
||
authenticated news is working. there are a few features left, and I'll log
those as bugs now.
right now, the username / password is cached on a per folder basis for the
session.
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 16•25 years ago
|
||
reopening, I'm having a lot of problems with auth news in today's build.
(bug I thought I fixed are not fixed.)
re-opening until it is ready for testing.
rats.
Status: RESOLVED → REOPENED
Comment 18•25 years ago
|
||
Will leave this alone for a couple of days, but will push beyond B1 if it gets
close.
Assignee | ||
Comment 19•25 years ago
|
||
I believe this to be fixed, except that we don't use single signon yet, but we
will.
phil (or someone) can you try news://poisonivy/xrated with (u=olive,p=popeye)
and see if it works for you on windows? it works for me on linux.
Status: REOPENED → RESOLVED
Closed: 25 years ago → 25 years ago
Resolution: --- → FIXED
Comment 20•25 years ago
|
||
I am going to try that!!
Comment 21•25 years ago
|
||
Yes. It works for me on the WinNT new profile for today's 1-28-09-M14 commercial
build. I may need to try Mac later.
Comment 22•25 years ago
|
||
Comment 23•25 years ago
|
||
I am still waiting for the Mac to verify this bug since it has been blocked by
bug#26308....
Comment 24•25 years ago
|
||
Seth, does the password authenticated dialog suppose to display one time or two
times? It seems it is displaying two times now... the first time is when I am
selecting the newsgroup after subscribe, the second time is when you selecting
for viewing/reading newsgroup msg, is that right?
I remember it used to display one time only, and it displayed just after you
select that newsgroup (xrated), can you tell me which way it suppose to work?
Assignee | ||
Comment 25•25 years ago
|
||
it works for me. here is what I am seeing
for news://poisonivy/xrated
start up, select the xrated newsgroup on poisonivy
get the news username dialog, enter olive (check remember checkbox)
if this is the first time I've used wallet, I get the wallet password dialog,
enter my wallet key twice.
get the news password dialog, enter popeye (check remember checkbox)
click on a message, it displays, no more prompts.
Assignee | ||
Comment 26•25 years ago
|
||
then, if I exit and start up, and try to read a message in xrated, I should only
get one prompt, the wallet database password prompt
Comment 27•25 years ago
|
||
It is indeed behaving differently than 4.x in that it will ask for user/password
when downloading headers and again when trying to read a new message. (I am not
using remember password).
I'm sure these and other issues would be spinoffs as separate bugs.
Assignee | ||
Comment 28•25 years ago
|
||
with 4.x, we had some tricks for remembering your password per session and we
always remembered your password per session, once you logged on. but as soon as
you quit, and came back, you had to log in again.)
in 5.0, we currently have two states:
1) rememeber my username / password in the current session and across sessions
2) never remember my password, even in the same session
it sounds like we need that to be:
1) remember my username / password in the current session and across sessions
2) remember my username / password in the current session, but not across
sessions.
I think this is what imap / and pop does.
I'll log that as a new bug against me.
Comment 29•25 years ago
|
||
Since Seth logged another bug#27726...
Marking as verified for this bug since it is working for all the platforms for
this specific newsgroup(news://poisonivy/xrated) now....
Status: RESOLVED → VERIFIED
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•