Opened 17 years ago

Closed 17 years ago

Last modified 16 years ago

#1089 closed defect (fixed)

qpopper does not work on Panther

Reported by: ssen@… Owned by: pguyot (Paul Guyot)
Priority: Normal Milestone:
Component: ports Version: 1.0
Keywords: Cc:
Port:

Description

qpopper, by default, uses crypt() to compare the over-the-wire cleartext password with the passwd entry for the user. With Panther, this no longer works because of shadow password support.

The attached patch does the following: 1) fixes destroot support. It's not clear to me this port was ever converted 2) enables pam support 3) creates a new SSL variant named "+ssl"

Attachments (2)

qpopper.diff (2.9 KB) - added by pguyot (Paul Guyot) 17 years ago.
cvs diff for qpopper to fix the trunk and adding various ui_msgs for the variants.
patch-pop_pass.c (4.5 KB) - added by pguyot (Paul Guyot) 17 years ago.
patch for pop_pass.c (goes in files/) to add DirectoryService authentication (instead of crypt() which just doesn't work on Panther)

Download all attachments as: .zip

Change History (11)

comment:1 Changed 17 years ago by ssen@…

comment:2 Changed 17 years ago by ssen@…

By the way, to enable the PAM, you probably want to copy /etc/pam.d/sshd to /etc/pam.d/pop3, or use another config with pam_securityserver.so

comment:3 Changed 17 years ago by waqar@…

* Bug 1088 has been marked as a duplicate of this bug. *

comment:4 Changed 17 years ago by waqar@…

Owner: changed from darwinports-bugs@… to pguyot@…

Assigning to the port owner.

comment:5 Changed 17 years ago by ssen@…

Resolution: fixed
Status: newclosed

I went ahead and committed this.

comment:6 Changed 17 years ago by pguyot (Paul Guyot)

Resolution: fixed
Status: closedreopened

OK. qpopper no longer works out of the box since one has to configure PAM to get it working with the current portfile. I'm reopening to mean that I'm working on it (basically, I'm working on finding out how to access shadow passwords on Panther, I didn't find anything in the documentation and I'm going to ask around, hints are welcomed).

comment:7 Changed 17 years ago by pguyot (Paul Guyot)

Status: reopenedassigned

I found. I need to implement DirectoryService as a method to authenticate users. I have a small snippet that tests the authentication that does work and I'll add it to pop_pass.c. I'll keep PAM as well (as a variant).

comment:8 Changed 17 years ago by pguyot (Paul Guyot)

dependson: 1168

Changed 17 years ago by pguyot (Paul Guyot)

Attachment: qpopper.diff added

cvs diff for qpopper to fix the trunk and adding various ui_msgs for the variants.

Changed 17 years ago by pguyot (Paul Guyot)

Attachment: patch-pop_pass.c added

patch for pop_pass.c (goes in files/) to add DirectoryService authentication (instead of crypt() which just doesn't work on Panther)

comment:9 Changed 17 years ago by pguyot (Paul Guyot)

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.