View Issue Details

This bug affects 1 person(s).
 2
IDProjectCategoryView StatusLast Update
13145Bug reportsSurvey participants (Tokens)public2018-01-16 15:15
Reportersnowsam Assigned Toc_schmitz  
PrioritynoneSeveritypartial_block 
Status closedResolutionfixed 
Product Version3.0.0-rc.x 
Summary13145: Unable to process IMAP bounces
Description

At https://github.com/LimeSurvey/LimeSurvey/blob/master/application/controllers/admin/tokens.php#L178
there is the line

$header = explode("\r\n", imap_body($mbox, $sMessageID, FT_UID & FT_PEEK)); // Don't mark messages as read

It should instead be:

$header = explode("\r\n", imap_body($mbox, $sMessageID, FT_UID | FT_PEEK)); // Don't mark messages as read

That difference in listing the multiple FT Flags was making LS not get anything for the message body, which meant it was never getting a match.

Could someone confirm and make a patch?

Steps To Reproduce

Go process deleted from an imap server.

Additional Information

Discussed at https://www.limesurvey.org/forum/installation-a-update-issues/112883-bounce-processing-false-negatives

TagsNo tags attached.
Bug heat2
Complete LimeSurvey version number (& build)Version 3.0.1+171228
I will donate to the project if issue is resolvedNo
BrowserFirefox 57.0.4 (64-bit) on windows
Database type & versionMySQL 5.5.53
Server OS (if known)Debian
Webserver software & version (if known)Apache2
PHP Version5.6.29-0+deb8u1

Users monitoring this issue

There are no users monitoring this issue.

Activities

c_schmitz

c_schmitz

2018-01-15 12:01

administrator   ~45800

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=26026

c_schmitz

c_schmitz

2018-01-16 15:15

administrator   ~45842

Version 3.0.4 released

Related Changesets

LimeSurvey: master e31d64f7

2018-01-15 12:01:30

c_schmitz

Details Diff
Fixed issue 13145: Unable to process IMAP bounces Affected Issues
13145
mod - application/controllers/admin/tokens.php Diff File

Issue History

Date Modified Username Field Change
2018-01-08 22:37 snowsam New Issue
2018-01-15 10:51 LouisGac Assigned To => c_schmitz
2018-01-15 10:51 LouisGac Status new => assigned
2018-01-15 12:01 c_schmitz Changeset attached => LimeSurvey master e31d64f7
2018-01-15 12:01 c_schmitz Note Added: 45800
2018-01-15 12:01 c_schmitz Resolution open => fixed
2018-01-15 12:01 c_schmitz Status assigned => resolved
2018-01-15 12:01 c_schmitz Fixed in Version => 3.0.3
2018-01-15 12:02 c_schmitz Fixed in Version 3.0.3 =>
2018-01-16 15:15 c_schmitz Note Added: 45842
2018-01-16 15:15 c_schmitz Status resolved => closed