Skip to content
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

UDP connection may be closed before it is actually established #175

Closed
ameshkov opened this issue Nov 10, 2015 · 2 comments
Closed

UDP connection may be closed before it is actually established #175

ameshkov opened this issue Nov 10, 2015 · 2 comments
Assignees
Milestone

Comments

@ameshkov
Copy link
Member

Look at this log for instance:

06:09:22.986 [Thread-2511] DEBUG c.a.a.filtering.commons.LogUtils - UDP id=1231 Creating new connection for UDP 172.16.16.1:38042 > 8.8.8.8:53 len=42
06:09:22.986 [Thread-2511] DEBUG c.a.a.filtering.commons.LogUtils - UDP id=1231 Connection has been created.
06:09:22.987 [pool-3-thread-492] DEBUG c.a.a.filtering.commons.LogUtils - UDP id=1231 [OUT] UDP 172.16.16.1:38042 > 8.8.8.8:53 len=42
06:09:22.987 [pool-3-thread-492] DEBUG c.a.a.filtering.commons.LogUtils - UDP id=1231 Open UDP connection to /8.8.8.8:53
06:09:22.987 [pool-4-thread-1] DEBUG c.a.a.filtering.vpn.VpnProxyService - UDP id=1231 Cleaning up connection
06:09:23.029 [Thread-2384] DEBUG c.a.c.io.SocketPollingSelector - UDP id=1231 Read 139 bytes from datagram channel
06:09:23.030 [Thread-2384] DEBUG c.a.a.filtering.commons.LogUtils - UDP id=1231 [IN] UDP 8.8.8.8:53 > 172.16.16.1:38042 len=139

So what happens because of this:

  1. Adguard considers this connection as cleaned up and removes it from the list of active connections.
  2. In fact it stays alive for all the time.
@ameshkov ameshkov added the Bug label Nov 10, 2015
@ameshkov ameshkov self-assigned this Nov 10, 2015
@ameshkov ameshkov added this to the 2.5 milestone Nov 10, 2015
@ameshkov
Copy link
Member Author

@ameshkov ameshkov added the VPN label Nov 10, 2015
@ameshkov ameshkov modified the milestones: 2.1 patch 2, 2.5 Nov 11, 2015
@ameshkov
Copy link
Member Author

Bug is fixed.

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

No branches or pull requests

1 participant