Open dblock opened 1 month ago
In https://github.com/opensearch-project/opensearch-java/pull/999 we are using reflection to call pathEncode from the right library. There be dragons.
pathEncode
Do not use reflection.
Alternatives attempted were to use a common library. A better approach is likely to change the class hierarchy so that the active transport library is doing the encoding.
What is the bug?
In https://github.com/opensearch-project/opensearch-java/pull/999 we are using reflection to call
pathEncodefrom the right library. There be dragons.What is the expected behavior?
Do not use reflection.
Alternatives attempted were to use a common library. A better approach is likely to change the class hierarchy so that the active transport library is doing the encoding.