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

flight3: respect curves configuration #588

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Snawoot
Copy link

@Snawoot Snawoot commented Sep 30, 2023

Description

As far as I understand Flight3 is pretty much like Flight1, but with Cookie. So I just reuse logic from Flight1 to ensure correct behaviour.

Reference issue

Fixes #587

@codecov
Copy link

codecov bot commented Oct 1, 2023

Codecov Report

All modified lines are covered by tests ✅

Files Coverage Δ
flight3handler.go 78.00% <100.00%> (+0.54%) ⬆️

... and 3 files with indirect coverage changes

📢 Thoughts on this report? Let us know!.

Reuse logic from Flight1 to ensure correct behaviour.

Signed-off-by: Vladislav Yarmak <vladislav-ex-src@vm-0.com>
@at-wat
Copy link
Member

at-wat commented May 13, 2024

Seems related to #475 (@daenney FYI)

@Snawoot Could you add a test as well as flight1?

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.

Flight3 does not respect Config.EllipticCurves
2 participants