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

Moscow subway Wi-Fi authorization issue #41

Closed
ameshkov opened this issue Sep 8, 2015 · 10 comments
Closed

Moscow subway Wi-Fi authorization issue #41

ameshkov opened this issue Sep 8, 2015 · 10 comments

Comments

@ameshkov
Copy link
Member

ameshkov commented Sep 8, 2015

Infinite redirect:
https://login.wi-fi.ru/am/UI/Login?org=mac&service=coa&client_mac=xx-xx-xx-xx-xx-xx&ForceAuth=true&redirect=https://login.wi-fi.ru/am/UI/Login

From the log:

13:08:38.310 [pool-3-thread-99] DEBUG c.a.f.proxy.BaseHttpProxyServer - TCP id=205 Processing GET request to http://www.google.com/trends/hottrends/atom/hourly?pn=p14
13:08:38.350 [pool-3-thread-99] DEBUG c.a.f.proxy.BaseHttpProxyServer - TCP id=205 HTTP/1.1 200 OK
Location: https://login.wi-fi.ru/am/UI/Login?org=mac&service=coa&client_mac=c0-bd-d1-c2-9c-27&ForceAuth=true
Content-Length: 351
Content-Type: text/html
13:08:38.352 [pool-3-thread-99] DEBUG c.a.f.filters.HttpResponseFilter - Filtering fixed length response of http://www.google.com/trends/hottrends/atom/hourly?pn=p14
13:08:38.353 [pool-3-thread-99] DEBUG c.a.f.filters.HttpResponseFilter - Response of http://www.google.com/trends/hottrends/atom/hourly?pn=p14 is definitely HTML, filtering it
13:08:38.354 [pool-3-thread-99] DEBUG c.a.f.filters.HttpResponseFilter - Using ContentEditingHtmlParser to parse the response

My idea: that page code filtering breaks authrozation.
We should disable html code filtering if Location header is present.

@ameshkov ameshkov added this to the 2.1 milestone Sep 8, 2015
@ameshkov ameshkov self-assigned this Sep 8, 2015
@ameshkov
Copy link
Member Author

I was wrong, that's not the problem.

Another strange thing in the log file: I see that DNS requests go to 8.8.8.8. Why is it used in Moscow subway?

Logs:
https://www.dropbox.com/s/l6ofv5cg64pwbvp/adguard%20%282%29.log?dl=0
https://www.dropbox.com/s/k6trq2c2rns54w7/adguard%20%283%29.log?dl=0

@ameshkov ameshkov assigned Revertron and unassigned ameshkov Sep 10, 2015
@ameshkov
Copy link
Member Author

Ticket ID: TGX-483-73779

@Revertron
Copy link

Fixed.

@ameshkov
Copy link
Member Author

Are you sure? He still has this issue

@ameshkov ameshkov reopened this Sep 11, 2015
@Revertron
Copy link

In ticket MIW-787-84428 user says that he is using UC Browser. But adblock extension in UC is blocking authorization. In system browser he is connecting without problem.

@ameshkov
Copy link
Member Author

User still has this problem:
TGX-483-73779

@ameshkov ameshkov reopened this Sep 30, 2015
@ameshkov
Copy link
Member Author

@Revertron extend logging in buildVpnService method in the next beta.
Let's log all connection properties including DNS servers (like what's done in addIpv4Routes)

@ameshkov
Copy link
Member Author

ameshkov commented Oct 5, 2015

Waiting for user's answer in TGX-483-73779

@ameshkov ameshkov modified the milestones: 2.2, 2.1 Oct 14, 2015
@vmalyuta
Copy link

1529154
1214655
1529108
open again

@ameshkov
Copy link
Member Author

I am quite sure we should file a report to AdguardFilters.

Also, I am sure we will need a recorded HAR file.

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

3 participants