Ticket #186 (closed enhancement: fixed)

Opened 8 months ago

Last modified 8 months ago

mod_vacation might flood admin user

Reported by: jens Owned by: smoku
Priority: minor Component: sm
Version: 2.1.21 Keywords:
Cc: Mei_Je@… Tracforge_linkmap:
Blocking: Blocked By:

Description

In my jabber server there is a user 'admin@…l' with acl type='all'. This user is usually not logged in. For this user has been deposited a vacation message. When another user sends a message to 'jabber.local' (sm self), the sm sends as 'jabber.local' unlimited messages to all users with acl type='all' and type='messages' until the server shuts down. Sometimes the storage server crashed because it stored many offline messages. I think that the error is as following: The sm forward the message to 'jabber.local' as 'jabber.local' to 'admin@…l'. For this user has been deposited a vacation message. The sm answers the sender as 'jabber.local' with the vacation message. In this case the sender is 'jabber.local', the sm self. This message will be forwarded to administrative users. In particular to 'admin@…l'. Now the loop beginning.

Change History

follow-up: ↓ 4   Changed 8 months ago by smoku

  • priority changed from critical to minor
  • status changed from new to infoneeded_new
  • type changed from defect to enhancement

Yes, this is the possible scenario. It's a problem with any auto-responder system.

How do you propose to solve it?

  Changed 8 months ago by smoku

  • status changed from infoneeded_new to new
  • summary changed from session manager sends messages in an endless loop to mod_vacation might flood admin user

  Changed 8 months ago by smoku

  • status changed from new to infoneeded_new

in reply to: ↑ 1   Changed 8 months ago by jens

  • status changed from infoneeded_new to new

Replying to smoku:

Yes, this is the possible scenario. It's a problem with any auto-responder system. How do you propose to solve it?

I have disabled the vacation message temporary.

follow-up: ↓ 6   Changed 8 months ago by smoku

  • status changed from new to infoneeded_new

But this is not a solution. How would you like it resolved?

in reply to: ↑ 5   Changed 8 months ago by jens

  • status changed from infoneeded_new to new

Replying to smoku:

But this is not a solution. How would you like it resolved?

The auto-responder shouldn't reply to messages which sent from sm. (sender has not @ in address) What do the auto-responder with broadcast messages? He shouldn't reply too.

  Changed 8 months ago by jens

  • cc Mei_Je@… added

  Changed 8 months ago by smoku

  • status changed from new to accepted

Good idea.

  Changed 8 months ago by smoku

  • status changed from accepted to closed
  • resolution set to fixed

[534] Send vacation reply only to humans (full JIDs). Fixes #186

Note: See TracTickets for help on using tickets.