Bug#536870: trn4 AUTHINFO GENERIC should honour NNTPAUTH

2009-07-14 Thread Colin Watson
On Tue, Jul 14, 2009 at 11:21:42AM +0100, Ian Jackson wrote:
 Package: trn4
 Version: 4.0-test76-15
 
 trn4 has some kind of support for AUTHINFO GENERIC but it doesn't
 implement the protocol as described in the original specification:
  http://ftp.nada.kth.se/pub/home/assar/nntp-hacks/GENERIC.FAQ
 
 Specifically, it requires the authentication command and arguments to
 be configured in ~/.trn/access rather than getting it from the
 NNTPAUTH variable.  This means that trn4 needs configuring separately
 from other newsreaders.  It's particularly inconvenient in the case
 where the authentication setup (the NNTPAUTH value) varies (for
 example, between different users on the same system).

Thanks for your report. I believe this patch will fix it (it at least
permits reading news on chiark, although I haven't tested posting). I'll
also upload this to unstable shortly, and it appears that the
dependencies should permit installing the .deb directly on chiark
without drama.

Index: datasrc.c
===
--- datasrc.c   (revision 3126)
+++ datasrc.c   (revision 3127)
@@ -48,6 +48,9 @@
vals[DI_NNTP_SERVER] = machine;
vals[DI_AUTH_USER] = read_auth_file(nntp_auth_file,
vals[DI_AUTH_PASS]);
+#ifdef USE_GENAUTH
+   vals[DI_AUTH_COMMAND] = getenv(NNTPAUTH);
+#endif
vals[DI_FORCE_AUTH] = getenv(NNTP_FORCE_AUTH);
new_datasrc(default,vals);
 }
@@ -86,6 +89,9 @@
if (machine) {
vals[DI_AUTH_USER] = read_auth_file(nntp_auth_file,
vals[DI_AUTH_PASS]);
+#ifdef USE_GENAUTH
+   vals[DI_AUTH_COMMAND] = getenv(NNTPAUTH);
+#endif
vals[DI_FORCE_AUTH] = getenv(NNTP_FORCE_AUTH);
}
 #endif

Thanks,

-- 
Colin Watson   [cjwat...@debian.org]



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#536870: trn4 AUTHINFO GENERIC should honour NNTPAUTH

2009-07-14 Thread Ian Jackson
Package: trn4
Version: 4.0-test76-15

trn4 has some kind of support for AUTHINFO GENERIC but it doesn't
implement the protocol as described in the original specification:
 http://ftp.nada.kth.se/pub/home/assar/nntp-hacks/GENERIC.FAQ

Specifically, it requires the authentication command and arguments to
be configured in ~/.trn/access rather than getting it from the
NNTPAUTH variable.  This means that trn4 needs configuring separately
from other newsreaders.  It's particularly inconvenient in the case
where the authentication setup (the NNTPAUTH value) varies (for
example, between different users on the same system).

Ian.



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org