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

Implement session resumption using session IDs #1238

Closed
ameshkov opened this Issue May 24, 2017 · 3 comments

Comments

Projects
None yet
1 participant
@ameshkov
Member

ameshkov commented May 24, 2017

We currently do not use it at all which is not very good from the performance point of view. I first wanted to wait for the new network library to be ready (https://github.com/AdguardTeam/CoreLibs), but apparently, there are some things that can be easily done right away.

Tests show that it significantly improves HTTPS filtering performance.

@ameshkov ameshkov added the SSL label May 24, 2017

@ameshkov ameshkov added this to the 2.9 R2 milestone May 24, 2017

@ameshkov ameshkov self-assigned this May 24, 2017

@ameshkov

This comment has been minimized.

Show comment
Hide comment
@ameshkov

ameshkov May 24, 2017

Member

Done, review ID: AFA-CR-11

Member

ameshkov commented May 24, 2017

Done, review ID: AFA-CR-11

@ameshkov ameshkov closed this May 24, 2017

@ameshkov

This comment has been minimized.

Show comment
Hide comment
@ameshkov

ameshkov May 26, 2017

Member

The current implementation is rather problematic because I am trying to re-verify certificate chain.

Member

ameshkov commented May 26, 2017

The current implementation is rather problematic because I am trying to re-verify certificate chain.

@ameshkov ameshkov reopened this May 26, 2017

@ameshkov

This comment has been minimized.

Show comment
Hide comment
@ameshkov

ameshkov May 26, 2017

Member

Fixed, review ID: AFA-CR-16

Member

ameshkov commented May 26, 2017

Fixed, review ID: AFA-CR-16

@ameshkov ameshkov closed this May 26, 2017

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