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

Caring of messageId in case of delivery report transit #23

Closed
vetss opened this Issue Feb 21, 2016 · 4 comments

Comments

Projects
None yet
2 participants
@vetss
Collaborator

vetss commented Feb 21, 2016

When SMSC GW is a transit point for SMPP messages (ESME1 -> SMSC GW -> ESME2), there is a possibility for delivery report transit back to originator (ESME2 -> SMSC GW -> ESME1). In this case we need to replace messageId field in the message content of receipt to the value that SMSC GW has returned to ESME1 in the submit_sm_response.

To achieve it we need to setu extra tables in cassandra database (a table per a day). If SMSC GW is configured to provide a receipt transit, at the step of delivering messages into ESME (RxSmpp) SMSC GW will check if a message contains a receipt request and store message data into a database.
At the step of receiving a reciept from SMPP (TxSmpp) SMSC GW will check if it is a receipt and then
look at the database and update the message content.

https://telestax.zendesk.com/tickets/31975#

@vetss vetss added the enhancement label Feb 21, 2016

@deruelle deruelle added this to the 3.0.0 milestone Apr 20, 2016

@vetss vetss self-assigned this Jun 30, 2016

@vetss vetss modified the milestones: 7.1.0, 7.0.0 Aug 3, 2016

@vetss vetss changed the title from Careing of messageId in case of delivery report transit to Caring of messageId in case of delivery report transit Sep 4, 2016

@vetss

This comment has been minimized.

Collaborator

vetss commented Sep 11, 2016

We need to add an extra SMSC GW server level config parameter:

incomeReceiptsProcessing - processing of incoming delivery receipts from remote SMSC GW: replacing of messageId in a receipt by a local messageId
default value: false

CLI:
smsc set incomereceiptsprocessing true
GUI:
Server options - Processing Options

vetss added a commit to vetss/smscgateway that referenced this issue Sep 13, 2016

Fixing RestComm#107 & RestComm#23: Providing an interface to get alre…
…ady sent by SMSC GW

messages to mproc rules, Caring of messageId in case of delivery report
transit

vetss added a commit to vetss/smscgateway that referenced this issue Sep 13, 2016

vetss added a commit to vetss/smscgateway that referenced this issue Sep 13, 2016

@vetss

This comment has been minimized.

Collaborator

vetss commented Sep 13, 2016

@vetss

This comment has been minimized.

Collaborator

vetss commented Sep 22, 2016

Manual is fixed by:
124e486

@vetss vetss closed this Sep 28, 2016

vetss added a commit to vetss/smscgateway that referenced this issue Oct 13, 2016

Fixing RestComm#23: delivery report transit - Adding of one more pars…
…ing for

delivery receipt transit (hex messageId format)

vetss added a commit to vetss/smscgateway that referenced this issue Oct 14, 2016

Fixing of RestComm#23: Caring of messageId in case of delivery report…
… transit -

changing conditions for which transit info is saved

@vetss vetss modified the milestones: 7.2.0, 7.1.0 Nov 17, 2016

@vetss vetss reopened this Nov 17, 2016

vetss added a commit to vetss/smscgateway that referenced this issue Nov 17, 2016

vetss added a commit to vetss/smscgateway that referenced this issue Nov 17, 2016

vetss added a commit to vetss/smscgateway that referenced this issue Nov 17, 2016

Fix for RestComm#23: delivery report transit - caring of Tlv
receipted_message_id case - Extra fix
@vetss

This comment has been minimized.

Collaborator

vetss commented Nov 17, 2016

Adding of Tlv support:
774abd6
5c88134
d2aad03
718d781
2ac3dfe
b08666f

vetss added a commit to vetss/smscgateway that referenced this issue Nov 20, 2016

vetss added a commit to vetss/smscgateway that referenced this issue Nov 20, 2016

vetss added a commit to vetss/smscgateway that referenced this issue Nov 24, 2016

@vetss vetss closed this Dec 22, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment