Skip to content

/AdguardForAndroid

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

Jump to bottom

We should flush DnsCache on protection start #91

Closed
ameshkov opened this issue Oct 15, 2015 · 3 comments
Closed

We should flush DnsCache on protection start #91

ameshkov opened this issue Oct 15, 2015 · 3 comments
Assignees
@Revertron
Labels
Bug
Milestone
2.1 patch

Comments

@ameshkov
Copy link
Member

@ameshkov ameshkov commented Oct 15, 2015

Look at HttpRemoteEndPoint class, DNS cache is used there as a static variable.
It can cause lots of issues when user is in Manual proxy mode as cache is never flushed.
@ameshkov ameshkov added the Bug label Oct 15, 2015
@ameshkov ameshkov added this to the 2.2 milestone Oct 15, 2015
@Revertron

This comment has been minimized.

Sign in to view
Copy link
Member

@Revertron Revertron commented Oct 15, 2015

Done.
@Revertron Revertron closed this Oct 15, 2015
@ameshkov ameshkov modified the milestones: 2.2, 2.1 patch Oct 16, 2015
@ameshkov

This comment has been minimized.

Sign in to view
Copy link
Member Author

@ameshkov ameshkov commented Oct 19, 2015

UPD: we should also clear it on network reconnect.
@ameshkov ameshkov reopened this Oct 19, 2015
@Revertron

This comment has been minimized.

Sign in to view
Copy link
Member

@Revertron Revertron commented Oct 20, 2015

Done.
@Revertron Revertron closed this Oct 20, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Revertron Revertron

Labels
Bug
Projects
None yet
Milestone
  2.1 patch
2 participants
@ameshkov @Revertron
You can’t perform that action at this time.