Bug 205374 - mail/cyrus-imapd25: no HTTP authentication via PAM due to typo
Summary: mail/cyrus-imapd25: no HTTP authentication via PAM due to typo
Status: In Progress
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Hajimu UMEMOTO
URL: https://lists.andrew.cmu.edu/pipermai...
Keywords: needs-qa, patch
Depends on:
Blocks:
 
Reported: 2015-12-17 11:12 UTC by ml
Modified: 2018-01-12 22:50 UTC (History)
1 user (show)

See Also:
bugzilla: maintainer-feedback? (ume)

Attachments
Add a patch for imap/httpd.c (1.32 KB, patch)
2015-12-17 11:12 UTC, ml 		no flags Details | Diff
Add a patch for imap/httpd.c (1.66 KB, patch)
2015-12-17 14:07 UTC, ml 		no flags Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description ml 2015-12-17 11:12:04 UTC 
Created attachment 164308 [details]
Add a patch for imap/httpd.c

Cyrus IMAP provides several protocols to connect with: POP3, IMAP, HTTP.

While it has always been possible to authenticate through PAM with POP3 and IMAP, this does not work with HTTP.

See https://lists.andrew.cmu.edu/pipermail/cyrus-devel/2015-April/003202.html for details.


This patch allows PLAIN authentication with HTTP; as noted above, it would break DIGEST AUTH, but this does not work anyway for us in the above scenario.
Comment 1 VK 2015-12-17 11:36:39 UTC 
Thanks for the submission.

Can you please attach the patch against the whole port, complete with PORTREVISION bump?
Comment 2 ml 2015-12-17 14:07:31 UTC 
Created attachment 164310 [details]
Add a patch for imap/httpd.c
Comment 3 ml 2015-12-17 14:08:22 UTC 
I added the PORTREVISION bump.

However, I thought I was actually providing a "patch against the whole port"... please correct me if it's not.
Comment 4 Hajimu UMEMOTO freebsd_committer freebsd_triage 2015-12-19 14:54:44 UTC 
Thank you for your report.
I got this report directly from Paul Dekkers at that time.  He posted it to cyrus-devel list as well.  Then, he got the answer that it breaks HTTP Digest Auth.  So, I decided not to update my ports until upstream accept the fix.
So, sorry but I suspend this PR until then.
Comment 5 Walter Schwarzenfeld 2018-01-12 22:50:05 UTC 
Any advance here?