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

Log if Same IP is being used or not [Android backport] #5260

Merged

Conversation

faern
Copy link
Member

@faern faern commented Oct 10, 2023

Just backports #5236 to the (previous) Android release branch. So we can expedite a quick release with this logging included.


This change is Reviewable

@faern faern requested a review from dlon October 10, 2023 11:03
@linear
Copy link

linear bot commented Oct 10, 2023

DES-386 Log when Same IP is being used

According to the latest plans with how we roll out Same IP, the app will not know about it at all. The API will just start handing out Same IP IPs to apps. But since there is substantial risk of bugs and issues with it (tricky kernel changes server side) we need good ways of debugging this.

Currently if the app sends a problem report, it's not possible/easy to figure out if the user has been assigned Same IP or not. The logs do not contain account or device identifiers, nor IPs. It would be much easier for us to debug Same IP issues if app logs could reveal whether or not Same IP was used.

Solution

Hardcode the currently known Same IP IPs. And if the device has those IPs assigned to it, log on each connection attempt that Same IP is being used.

if device.ips.contains(SAME_IP_IP) {
    log::debug!("Same IP is being used");
}

This will likely be removed a while after the Same IP rollout reaches 100% and that should be noted in the code.

Copy link
Member

@dlon dlon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@faern faern merged commit 170925b into prepare-android/2023.6 Oct 10, 2023
22 of 27 checks passed
@faern faern deleted the log-when-same-ip-is-being-used-des-386 branch October 10, 2023 11:38
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.

2 participants