Skip to content

pjlib-util: forget the cached DNS errors when the nameservers' state is reset - #5328

Open
vlaushkin wants to merge 1 commit into
pjsip:masterfrom
vlaushkin:reset-ns-forgets-refusals
Open

vlaushkin wants to merge 1 commit into
pjsip:masterfrom
vlaushkin:reset-ns-forgets-refusals

Conversation

@vlaushkin

Copy link
Copy Markdown
Contributor

A small follow-up to #5303. The practical gain is modest; it mostly makes the reset consistent.

After an IP change the resolver trusts the nameservers again, but still serves their cached errors, such as a refusal, for up to a minute. During that minute a SIP SRV lookup falls back to the domain's address. This happens only when the nameservers stay the same, e.g. a captive portal that refuses queries before login.

Now the reset drops the cached errors too. Other cached answers stay.

Left out: a lookup in flight during the reset may cache its error again.

Tested in resolver_test.

@vlaushkin

Copy link
Copy Markdown
Contributor Author

The GnuTLS failure in CI Ubuntu is unrelated: pjsip-test aborted in the TCP keep-alive timer, which sent on a transport being closed. Could you rerun it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant