[vchkpw] Re: chkuser wrongly accept emails for default@

2007-04-18 Thread Stephane Bouvard (ML)
Hi,

,- - [ Le mercredi 18 avril 2007 vers 9:33 tonix (Antonio Nati) écrivait: ] - -
|

 I don't see any reason to check for the content 
 of alias, looking for a bouncing string. Apart 
 .qmail-default, I don't see a reason why a 
 .qmail-ALIAS should contain a bouncing string.

There is for me a reason : when using a catch-all if you want to disable some 
specific address...  i know that actually if the .qmail-default specify a 
catch-all, chkuser does not look further and accept the mail, but it should be 
easy in that case to still verify if the specific user is not configured to 
bounce...

Anyway, for me, if a .qmail-xyz specify bounce-no-mailbox for any reason, i 
do not see why chkuser should accept the mail and let qmail bounce it as it's 
easy to avoid...  it's an opengate for spammers.

|
`- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -



-- 
Bien à vous...
 _
(_'  L'informatique est ma passion, vous la simplifier, mon métier !
,_)téphane Bouvard [antarex AT freenet DOT be] http://www.antarex.be



[vchkpw] Re: chkuser wrongly accept emails for default@

2007-04-18 Thread Stephane Bouvard (ML)
Bonjour,

,- - [ Le mercredi 18 avril 2007 vers 8:43 Jeremy Kister écrivait: ] - -
|

 Great idea.  I'm no C guy, but shouldn't that second conditinal be
 inside the first?

I'm no C guy nope, i've used the same existing code than for a mailman list, 
wich use the same principe.

|
`- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -



-- 
Bien à vous...
 _
(_'  L'informatique est ma passion, vous la simplifier, mon métier !
,_)téphane Bouvard [antarex AT freenet DOT be] http://www.antarex.be



Re: [vchkpw] Re: chkuser wrongly accept emails for default@

2007-04-18 Thread tonix (Antonio Nati)

At 10.26 18/04/2007, Stephane Bouvard (ML) wrote:

Hi,

,- - [ Le mercredi 18 avril 2007 vers 9:33 tonix 
(Antonio Nati) écrivait: ] - -

|

 I don't see any reason to check for the content
 of alias, looking for a bouncing string. Apart
 .qmail-default, I don't see a reason why a
 .qmail-ALIAS should contain a bouncing string.

There is for me a reason : when using a 
catch-all if you want to disable some specific 
address...  i know that actually if the 
.qmail-default specify a catch-all, chkuser does 
not look further and accept the mail, but it 
should be easy in that case to still verify if 
the specific user is not configured to bounce...


Internal logic should be changed. I have to 
change/extend it for other reasons, I will look for this also in case.


Anyway, for me, if a .qmail-xyz specify 
bounce-no-mailbox for any reason, i do not see 
why chkuser should accept the mail and let qmail 
bounce it as it's easy to avoid...  it's an opengate for spammers.


Let's try to distinguish problems.

.qmail-default has an architectural reason to 
exist, as qmail architecture delivers to 
.qmail-default all emails for not existing users. 
Inside .qmail-default there is the logic for 
rejecting/deleting/storing all those messages. We 
simply know default is a fake alias, that 
must exist but has nothing to do with whatever other alias you may create.


It would be good if chkuser add an option to 
reject default rcpt, as it is a fake rcpt. This will close a qmail hole.


Different matter is to handle in a more extended 
way users/aliases, despite of bounce/delete/catchall.


Additional checking could be done (I'm thinking 
about quota checking) even if catchall/delete is specified.


I suggest also to introduce a new notation for 
rejecting users/aliases with a custom message 
(i.e.: reject user has changed address. Write to 
[EMAIL PROTECTED]). This would be a lot more useful than 
barely put a generic bounce string.


This additional checking should be done on 
aliases AND on .qmail-default inside each user's 
directory. Not to mention some parts should be 
rewritten  in local delivery, as it does not use SMTP.


Ciao,

Tonino



|
`- - - - - - - - - - - - - - - - - - - - - - - - 
- - - - - - - - - - - - - - - -




--
Bien à vous...
 _
(_'  L'informatique est ma passion, vous la simplifier, mon métier !
,_)téphane Bouvard [antarex AT freenet DOT be] http://www.antarex.be




Re: [vchkpw] Re: chkuser wrongly accept emails for default@

2007-04-18 Thread Tom Collins

On Apr 18, 2007, at 2:31 AM, tonix (Antonio Nati) wrote:
I suggest also to introduce a new notation for rejecting users/ 
aliases with a custom message (i.e.: reject user has changed  
address. Write to [EMAIL PROTECTED]). This would be a lot more useful than  
barely put a generic bounce string.


Until that's a part of chkuser, Stephane (and others) can use qmail's  
bouncesaying program in their .qmail-alias files.


# cat .qmail-someaddress
|/var/qmail/bin/bouncesaying 'user has changed address. Write to  
[EMAIL PROTECTED]'


Please be aware that vdelivermail should ONLY be in a domain's .qmail- 
default file.  Putting it into a .qmail-alias file or a user's .qmail  
file can introduce a mail loop (which vdelivermail should detect and  
stop looping) and probably won't accomplish what you want it to.


--
Tom Collins  -  [EMAIL PROTECTED]
Vpopmail - virtual domains for qmail: http://vpopmail.sf.net/
QmailAdmin - web interface for Vpopmail: http://qmailadmin.sf.net/




[vchkpw] Re: chkuser wrongly accept emails for default@

2007-04-18 Thread Stephane Bouvard (ML)
Hi,

,- - [ Le mercredi 18 avril 2007 vers 17:34 Tom Collins écrivait: ] - -
|
 Until that's a part of chkuser, Stephane (and others) can use qmail's  
 bouncesaying program in their .qmail-alias files.

To allow every spammer on earth use our servers to spam using the bounce ?  I 
do not like the idea that much...

It's why i'm looking for a workaround for the default@ fake alias, spammers use 
it, i will not add another hole...

|
`- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -



-- 
Bien à vous...
 _
(_'  L'informatique est ma passion, vous la simplifier, mon métier !
,_)téphane Bouvard [antarex AT freenet DOT be] http://www.antarex.be



Re: [vchkpw] Re: chkuser wrongly accept emails for default@

2007-04-18 Thread tonix (Antonio Nati)

At 17.34 18/04/2007, you wrote:

On Apr 18, 2007, at 2:31 AM, tonix (Antonio Nati) wrote:
I suggest also to introduce a new notation for rejecting users/ 
aliases with a custom message (i.e.: reject user has changed

address. Write to [EMAIL PROTECTED]). This would be a lot more useful than
barely put a generic bounce string.


Until that's a part of chkuser, Stephane (and others) can use qmail's
bouncesaying program in their .qmail-alias files.

# cat .qmail-someaddress
|/var/qmail/bin/bouncesaying 'user has changed address. Write to
[EMAIL PROTECTED]'


So I should add in chkuser a check about a program called 
bouncesaying, both as alias .qmail-ALIAS or qmail-default within 
user's directory, just to complete this check? Which characters are 
used to contain the string: ' or  or any character?


Tonino




Re: [vchkpw] Re: chkuser wrongly accept emails for default@

2007-04-18 Thread Jeremy Kister

On 4/18/2007 11:34 AM, Tom Collins wrote:
Please be aware that vdelivermail should ONLY be in a domain's 
.qmail-default file.  Putting it into a .qmail-alias file or a user's 
.qmail file can introduce a mail loop (which vdelivermail should detect 
and stop looping) and probably won't accomplish what you want it to.


Good reminder.

I've integrated my flavor of Stephane's changes into my code and it's 
working great.  I can now `echo # bounce-no-mailbox  .qmail-username` 
and it will be rejected at smtp time.



Thanks Stephane.


--

Jeremy Kister
http://jeremy.kister.net./