Closed Bug 12924 Opened 25 years ago Closed 25 years ago

[DOGFOOD][FEATURE] support password authenticated news servers

Categories

(MailNews Core :: Backend, defect, P3)

defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: sspitzer, Assigned: sspitzer)

References

Details

(Whiteboard: [PDT-])

and leverage mscott's GetPassword() work in nsMsgIncomingServer().
Blocks: 8213
Status: NEW → ASSIGNED
Target Milestone: M11
accepting, adding a dependency, marking m11
Triage to M15
Shaver's taking a look at implementing this for us today. Thank you shaver!
Summary: [FEATURE] support password authenticated news servers → [DOGFOOD][FEATURE] support password authenticated news servers
marking dogfood.
Whiteboard: [PDT-]
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
Target Milestone: M11 → M12
m12
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
*** 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: shaver → sspitzer
Status: ASSIGNED → NEW
taking this from mike, he's overloaded. I got the fix in my tree. checking in early m14.
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
QA Contact: lchiang → huang
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
Clearing FIXED resolution due to reopen.
Resolution: FIXED → ---
Will leave this alone for a couple of days, but will push beyond B1 if it gets close.
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 ago25 years ago
Resolution: --- → FIXED
I am going to try that!!
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.
There was bug#25904 logged for Mac only...need to wait for bug#25904 fixed in order to verify this bug.
I am still waiting for the Mac to verify this bug since it has been blocked by bug#26308....
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?
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.
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
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.
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.
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
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.