Skip to content

Comments

fix: extMasterSecret mismatch with extended_master_secret extension#307

Merged
gaukas merged 1 commit intorefraction-networking:masterfrom
adotkhan:fix-ems
Jul 20, 2024
Merged

fix: extMasterSecret mismatch with extended_master_secret extension#307
gaukas merged 1 commit intorefraction-networking:masterfrom
adotkhan:fix-ems

Conversation

@adotkhan
Copy link
Contributor

Fixes an issue where clientHelloMsg.extendedMasterSecret/PubClientHelloMsg.Ems is always true, even if the extended_master_secret extension is not set in the Client Hello (e.g. from a random TLS profile).

Copy link
Contributor

@gaukas gaukas left a comment

Choose a reason for hiding this comment

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

Thanks for the fix! This change looks good as it should be the responsibility of (*ExtendedMasterSecretExtension).writeToUConn(uc *UConn) to set this field.

@gaukas gaukas merged commit 23de245 into refraction-networking:master Jul 20, 2024
adotkhan added a commit to Psiphon-Labs/utls that referenced this pull request Dec 10, 2024
adotkhan added a commit to Psiphon-Labs/utls that referenced this pull request Dec 10, 2024
adotkhan added a commit to Psiphon-Labs/utls that referenced this pull request Feb 5, 2025
sardanioss pushed a commit to sardanioss/utls that referenced this pull request Jan 16, 2026
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