FreeSWITCH

SWITCH_CAUSE_NO_USER_RESPONSE is translated from 480 to 408

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Not A Bug
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: mod_sofia
  • Labels:
    None
  • Platform:
    Linux x86/gcc
  • CPU Info:
    Hide
    processor : 0
    vendor_id : AuthenticAMD
    cpu family : 16
    model : 6
    model name : AMD Athlon(tm) II X2 240 Processor
    stepping : 2
    cpu MHz : 2800.000
    cache size : 1024 KB
    physical id : 0
    siblings : 2
    core id : 0
    cpu cores : 2
    apicid : 0
    initial apicid : 0
    fpu : yes
    fpu_exception : yes
    cpuid level : 5
    wp : yes
    flags : fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm 3dnowext 3dnow constant_tsc rep_good nonstop_tsc extd_apicid pni monitor cx16 popcnt lahf_lm cmp_legacy svm extapic cr8_legacy abm sse4a misalignsse 3dnowprefetch osvw ibs skinit wdt
    bogomips : 5627.38
    TLB size : 1024 4K pages
    clflush size : 64
    cache_alignment : 64
    address sizes : 48 bits physical, 48 bits virtual
    power management: ts ttp tm stc 100mhzsteps hwpstate

    processor : 1
    vendor_id : AuthenticAMD
    cpu family : 16
    model : 6
    model name : AMD Athlon(tm) II X2 240 Processor
    stepping : 2
    cpu MHz : 800.000
    cache size : 1024 KB
    physical id : 0
    siblings : 2
    core id : 1
    cpu cores : 2
    apicid : 1
    initial apicid : 1
    fpu : yes
    fpu_exception : yes
    cpuid level : 5
    wp : yes
    flags : fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm 3dnowext 3dnow constant_tsc rep_good nonstop_tsc extd_apicid pni monitor cx16 popcnt lahf_lm cmp_legacy svm extapic cr8_legacy abm sse4a misalignsse 3dnowprefetch osvw ibs skinit wdt
    bogomips : 5628.00
    TLB size : 1024 4K pages
    clflush size : 64
    cache_alignment : 64
    address sizes : 48 bits physical, 48 bits virtual
    power management: ts ttp tm stc 100mhzsteps hwpstate
    Show
    processor : 0 vendor_id : AuthenticAMD cpu family : 16 model : 6 model name : AMD Athlon(tm) II X2 240 Processor stepping : 2 cpu MHz : 2800.000 cache size : 1024 KB physical id : 0 siblings : 2 core id : 0 cpu cores : 2 apicid : 0 initial apicid : 0 fpu : yes fpu_exception : yes cpuid level : 5 wp : yes flags : fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm 3dnowext 3dnow constant_tsc rep_good nonstop_tsc extd_apicid pni monitor cx16 popcnt lahf_lm cmp_legacy svm extapic cr8_legacy abm sse4a misalignsse 3dnowprefetch osvw ibs skinit wdt bogomips : 5627.38 TLB size : 1024 4K pages clflush size : 64 cache_alignment : 64 address sizes : 48 bits physical, 48 bits virtual power management: ts ttp tm stc 100mhzsteps hwpstate processor : 1 vendor_id : AuthenticAMD cpu family : 16 model : 6 model name : AMD Athlon(tm) II X2 240 Processor stepping : 2 cpu MHz : 800.000 cache size : 1024 KB physical id : 0 siblings : 2 core id : 1 cpu cores : 2 apicid : 1 initial apicid : 1 fpu : yes fpu_exception : yes cpuid level : 5 wp : yes flags : fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm 3dnowext 3dnow constant_tsc rep_good nonstop_tsc extd_apicid pni monitor cx16 popcnt lahf_lm cmp_legacy svm extapic cr8_legacy abm sse4a misalignsse 3dnowprefetch osvw ibs skinit wdt bogomips : 5628.00 TLB size : 1024 4K pages clflush size : 64 cache_alignment : 64 address sizes : 48 bits physical, 48 bits virtual power management: ts ttp tm stc 100mhzsteps hwpstate
  • FreeSWITCH GIT Revision:
    HEAD
  • Reproduced with GIT HEAD?:
    YES

Description

sofia_glue_sip_cause_to_freeswitch() interprets a 480 as SWITCH_CAUSE_NO_USER_RESPONSE.

hangup_cause_to_sip() on the other hand interprets SWITCH_CAUSE_NO_USER_RESPONSE as 408.

Since 408 means "Request Timeout" the latter is IMHO wrong.
hangup_cause_to_sip() should rather interpret NO_USER_RESPONSE as 480.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Anthony Minessale II added a comment -
The translation is derived from RFC 4497 and they say cause 18 should be 408
I'm reluctant to go against unless there is a compelling reason.

 
Show
Anthony Minessale II added a comment - The translation is derived from RFC 4497 and they say cause 18 should be 408 I'm reluctant to go against unless there is a compelling reason.  
Hide
Permalink
Markus Bucher added a comment -
OK, I see that it is the RFC that has this "bug".
Since there is defined the translation of 480 to cause 18, too.

My reason for changing this would be that if you pass through the hangup cause of leg B, the UA on leg A will get a wrong impression of what actually went wrong.
Show
Markus Bucher added a comment - OK, I see that it is the RFC that has this "bug". Since there is defined the translation of 480 to cause 18, too. My reason for changing this would be that if you pass through the hangup cause of leg B, the UA on leg A will get a wrong impression of what actually went wrong.
Hide
Permalink
Ognjen Seslija added a comment -
See FS-4824.
This is why Reason header comes to rescue.
Show
Ognjen Seslija added a comment - See FS-4824 . This is why Reason header comes to rescue.
Hide
Permalink
Jeff Lenk added a comment -
Setting to resolved as this is working as needed
Show
Jeff Lenk added a comment - Setting to resolved as this is working as needed
Hide
Permalink
Auto Admin added a comment -
Due to a long period of inactivity (13 or more days), this issue is due to be automatically close within 24 hours.
If this issue is not actually resolved, please reopen it and add appropriate comments to help developers fix the issue.

Thanks,
  Jira Admin
Show
Auto Admin added a comment - Due to a long period of inactivity (13 or more days), this issue is due to be automatically close within 24 hours. If this issue is not actually resolved, please reopen it and add appropriate comments to help developers fix the issue. Thanks,   Jira Admin
Hide
Permalink
Auto Admin added a comment -
Since there has been no change to the status of this issue, it is being closed for inactivity

Thanks,
  Jira Admin
Show
Auto Admin added a comment - Since there has been no change to the status of this issue, it is being closed for inactivity Thanks,   Jira Admin

People

  • Assignee:
    Anthony Minessale II
    Reporter:
    Markus Bucher
Vote (0)
Watch (3)

Dates

  • Created:
    Updated:
    Resolved: