search

chiangf / Flask-Elasticsearch

Flask extension that provides simple integration with Elasticsearch
MIT License
134 stars 21 forks source link

Elasticsearch flask extension cannot support SSL #3

Closed nelsnelson closed 9 years ago

nelsnelson commented 9 years ago

It could support SSL, but it doesn't currently, because there is no way to add the keyword arguments to the Elasticsearch(...) instantiation at https://github.com/chiangf/Flask-Elasticsearch/blob/master/flask_elasticsearch.py#L34.

In order to support this, I'll create a patch.

nelsnelson commented 9 years ago

Patch: https://github.com/chiangf/Flask-Elasticsearch/pull/4

chiangf commented 9 years ago

Merged #4

nelsnelson commented 9 years ago

Thank you, sir. I, however, did not include a version bump in setup.py. :-(

chiangf commented 9 years ago

No worries, I've bumped it, thanks for the pull request!

On Mon, Jun 22, 2015 at 7:46 PM, Nels Nelson notifications@github.com wrote:

Thank you, sir. I, however, did not include a version bump in setup.py. :-(

— Reply to this email directly or view it on GitHub https://github.com/chiangf/Flask-Elasticsearch/issues/3#issuecomment-114305774 .