Article delegate-en/2155 of [1-5169] on the server localhost:119
  upper oldest olders older1 this newer1 newers latest
search
[Top/Up] [oldest] - [Older+chunk] - [Newer+chunk] - [newest + Check]
[Reference:<_A2154@delegate-en.ML_>]
Newsgroups: mail-lists.delegate-en

[DeleGate-En] Re: problems with smtp-auth
06 Mar 2003 12:20:27 GMT feedback@delegate.org (Yutaka Sato)


In message <265415BA-4FC7-11D7-90E9-00000000000X@ilink.de> on 03/06/03(20:31:19)
you Martin Vetter <pnieabdyi-xtqvdm5mnljr.ml@ml.delegate.org> wrote:
 |i try to make delegated 8.3.4 work as a proxy,
 |adding smtp-authorization between an already running
 |sendmail 8.9.3 as client, which does not know about smtp-auth, and a
 |sendmail 8.12.6 as server, requiring smtp-authorization
 |for access to relay services.
 |
 |i thought something like this should do the trick:
 |
 |   delegated -f -vv -Pport \
 |             SERVER=smtp://server.domain.com/ \
 |             MYAUTH=login:password
 |
 |when i "telnet localhost port", i can see
 |delegated immediately tries to authenticate via
 |"AUTH PLAIN ..." (good), but with no success (bad).
 |
 |it seems to me that delegated sends (base64 encoded)
 |'\0login\0password\0' instead of '\0login\0password',
 |which i would think would be more correct ...

Thank you. I'll fix it so.

 |ps:
 |
 |in delegate8.3.4/src/smtp.c, shouldn't this:
 |
 |   if( isinList(val,"login") )
 |     SMTP_doauth |= A_PLAIN;
 |   if( isinList(val,"plain") )
 |     SMTP_doauth |= A_LOGIN;
 |
 |perhaps more look like this:
 |
 |   if( isinList(val,"login") )
 |     SMTP_doauth |= A_LOGIN;
 |   if( isinList(val,"plain") )
 |     SMTP_doauth |= A_PLAIN;

I see :)
I'll modify src/smtp.c as the enclosed patch.


Cheers,
Yutaka
--
  @ @ Yutaka Sato <y.sato@delegate.org> http://www.delegate.org/y.sato/
 ( - ) National Institute of Advanced Industrial Science and Technology (AIST)
_<   >_ 1-1-4 Umezono, Tsukuba, Ibaraki, 305-8568 Japan
Do the more with the less -- B. Fuller

*** ../delegat8.5.0/src/smtp.c	Wed Jan  1 15:45:04 2003
--- smtp.c	Thu Mar  6 21:13:40 2003
***************
*** 80,88 ****
  		if( val[0] == 0 ){
  			SMTP_doauth = A_PLAIN | A_LOGIN;
  		}else{
- 			if( isinList(val,"login") )
- 				SMTP_doauth |= A_PLAIN;
  			if( isinList(val,"plain") )
  				SMTP_doauth |= A_LOGIN;
  		}
  	}
--- 80,88 ----
  		if( val[0] == 0 ){
  			SMTP_doauth = A_PLAIN | A_LOGIN;
  		}else{
  			if( isinList(val,"plain") )
+ 				SMTP_doauth |= A_PLAIN;
+ 			if( isinList(val,"login") )
  				SMTP_doauth |= A_LOGIN;
  		}
  	}
***************
*** 784,790 ****
  		if( *ap == ':' ){
  			wordscanY(ap+1,pp,128,"^\r\n");
  		}
! 		pp += strlen(pp) + 1;
  
  		str_to64(plain,pp-plain,bplain,sizeof(bplain),1);
  		if( pp = strpbrk(bplain,"\r\n") )
--- 784,790 ----
  		if( *ap == ':' ){
  			wordscanY(ap+1,pp,128,"^\r\n");
  		}
! 		pp += strlen(pp);
  
  		str_to64(plain,pp-plain,bplain,sizeof(bplain),1);
  		if( pp = strpbrk(bplain,"\r\n") )

  admin search upper oldest olders older1 this newer1 newers latest
[Top/Up] [oldest] - [Older+chunk] - [Newer+chunk] - [newest + Check]
@_@V