Skip to content

Commit

Permalink
feat(#1192): disable ssl verify for elasticsearch http client (#1193)
Browse files Browse the repository at this point in the history
  • Loading branch information
frascuchon committed Feb 23, 2022
1 parent e6489c7 commit 007688f
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 1 deletion.
5 changes: 4 additions & 1 deletion src/rubrix/server/commons/es_wrapper.py
Expand Up @@ -64,7 +64,10 @@ def get_instance(cls) -> "ElasticsearchWrapper":
"""

if cls._INSTANCE is None:
es_client = OpenSearch(hosts=settings.elasticsearch)
es_client = OpenSearch(
hosts=settings.elasticsearch,
verify_certs=settings.elasticsearch_ssl_verify,
)
cls._INSTANCE = cls(es_client)

return cls._INSTANCE
Expand Down
4 changes: 4 additions & 0 deletions src/rubrix/server/commons/settings.py
Expand Up @@ -59,6 +59,7 @@ class ApiSettings(BaseSettings):
__DATASETS_RECORDS_INDEX_NAME__ = ".rubrix<NAMESPACE>.dataset.{}.records-v0"

elasticsearch: str = "http://localhost:9200"
elasticsearch_ssl_verify: bool = True
cors_origins: List[str] = ["*"]

docs_enabled: bool = True
Expand Down Expand Up @@ -116,6 +117,9 @@ def obfuscated_elasticsearch(self) -> str:
class Config:
# TODO: include a common prefix for all rubrix env vars.
fields = {
"elasticsearch_ssl_verify": {
"env": "RUBRIX_ELASTICSEARCH_SSL_VERIFY",
},
"metadata_fields_limit": {"env": "RUBRIX_METADATA_FIELDS_LIMIT"},
"namespace": {
"env": "RUBRIX_NAMESPACE",
Expand Down

0 comments on commit 007688f

Please sign in to comment.