5c1ad6403e
* This commit changes two things: * Pom.xml --> ES- Restclient version to 7.17.5 from 7.9.2 * ElasticSearchPlugin.java ---> changed the testDatasource method to expect a 200 instead of a 200 or 404 HTTP status code This commit fixes a high-priority bug with github issue https://github.com/appsmithorg/appsmith/issues/14909 * it can now negotiate around a 403 forbidden error as it does try to send head only to user provided URI This commit has been tested: * manually for CRUD at local with ES version 7.9.2 * manually for CR at user provide aws instance running 7.9.3 on docker * | | BugFix: | | This commit changes two files: | | * ElasticSearchPlugin.java --> added feature to distinguish between unauthorized and not found datasources. | * ElasticSearchPluginTest.java ---> added testcases to verify the unauthorized and not found test cases | | This commit adds features on top of previous commit https://github.com/appsmithorg/appsmith/issues/14909 | * this commit adds the feature of more readable error messages while testing the elasticsearch datasource. i.e wrong endpoint errors and unauthorized issues | | This commit has been tested: | * Junit | * manually * removed leftover comments from the parent commit and updated response texts for better readability * removed wildcard imports from appsmith-plugins/elasticSearchPlugin/src/test/java/com/external/plugins/ElasticSearchPluginTest.java |
||
|---|---|---|
| .. | ||
| src | ||
| plugin.properties | ||
| pom.xml | ||