Non-queue runners should not create var/queue/* directories

Bug #1095422 reported by Barry Warsaw
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
GNU Mailman
Low
Unassigned

Bug Description

The LMTP and REST runners are not queue runners. They are daemon processes which do not scan directories. They should not create directories in var/queue, probably by disabling the creation of the Switchboard in their subclass.

Related branches

Revision history for this message
Sandesh Kumar Agrawal (sakuag333) wrote :

in mailman.app.config , there is function _expand_paths(self) which contains :

queue_dir = category.queue_dir

this results in queue_dir to get added to self.path

and thus when ensure_directories_exist() is being called in config.py , it make a directory var/queue

removing "queue_dir = category.queue_dir " will not allow the directory to be formed.

Since i am new to mailman, can you please guide me , how to push this change to mailman 3 code ?

Barry Warsaw (barry)
Changed in mailman:
status: Triaged → Fix Committed
Barry Warsaw (barry)
Changed in mailman:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers