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

RC4 security in TLS #8

Closed
hannesm opened this issue Mar 3, 2014 · 4 comments · Fixed by #223
Closed

RC4 security in TLS #8

hannesm opened this issue Mar 3, 2014 · 4 comments · Fixed by #223

Comments

@hannesm
Copy link
Member

hannesm commented Mar 3, 2014

problem: some bytes can be recovered

solution: throw RC4 away and switch to CBC

references

@hannesm
Copy link
Member Author

hannesm commented Oct 1, 2014

there's now even a proposed standard rfc, shepherded... http://www.ietf.org/id/draft-ietf-tls-prohibiting-rc4-01.txt

@hannesm
Copy link
Member Author

hannesm commented Oct 23, 2014

talking to @Andreas23 I think we should remove RC4 from the default_config ciphers (but still list as supported, since our implementation does support RC4).

hannesm added a commit that referenced this issue Jan 12, 2015
disable RC4 by default (users who need RC4 can tweak the configuration)

fixes #8
@pqwy pqwy closed this as completed in #223 Jan 15, 2015
@g2p
Copy link
Contributor

g2p commented Mar 16, 2015

Is RC4 really disabled, or just not advertised as per the commit message?
The RFC says RC4 must never be negotiated (Firefox roadmap): https://tools.ietf.org/html/rfc7465
Otherwise an active attacker would drop a connection and force the use of RC4.

@hannesm
Copy link
Member Author

hannesm commented Mar 16, 2015

in the default config in master, RC4 is disabled -- it can not be negotiated. You can still configure OCaml-TLS to use RC4. The 0.3.0 release still advertises RC4 by default (but there'll be a 0.4.0 really soon now)!
As example, https://ownme.ipredator.se will never negotiate RC4 (while https://tls.openmirage.org will).

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

Successfully merging a pull request may close this issue.

2 participants