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

Suggestion: upgrade to https://github.com/elli-lib/elli #33

Closed
CharlesOkwuagwu opened this issue Feb 24, 2017 · 4 comments
Closed

Suggestion: upgrade to https://github.com/elli-lib/elli #33

CharlesOkwuagwu opened this issue Feb 24, 2017 · 4 comments

Comments

@CharlesOkwuagwu
Copy link
Contributor

https://github.com/elli-lib/elli seems to be in active development.

Should we upgrade?

@CharlesOkwuagwu CharlesOkwuagwu changed the title Suggestion: upgrade https://github.com/elli-lib/elli Suggestion: upgrade to https://github.com/elli-lib/elli Feb 24, 2017
@CrowdHailer
Copy link
Owner

Sounds good. I just asked them about what changes might be forthcoming

@CharlesOkwuagwu
Copy link
Contributor Author

CharlesOkwuagwu commented Feb 25, 2017

We can aspire to this: http://ezgr.net/increasing-security-erlang-ssl-cowboy/

All of the above options result in getting an A rating from SSLLabs, which is a good compromise between security and compatibility.

I'm still not sure how to make the suggested security updates to elli's ssl setup/config.

These are the kind of things we can tackle together with the guys who actively maintain that elli fork I mentioned

@CrowdHailer
Copy link
Owner

I think I am entirely on board with making this switch. Sounds like it is active. Again I am currently focusing on some different areas but would accept a PR on this.

Also of note the the elli guys. I was not able to add all the Raxx features to the elli adapter. If you check the test suite in raxx_elli there are only two files(not three as in raxx_cowboy and ace_http) There were some issues with how elli worked when it came to implementing chunked responses

@CrowdHailer
Copy link
Owner

I am not currently using elli so will not be making this change myself. Anyone wishing to do this is welcome to open a PR

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