#733 Item count limits for archive stores

Reporter Ge0rG
Owner MattJ
Created
Updated
Stars ★★★ (5)
Tags
  • Status-Fixed
  • Priority-High
  • Milestone-0.12
  • Type-Enhancement
  1. Ge0rG on

    To prevent server storage DoS, there should be per-user limits to the disk space allocated to individual users. While having such a limit might not be practical, it would already be great to have a per-user limit on the number of offline messages and MAM messages. I'm not sure which behaviour should ensue when the storage is full (sending an error sounds reasonable for mod_offline, but mod_mam should maybe just kill the oldest messages).

  2. Lunar on

    I'd like to see this addition as well. I've witnessed a DoS where one user received over 17GB in offline messages.

  3. Zash on

    Changes
    • tags Status-Accepted
  4. MattJ on

    Decided to go with a limit on the number of items in the store, as this is the easiest to implement across storage backends. It doesn't have to be 100% accurate.

    Changes
    • title Disk space limits for mod_offline and mod_mam Item count limits for archive stores
  5. Zash on

    Changes
    • owner MattJ
    • tags Status-Started
  6. MattJ on

    Changes
    • tags Milestone-0.11
  7. MattJ on

    This is almost complete, but has a couple of minor issues before we can merge it. It won't be ready for 0.11.

    Changes
    • tags Milestone-0.11 Milestone-0.11
  8. MattJ on

    Changes
    • tags Milestone-0.11 Milestone-0.12
  9. MattJ on

    Changes
    • tags Priority-High
  10. Zash on

    Done in https://hg.prosody.im/trunk/rev/f76bd399267c and following commits

    Changes
    • tags Status-Fixed
  11. dittler on

    great news. thanks!

New comment

Not published. Used for spam prevention and optional update notifications.