Opened 5 years ago

Last modified 4 months ago

#10782 new task

Improve the spec of UniformDH

Reported by: asn Owned by:
Priority: Medium Milestone:
Component: Archived/Obfsproxy Version:
Severity: Normal Keywords:
Cc: phw, yawning Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

UniformDH is used by obfs3 and scramblesuit currently, and it might get used by more projects in the future. Yawning suggested to improve its spec to make its adoption easier.

Yawning suggested adding test vectors. We can look at test vectors of other key exchange protocols to see how they should look like. Example:
https://tools.ietf.org/html/rfc6932#appendix-A.1

Some more suggestions:

14:54 < Yawning> *looks at the list of gotchas*
14:54 < Yawning> spec should clarify that 0s are inserted if the public key is shorter than 1536 bits (probably obvious)
14:55 < Yawning> Should clarify that abs(p - X) is sent (99% sure that's what happens)
14:55 < Yawning> spec says to simply raise the public key, when it's another mod exp operation
14:56 < Yawning> apart from "wtb test vectors" those where the things i found
15:00 < Yawning> I also was sort of sad that MAX_PADDING isn't a power of 2, but probably too late to change that and that might have been deliberate

Child Tickets

Change History (8)

comment:1 Changed 5 years ago by phw

Cc: phw added

comment:2 Changed 5 years ago by yawning

Cc: yawning@… added

comment:3 Changed 5 years ago by yawning

comment:4 Changed 5 years ago by arma

Cc: yawning added; yawning@… removed

comment:5 Changed 5 years ago by asn

Component: Pluggable transportObfsproxy

comment:6 Changed 18 months ago by teor

Severity: Normal

Set all open tickets without a severity to "Normal"

comment:7 Changed 4 months ago by teor

Owner: asn deleted
Status: newassigned

asn does not need to own any obfuscation tickets any more. Default owners are trouble.

comment:8 Changed 4 months ago by cohosh

Status: assignednew

tickets are unassigned, reverting to 'new'

Note: See TracTickets for help on using tickets.