RestComm / cloudhopper-smpp

Efficient, scalable, and flexible Java implementation of the Short Messaging Peer to Peer Protocol (SMPP)
Other
14 stars 22 forks source link

Issue with sending multi-part messages through CloudHopper #27

Open omkusre opened 5 years ago

omkusre commented 5 years ago

Hi Team,

We are using cloud hopper library for sending SMS messages across various geographies through our partners. We have observed one peculiar issue with message sending (MT Messages).

In the DefaultSmppSession, we are using method sendRequestPdu(pdu, timeout, true). This method seem to have 2 sections,

  1. To create offer - This section typically returns a future object based on window size and stores it in concurrentHashMap.
  2. To Send message through Channel - This section actually sends message over network through jboss/netty library.

The issue we have observed is, these 2 sections are pretty much out of sync.

We also tried to log freeSize when exception occurred, it still seem to have enough window to accommodate the submit_sm/s we are trying to send.

Log's Extract:

Due to this issue, we have observed random failures for submit_sms when traffic is increased. This is primarily observed for multiple part messages. Could you please take a look at this issue on priority and advise.

Thanks, Omkar Kusre