Warning: Can't synchronize with repository "(default)" (No changeset 96d22ec3fa3ef6de3ea8dc0d7d398adc9aa071cf in the repository). Look in the Trac log for more information.

Ticket #353 (closed defect: fixed)

Opened 4 years ago

Last modified 4 years ago

Incoming message is considered stored unsent

Reported by: PaulFertser Owned by: daniel
Priority: minor Milestone: milestone5.5
Component: framework/ogsmd Version:
Keywords: Cc:

Description

I received an incoming SMS just in time for Zhone to check for stored unsent messages and it sent it back :)

2009.02.08 12:02:02 ogsmd    DEBUG    (<MiscChannel via /dev/pts/1>: last communication with modem was 7 seconds ago. Sending EOF to wakeup)
2009.02.08 12:02:03 ogsmd.channel DEBUG    <MiscChannel via /dev/pts/1>: sending 11 bytes: 'AT+CMGL=2\r\n'
2009.02.08 12:02:07 ogsmd.channel DEBUG    <MiscChannel via /dev/pts/1>: _readyToRead: watch timeout = 5734
2009.02.08 12:02:07 ogsmd.channel DEBUG    <MiscChannel via /dev/pts/1>: got 61 bytes from: '\r\n+CMGL: 14,0,"005000610073006800610020004B006100620061",77\r\n'
2009.02.08 12:02:07 ogsmd.channel DEBUG    <MiscChannel via /dev/pts/1>: _readyToRead: watch timeout = 5734
2009.02.08 12:02:07 ogsmd.channel DEBUG    <MiscChannel via /dev/pts/1>: got 178 bytes from: '07919761989901F0240B919761137480F20008902080212053213A042104350433043E0434043D044F002004320435044704350440043E043C00200438002004370430043204420440043000200434043D0435043C\r\n\r\nOK\r\n'
2009.02.08 12:02:07 ogsmd.channel DEBUG    <MiscChannel via /dev/pts/1>: COMPLETED 'AT+CMGL=2' => ['+CMGL: 14,0,"005000610073006800610020004B006100620061",77', '07919761989901F0240B919761137480F20008902080212053213A042104350433043E0434043D044F002004320435044704350440043E043C00200438002004370430043204420440043000200434043D0435043C', 'OK']
2009.02.08 12:02:07 ogsmd.channel DEBUG    <MiscChannel via /dev/pts/1>: _readyToSend: watch timeout = None
2009.02.08 12:02:07 ogsmd.channel DEBUG    <MiscChannel via /dev/pts/1>: _readyToSend: watch timeout = None
2009.02.08 12:02:07 ogsmd.channel DEBUG    <MiscChannel via /dev/pts/1>: sending 12 bytes: 'AT+CMSS=14\r\n'
2009.02.08 12:02:08 ogsmd.channel DEBUG    <UnsolicitedResponseChannel via /dev/pts/2>: _readyToRead: watch timeout = None
2009.02.08 12:02:08 ogsmd.channel DEBUG    <UnsolicitedResponseChannel via /dev/pts/2>: got 14 bytes from: '\r\n%CPRI: 1,2\r\n'
2009.02.08 12:02:08 ogsmd.device INFO     org.freesmartphone.GSM.Network.CypherStatus: enabled: unknown
2009.02.08 12:02:12 ogsmd.channel DEBUG    <MiscChannel via /dev/pts/1>: _readyToRead: watch timeout = 5737
2009.02.08 12:02:12 ogsmd.channel DEBUG    <MiscChannel via /dev/pts/1>: got 20 bytes from: '\r\n+CMSS: 188\r\n\r\nOK\r\n'
2009.02.08 12:02:12 ogsmd.channel DEBUG    <MiscChannel via /dev/pts/1>: COMPLETED 'AT+CMSS=14' => ['+CMSS: 188', 'OK']

Change History

comment:1 Changed 4 years ago by jluebbe

Zhone explicitly checks for unsent, so this seems to be a modem bug...

Mickey, is there a way around this except checking twice?

comment:2 Changed 4 years ago by daniel

Can you please show the portion of the log where you receive the message? Somehow I can't really believe that as incoming and outgoing SMS have a slightly different format. Did this message arrive at the destination?

comment:3 Changed 4 years ago by PaulFertser

I checked several times that there were not any incoming message notifications prior to that. Nothing related to messages, actually.

And i can confirm that this message was really sent to me by friend and was immediately (heard that with GSM interference on speakers) sent back, he confirmed that he delivered the same message he sent.

comment:4 Changed 4 years ago by daniel

  • Status changed from new to assigned
  • Owner changed from mickey to daniel

I can't really see the correct way to handle this..

Apart from the issue that the modem seems to be broken in that regard (no wonder here) and we can't really do anything about it I believe the "feature" of zhone checking for unsent messages and sending them really has bitten us one too many times now.

We should just disable that as sending SMS is pretty reliable now, no need for this cruft to exist.

comment:5 Changed 4 years ago by daniel

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

I removed the timer in zhone in commit [9c204f11bfc37d388c44acf639abb124fa832ab0] so this should be fixed now.

I don't know what we can/should do about the "incoming messages appear as unsent" issue, though. If you have any more ideas, please tell.

Note: See TracTickets for help on using tickets.