Opened 4 years ago

Closed 3 years ago

Last modified 3 years ago

#1486142 closed Feature Patches (fixed)

Request for DSN status when sending messages

Reported by: jasb Owned by: alec
Priority: 5 Milestone: 0.5-beta
Component: SMTP connection Version: git-master
Severity: normal Keywords: dsn override
Cc: eric@…

Description

Hi,

When sending a message, there's only the option to ask for a read receipt.
Could be added the option to ask for a DSN (Delivery Status Notification) to make sure the message was delivered?

Thanks,
Jorge,

Change History (7)

comment:1 Changed 4 years ago by alec

RFC 1891

comment:2 Changed 4 years ago by jasb

Sorry Alec!
Can you tell me the ticket number so that i can track that?
I tried to search but didn't found it :P

comment:3 Changed 3 years ago by alec

  • Milestone changed from later to 0.5-beta
  • Owner set to alec

comment:4 Changed 3 years ago by alec

  • Resolution set to fixed
  • Status changed from new to closed

Done in [f22ea7ba].

comment:5 Changed 3 years ago by lacri

  • Cc eric@… added
  • Keywords dsn override added
  • Resolution fixed deleted
  • Status changed from closed to reopened
  • Type changed from Feature Requests to Feature Patches

add option to override dsn settings
in program/steps/mail/compose.inc
add under this line

$OUTPUT->set_env('top_posting', $CONFIG['top_posting']);

this line to set env and display when dsn_default not override

$OUTPUT->set_env('dsn_default', !in_array('dsn_default', (array)$CONFIG['dont_override']));

and in skins/default/templates/compose.html
search to

</tr><tr>
 	    		<td><label for="rcmcomposedsn"><roundcube:label name="dsn" />:</label></td>
         <td><roundcube:object name="dsnCheckBox" form="form" id="rcmcomposedsn" /></td>
 </tr><tr>

and replace with

</tr>
    <roundcube:if condition="env:dsn_default" />
    <tr>
 	    		<td><label for="rcmcomposedsn"><roundcube:label name="dsn" />:</label></td>
         <td><roundcube:object name="dsnCheckBox" form="form" id="rcmcomposedsn" /></td>
    </tr>
    <roundcube:endif />
    <tr>

comment:6 Changed 3 years ago by alec

  • Resolution set to fixed
  • Status changed from reopened to closed

This is not a good solution. Wontfix.

comment:7 Changed 3 years ago by lacri

Sorry my description option to override is not correct ...

a better and correct description or declaration is many servers e.g. exim dont support dsn by default or without patches and dsn is a lot discussed feature if is needed.

my small patch overrides dsn settings and display when the webmail admin thinks i don't need dsn and i will deactivate dsn support and i will not displaying
then i think a not displaying and not supported feature is better than a displayed feature and thats not supported.

because admin can set dsn_default to 0 in config and add dsn_default to override array and the user dont see dsn feature or option and cant set dsn and the user becomes no unnecessary error message ... feature dont support.

Have a nice Weekend Alec :)

Note: See TracTickets for help on using tickets.