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

Connection information section shows incorrect IP(v6) - NOT actual webserver #175

Closed
vom513 opened this issue Feb 10, 2020 · 6 comments
Closed

Comments

@vom513
Copy link

vom513 commented Feb 10, 2020

Affected Version:

2.0.2(73)

Is this a Test Flight version or the App Store version?

App Store

Device and iOS Version:

iPhone X - 13.3.1

What steps will reproduce the problem?

  1. Enter "burn.net" into app.
  2. Observe "connection information".

What is the expected output?

The IP address of burn.net: 2607:f2f8:a4c8::2

What do you see instead?

An IP address from the same prefix, but I have no idea what the specific host bits are corresponding to:

2607:f2f8:a4c8:0:8100:e32b:7d3a:b5a4

Please provide any additional information below.

Screenshot - https://imgur.com/a/pGbdXlS

@vom513
Copy link
Author

vom513 commented Feb 10, 2020

Sorry for replying to myself, but I tried another site (arin.net). Similar behavior. Looks like the "connection information" has the first 4 hextets correct. After that it's "8100:e32b:7d3a:b5a4" again for me.

@ecnepsnai
Copy link
Member

Hi @vom513 what crypto engine are you using (Look under Advanced Settings). Does this problem happen with both iOS and OpenSSL ?

@vom513
Copy link
Author

vom513 commented Feb 11, 2020

It's currently set to iOS (I assume this is the default - I didn't set this).

Setting this to OpenSSL has the same behavior (I put it back in iOS after testing).

A couple more points:

  • This also happens on my iPad (latest iPad OS).
  • The "right side" of this address is different now. I still have the correct leading hextets. In the case of burn.net - this is 2607:f2f8:a4c8:0. I notice that this is exactly half the address.

@ecnepsnai
Copy link
Member

ecnepsnai commented Feb 16, 2020

I'm still looking into this. The logic used to get the servers IP address isn't complex so I'm not ruling out that this is potentially an iOS bug.

int sock_fd = <...> // Assume I already have the fd
struct sockaddr addr;
socklen_t addr_len = sizeof(addr);
getpeername(sock_fd, &addr, &addr_len);
char addressString[INET6_ADDRSTRLEN];
inet_ntop(AF_INET6, &((struct sockaddr_in6 *)&addr)->sin6_addr, addressString, INET6_ADDRSTRLEN);

@ecnepsnai ecnepsnai added this to the 2.0.3 milestone Feb 17, 2020
@ecnepsnai
Copy link
Member

Fix identified - will be included in next update.

@vom513
Copy link
Author

vom513 commented Feb 17, 2020

Nice - appreciate the work on this.

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

No branches or pull requests

2 participants