search

knutin / elli

Simple, robust and performant Erlang web server
MIT License
663 stars 79 forks source link

Replace uri decoder from inets #110

Closed cvik closed 6 years ago

cvik commented 6 years ago

Replaces all calls to 'http_uri:decode/1' from the inets application with a custom function local to elli. This allows the removal of inets as a dependency.

cvik commented 6 years ago

Crap. Didn't think to support older versions.

andreashasse commented 6 years ago

Thanks for the pull request! @knutin: It might be time to drop support for erlang versions older than R16. I propose that we drop that later and accept this PR as is.

tsloughter commented 6 years ago

please open this on https://github.com/elli-lib/elli

Also note that this has been recognized in OTP and a uri module in the stdlib is coming: https://github.com/erlang/otp/pull/1551

I was just going to wait for that.

cvik commented 6 years ago

Ah, I missed that this had been forked. I have opened the PR elli-lib/elli#36.

cvik commented 6 years ago

I'm closing this one since it has been moved to elli-lib/elli#36.