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

In House Erlang Patch #8

Open
zkessin opened this issue Mar 27, 2014 · 2 comments
Open

In House Erlang Patch #8

zkessin opened this issue Mar 27, 2014 · 2 comments

Comments

@zkessin
Copy link
Contributor

zkessin commented Mar 27, 2014

The package for erlang is not using the ESL or OTP repo, but to the branch new_crypto_dtls on @RoadRunnr 's repo. This branch patches a bunch of SSL code, in ways that I am not qualified to evaluate.

  1. The development runtime and the production runtime are not the same, which can cause bugs that are not reproducible
  2. Crypto code is very hard to get right, and I for one do not have the know how to evaluate this
@zkessin zkessin added the bug label Mar 27, 2014
@zkessin zkessin changed the title Non Standard Erlang In House Erlang Patch Mar 27, 2014
@RoadRunnr
Copy link
Member

The change has been submitted upstream and upstream is in the process of integrating it. Until that
time this is a required product feature with ZERO alternatives.

  1. The development runtime and the production runtime are not the same, which can cause bugs that are not reproducible

wrong, the Erlang runtime in question is only used in SCG builds, the development runtime for the SCG is exactly the same, otherwise the development would not work

  1. Crypto code is very hard to get right, and I for one do not have the know how to evaluate this

First, this does not touch crypto code, it only touches the state machine that feeds the crypto code.

When you can't evaluate this code and see this as a problem, why are you trusting the Erlang SSL App in the first place which you can't evaluate either?

@zkessin
Copy link
Contributor Author

zkessin commented Mar 28, 2014

Ok, first of all it is tplino-core, if it is only needed for SCG then put it there.

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

No branches or pull requests

2 participants