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.

Sign up for GitHub

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

Jump to bottom

Netx research and refactoring in Sprint 14 #576

Closed
1 task done
bassosimone opened this issue May 3, 2020 · 2 comments
Closed
1 task done

Netx research and refactoring in Sprint 14 #576

bassosimone opened this issue May 3, 2020 · 2 comments
Assignees
@bassosimone
Labels
effort/XS Extra small effort priority/low Low priority
Milestone
Sprint 14 - Ponyo

Comments

@bassosimone
Copy link
Contributor

bassosimone commented May 3, 2020 •
edited
Loading

Following up from #543.

  • possibly use deadline during TLS handshake

This refactoring is functional to measure performance metrics. In turn, by doing that we will be able to measure HTTP fetching performance metrics for Psiphon. This is also a precondition to rewrite Web Connectivity in Go.
@bassosimone bassosimone added priority/high High priority effort/L Large effort labels May 3, 2020
@bassosimone bassosimone added this to the Sprint 13 - Orca milestone May 3, 2020
@bassosimone bassosimone self-assigned this May 3, 2020
@bassosimone bassosimone added effort/M Medium effort and removed effort/L Large effort labels May 3, 2020
This was referenced May 3, 2020
Closed
Closed
@bassosimone bassosimone added the effort/XS Extra small effort label May 11, 2020
@bassosimone bassosimone changed the title Netx research and refactoring in Sprint 13 Netx research and refactoring in Sprint 14 May 22, 2020
@bassosimone bassosimone added priority/medium Medium priority effort/S Small effort priority/low Low priority and removed priority/high High priority effort/M Medium effort effort/XS Extra small effort priority/medium Medium priority labels May 25, 2020
bassosimone added a commit that referenced this issue May 28, 2020
@bassosimone
tls/timeout: use deadline rather than context
f348116 
This should lead to a more understandable sequence of low level events
where we don't see anymore the "use of closed network connection".

Part of #576.
This was referenced May 28, 2020
Merged
Closed
@bassosimone bassosimone added effort/XS Extra small effort and removed effort/S Small effort labels May 28, 2020
@bassosimone
Copy link
Contributor Author

Effort reduced to XS because I spent more time with data analysis (#619)

@bassosimone
Copy link
Contributor Author

Done!

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

@bassosimone bassosimone

Labels
effort/XS Extra small effort priority/low Low priority
Projects
None yet
Milestone
Sprint 14 - Ponyo
Development

No branches or pull requests
1 participant
@bassosimone