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

Fix NPE in connection quality #4039

Merged
merged 1 commit into from
Dec 22, 2023
Merged

Fix NPE in connection quality #4039

merged 1 commit into from
Dec 22, 2023

Conversation

aitorvs
Copy link
Collaborator

@aitorvs aitorvs commented Dec 22, 2023

Task/Issue URL: https://app.asana.com/0/488551667048375/1206222242762043/f

Description

Fix NPE when aggregating connection quality

Steps to test this PR

Requirements:

  • test in emulator
  • mac computer connected to VPN

Test

  • filter logcat by Connection Quality
  • install from this branch and enable VPN
  • see connection quality measurements
  • increase latency by connecting your computer to an egress very far away
  • see connection quality measurements increasing and changing class

@aitorvs
Copy link
Collaborator Author

aitorvs commented Dec 22, 2023

Current dependencies on/for this PR:

This stack of pull requests is managed by Graphite.

@aitorvs aitorvs merged commit 951d355 into develop Dec 22, 2023
7 checks passed
@aitorvs aitorvs deleted the fix/aitor/vpn/npe branch December 22, 2023 12:31
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