Resolved In Schedule script cant find addional file who is needed

[hintzsche]

Hello,

i created a script that deletes all emails older than 90 days for my server. Then i created a schedule in schedules and tasks and entered the /script.sh in run a command. When i test run the schedule it tells me that the script cant find the file where the user data is stored. The user data file is in the same direcotry as the script file. Where is my mistake? Thanks for your help.

 

[Nik G]

hello @hintzsche
could you please share your script?

 

[Kaspar]

Then i created a schedule in schedules and tasks and entered the /script.sh in run a command. When i test run the schedule it tells me that the script cant find the file where the user data is stored. The user data file is in the same direcotry as the script file. Where is my mistake? Thanks for your help.

You'll probably need to specify the full directory path.

Also, as an alternative, you can use the doveadm utility (if you have Dovecot installed).

For example, you can run the following doveadm command to delete messages from the Inbox folder of info@example.com which are older than 90 days.

doveadm expunge -u "info@example.com" mailbox INBOX before 90d

Or to delete messages for all users on all mailbox folders older than 90 day you can use:

doveadm expunge -A mailbox '*' before 90d

More info on doveadm expunge: Tools/Doveadm/Expunge - Dovecot Wiki

 

[hintzsche]

the mbox.txt is in the same directory as the script.sh

the script is:

#!/bin/bash
for mailbox in $(cat mbox.txt); do
doveadm expunge -u $mailbox mailbox 'INBOX' before 90d
done

I know doveadm this is what the script is using. but i want to use the mbox.txt file cause then i just have to change the txt file if I want to add more email adresses. what do you mean with the exact path both files are in the root directory /. if i chane mbox.txt to /mbox.txt or ./mbox.txt nothing changes. thats not the problem i guess.

 

[Kaspar]

Try to specify the file path with double quotes. Like $(cat "\mbox.txt");

 

[Kaspar]

Also, you can use the -F option on the doveadm expunge utility to specify a file with users. Which would eliminate the need for a for loop. Leaving you with a nice onliner as a script

doveadm expunge -F "/mbox.txt" mailbox 'INBOX' before 90d

(I have not tested this, but according this the documentation this should work)

 

[hintzsche]

Thank you very much. But when i use the command without a script i have to write a comand for every single folder? I mean i want to delete emails in every folder exept archive.

 

[Kaspar]

But when i use the command without a script i have to write a comand for every single folder?

Sorry, I don't quite understand what you're saying. Why would you have to write a command for every single folder?

 

[hintzsche]

i dont want to, i have to when i just use the doveadm command. Because doveadm expunge -u user@domain.com mailbox INBOX before 90d is one comand and one folder. That would be a very long list of commands for every user and a folder In Schedules and tasks. Therefore i need a script to specify Folders and an extra file to easely change or add users.

 

[Kaspar]

Perhaps I misunderstand what you're saying. Let me reiterate. Your opening question was about getting your code to work when running a cronjob. You can solve that by wrapping the file path in your code in double quotes. Like $(cat "\mbox.txt") instead of $(cat mbox.txt).

My previous post was merely a suggestion on how to shorten your code with the same result. Because this code:

for mailbox in $(cat "\mbox.txt"); do
    doveadm expunge -u $mailbox mailbox INBOX before 90d
done

does the same as this code (but it is shorter).

doveadm expunge -F "/mbox.txt" mailbox INBOX before 90d

Obviously both get the job done just fine.

 

[hintzsche]

The actual script is longer. i shortened it because it is not neccesarry for understanding the problem. So i have to change the script from

#!/bin/bash
for mailbox in $(cat mbox.txt); do
doveadm expunge -u $mailbox mailbox 'INBOX' before 90d
" " mailbox.SEND' before 90d
" ".DRAFT before 90d
done

to

#/bin/bash
do
doveadm expunge -F "/mbox.txt" mailbox 'INBOX' before 90d
doveadm expunge -F "/mbox.txt" mailbox mailbox.SEND before 90d
so on and so forth
done

have i understand that correct? Sorry i thought you mean i shall use this command instead of the script

 

[Kaspar]

Ohw, I see. Yes, in that case my suggestion does not help you much. You still need to use the for loop because of all the mailbox folders.