Patch: wap_push_ppg.c

"Nikos Balkanas" <[email protected]>
Newsgroups gmane.comp.mobile.kannel.devel
Message-ID <005901ca5bd0$2401bfa0$02b2a8c0@tardis>
Hi,

Another trivial patch for wap. The code disables dlrs, if no dlr_url is specified, despite if dlr_mask is set. These are 2 different variables, and I don't see why they should be linked like that. I just want the DLR in my access logs, don't need the URL. I shouldn't have to setup a web server just to get the DLR from the SMSc.

BR,
Nikos
wap_push_ppg.diff (application/octet-stream, 554 B)
Index: gw/wap_push_ppg.c
===================================================================
RCS file: /home/cvs/gateway/gw/wap_push_ppg.c,v
retrieving revision 1.76
diff -a -u -r1.76 wap_push_ppg.c
--- gw/wap_push_ppg.c	12 Jan 2009 16:46:56 -0000	1.76
+++ gw/wap_push_ppg.c	2 Nov 2009 15:11:40 -0000
@@ -3319,10 +3319,6 @@
     long dlr_mask;
     long masklen;    
 
-    if (dlr_url == NULL) {
-        return 0; 
-    }
-
     dlrmaskos = http_header_value(headers, octstr_imm("X-Kannel-DLR-Mask"));
     if (dlrmaskos == NULL) { 
         return 0;
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.