JanusGraph / JanusGraph/janusgraph

Fortify: insecure transport risks in ES tests?

Open
#798 0 comments 0 reactions 0 assignees View on GitHub
area/testing index/elasticsearch static-analysis
Dominant language
Java
Stars
5.8k
Forks
1.2k
Avg merge
13h 53m
Merged PRs (30d)
6

Description

Fortify found one instance on ElasticSearchConfigTest:93, one at ElasticSearchIndexTests:78, and one instance at ElasticSearchMultiTypeIndexTest:54. These are tests so I guess we can ignore them, but it raises a different question. Can we add a test using self signed certificates to make sure HTTPS works? if it is possible, could we replace HttpClient with one that uses HTTPS?

Contributor guide

Open the contributing guide

Research direction

Start with ElasticSearchConfigTest:93, ElasticSearchIndexTests:78, and ElasticSearchMultiTypeIndexTest:54, then trace how HttpClient is configured in those tests. Determine whether self-signed HTTPS certificates can be exercised and whether the client can use HTTPS. Done means the relevant tests cover HTTPS successfully without insecure transport warnings.

Written by the indexing model from the issue text.

Assessment

Tech stack
elasticsearch, java
Domain
databases, security, testing-qa
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.