[Home]

Summary:ASTERISK-02363: [patch] callerid.c TELEKOM callerid
Reporter:Martin Vit (festr)Labels:
Date Opened:2004-09-07 16:36:58Date Closed:2008-01-15 15:07:05.000-0600
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) czech-telekom-patch-callerid.c
Description:fix callerid_feed for use in czech republic

p.s:
i dont know, if this is right place to post patch for callerid.c

****** ADDITIONAL INFORMATION ******

there is code 2 and 3 for decoding callerid. Czech telekom: 3 for callerid of caller, 4 is our callerid. So i have removed case 3: /* Number (for Zebble) */.

there are also other 2 codes, 17 - empty string, 48 - string of vendor. I dont need this.
Comments:By: Martin Vit (festr) 2004-09-07 16:40:25

edit:
there is code 2 and 3 for decoding callerid. Czech telekom: "2" for callerid of caller, "3" is our callerid. So i have removed case 3: /* Number (for Zebble) */.
By: Mark Spencer (markster) 2004-09-07 23:16:13

I believe in Canada code 3 was used for containing the number which is why t is there.  Your patch seems to delete the case for 3 entirely, so I don't think I understand...
By: Martin Vit (festr) 2004-09-08 04:02:43

aha,

situation in czech telecom:

send 4 messages in this order

2 - caller id of caller
3 - my caller id
17 - empty
48 - vendor of PBX

so 3 will overwrite callerid to my callerid which i dont want

solution for this should be in configurable options somwhere in zapata.conf (or overriding with timezones)
By: Mark Spencer (markster) 2004-09-08 22:27:10

I just made it only take the first number.  That should keep everybody happy.
By: Digium Subversion (svnbot) 2008-01-15 15:07:05.000-0600

Repository: asterisk
Revision: 3750

U   trunk/callerid.c

------------------------------------------------------------------------
r3750 | markster | 2008-01-15 15:07:04 -0600 (Tue, 15 Jan 2008) | 2 lines

Only accept first callerid number (bug ASTERISK-2363)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=3750