Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Managesieve 8.9 hangs on filters containing ".. " (dot dot space) #5928

Closed
Kurgan- opened this issue Aug 29, 2017 · 3 comments
Closed

Managesieve 8.9 hangs on filters containing ".. " (dot dot space) #5928

Kurgan- opened this issue Aug 29, 2017 · 3 comments
Milestone

Comments

@Kurgan-
Copy link

Kurgan- commented Aug 29, 2017

Managesieve plugin version 8.9 (in Roundcube 1.3.0) hangs while retrieving contents of a filter that has a text field (it's a vacation auto reply filter) where there is a row that begins whith ".. " (dot dot space)

When it hangs, it runs forever until php times out.

In Apache, I see this in the log:

172.16.2.153 - - [29/Aug/2017:08:48:09 +0200] "GET /?_task=settings&_action=plugin.managesieve HTTP/1.1" 500 3177 "https://webmail.XXXXXXX.com/?_task=settings" "Mozilla/5.0 (Windows NT 10.0; WOW64; rv:54.0) Gecko/20100101 Firefox/54.0"

Previous versions (an old roundcube installation) worked.

Removing the ".. " from the beginning of the text row solved the issue.

Managesieve server is from Dovecot (version 2.2.13), and if I use a different sieve client it does not hang, so it seems Dovecot is not at fault.

The sieve filter that causes the issue is attached.
causes-hang.sieve.txt

@alecpl
Copy link
Member

alecpl commented Aug 29, 2017

Sounds like a duplicate of #5838, could you confirm?

@Kurgan-
Copy link
Author

Kurgan- commented Aug 29, 2017

It could be. If the attached sieve file works properly after the patch proposed in 5838 (which means you can open it in the web interface) I suppose that it's in fact a duplicate.

Thanks

@alecpl alecpl added this to the 1.3.1 milestone Aug 29, 2017
@alecpl
Copy link
Member

alecpl commented Aug 29, 2017

Confirmed. The attached file works for me.

@alecpl alecpl closed this as completed Aug 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants