-
Notifications
You must be signed in to change notification settings - Fork 58
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
DNS is new resolved for a new Dialog after "Authentication Required" -> Authentication failes #22
Comments
Here is a Log for this bevaviour
|
Agreed this is a problem. I've noticed it before myself. Will look into this this week. |
Perfect, thx a lot
Von meinem iPhone gesendet
Am 22.11.2018 um 17:50 schrieb Dave Horton <notifications@github.com<mailto:notifications@github.com>>:
Agreed this is a problem. I've noticed it before myself. Will look into this this week.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#22 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AYvpiDx6uQWmywKZXMmyrLkPLkFBTn83ks5uxtXFgaJpZM4Yvc3Y>.
|
Hi, So some headers (or parameters on headers) comming from the SIP Phone (in the Proxy scenario), suppress the new DNS Resolve in DrachtIO Server? (VIA, RecordRoute?) |
The DNS resolve issue I would expect you to experience when doing an app that is either a UAC or a B2BUA and receive a challenge to an INVITE (or REGISTER) request. When proxying, I think I may be stashing the original server is proxying there -- need to check that. In any case, I will shortly have a fix for you to test with your original test case. Do you have the ability to test with a version that I tag in github, rather than pulling from npmjs ? I'd like to have your tests pass first before I publish to npm |
I pushed a fix to github, you should be able to test it by changing your package.json to have
can you try testing with that? If it works for you then I will push it to npm as version 4.4.1 |
I can try this on Monday, i‘m not in the office for the weekend
Von meinem iPhone gesendet
Am 23.11.2018 um 18:48 schrieb Dave Horton <notifications@github.com<mailto:notifications@github.com>>:
I pushed a fix to github, you should be able to test it by changing your package.json to have
drachtio-srf: davehorton/drachtio-srf#v4.4.1-rc1
can you try testing with that? If it works for you then I will push it to npm as version 4.4.1
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#22 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AYvpiMvOm53jtpr_bc9kj8b_9J62kn49ks5uyDTygaJpZM4Yvc3Y>.
|
Works with our provider. Perfect. |
Sorry, possible bug from our side |
Our Provider uses DNS to spread request over different machines
Take a look at ``nslookup sip.solucon.com"
Every DNS Call gets a new TOP-HIT.
This ends in a Problem:
IF you Start an INVITE Dialog and gets an Authentication Required the new Dialog with prepard Authentication Headers goes to another Server, which does not know anthing about the nounce etc.
IF you fix the Sip Adress from "sip.solucon.com" to 92.197.181.228 everything is fine.
So DNS should only be resolved per CALLID, not per Request or dialog
The text was updated successfully, but these errors were encountered: