-
Notifications
You must be signed in to change notification settings - Fork 297
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use https protocol in local nodes per default #1734
Conversation
The build issue is something I should fix. I'll try to work on this later today. ;) |
Great, thanks! |
@HansBambel could you rebase on master please? I'd like to see if the recent changes I did will work... |
Sorry, I actually did a rebase, but when pulling again a merge was done by default :-/ If you want me to fix it, I can try to do that.
|
Weird. Apparently the new push did not trigger the CI. I'm going to check. |
I think I fixed the CI problem... Let see... Could you rebase or merge the changes from master branch? |
It was not running anymore
And log in the user before starting the job. Remove non needed caching docker images
…added to the install phase (default plugin behavior)
Bumps [stefanzweifel/git-auto-commit-action](https://github.com/stefanzweifel/git-auto-commit-action) from 4 to 5. - [Release notes](https://github.com/stefanzweifel/git-auto-commit-action/releases) - [Changelog](https://github.com/stefanzweifel/git-auto-commit-action/blob/master/CHANGELOG.md) - [Commits](stefanzweifel/git-auto-commit-action@v4...v5) --- updated-dependencies: - dependency-name: stefanzweifel/git-auto-commit-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [org.jsoup:jsoup](https://github.com/jhy/jsoup) from 1.16.1 to 1.16.2. - [Release notes](https://github.com/jhy/jsoup/releases) - [Changelog](https://github.com/jhy/jsoup/blob/master/CHANGES) - [Commits](jhy/jsoup@jsoup-1.16.1...jsoup-1.16.2) --- updated-dependencies: - dependency-name: org.jsoup:jsoup dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
@dadoonet Done! |
Could you rebase again? I'm trying to fix something../ If it fails again, I'll do the github actions changes in another repo... :) |
* Refactor the CLI We can add more tests and make it a bit more flexible... * Add a basic docker compose example without Elastic It helps to test the behavior of the container when we don't have any input. Related to dadoonet#1727 * Don't hide console logs when running with Docker I believe it was a mistake doing this. It's better to let the user control the level of logs they want when running the container. In case of error, it now displays correctly the logs in the docker compose output. Closes dadoonet#1727.
Bumps [org.xerial:sqlite-jdbc](https://github.com/xerial/sqlite-jdbc) from 3.43.2.0 to 3.43.2.1. - [Release notes](https://github.com/xerial/sqlite-jdbc/releases) - [Changelog](https://github.com/xerial/sqlite-jdbc/blob/master/CHANGELOG) - [Commits](xerial/sqlite-jdbc@3.43.2.0...3.43.2.1) --- updated-dependencies: - dependency-name: org.xerial:sqlite-jdbc dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [co.elastic.clients:elasticsearch-java](https://github.com/elastic/elasticsearch-java) from 8.10.3 to 8.10.4. - [Release notes](https://github.com/elastic/elasticsearch-java/releases) - [Changelog](https://github.com/elastic/elasticsearch-java/blob/main/CHANGELOG.md) - [Commits](elastic/elasticsearch-java@v8.10.3...v8.10.4) --- updated-dependencies: - dependency-name: co.elastic.clients:elasticsearch-java dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
Let see if this strategy works better...
Done :) |
@HansBambel I think I fixed the problem in another repo and applied the changes here. Could you rebase again please? I'm sorry for this :) |
This reverts commit 4a5bd57.
Bumps `tika.version` from 2.9.0 to 2.9.1. Updates `org.apache.tika:tika-parent` from 2.9.0 to 2.9.1 - [Changelog](https://github.com/apache/tika/blob/main/CHANGES.txt) - [Commits](apache/tika@2.9.0...2.9.1) Updates `org.apache.tika:tika-core` from 2.9.0 to 2.9.1 - [Changelog](https://github.com/apache/tika/blob/main/CHANGES.txt) - [Commits](apache/tika@2.9.0...2.9.1) Updates `org.apache.tika:tika-parsers-standard-package` from 2.9.0 to 2.9.1 Updates `org.apache.tika:tika-parser-scientific-module` from 2.9.0 to 2.9.1 Updates `org.apache.tika:tika-parser-sqlite3-module` from 2.9.0 to 2.9.1 Updates `org.apache.tika:tika-langdetect-optimaize` from 2.9.0 to 2.9.1 --- updated-dependencies: - dependency-name: org.apache.tika:tika-parent dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: org.apache.tika:tika-core dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: org.apache.tika:tika-parsers-standard-package dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: org.apache.tika:tika-parser-scientific-module dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: org.apache.tika:tika-parser-sqlite3-module dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: org.apache.tika:tika-langdetect-optimaize dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [org.fusesource.jansi:jansi](https://github.com/fusesource/jansi) from 2.4.0 to 2.4.1. - [Release notes](https://github.com/fusesource/jansi/releases) - [Changelog](https://github.com/fusesource/jansi/blob/master/changelog.md) - [Commits](fusesource/jansi@jansi-2.4.0...jansi-2.4.1) --- updated-dependencies: - dependency-name: org.fusesource.jansi:jansi dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [org.apache.maven.plugins:maven-javadoc-plugin](https://github.com/apache/maven-javadoc-plugin) from 3.5.0 to 3.6.0. - [Release notes](https://github.com/apache/maven-javadoc-plugin/releases) - [Commits](apache/maven-javadoc-plugin@maven-javadoc-plugin-3.5.0...maven-javadoc-plugin-3.6.0) --- updated-dependencies: - dependency-name: org.apache.maven.plugins:maven-javadoc-plugin dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [com.google.guava:guava](https://github.com/google/guava) from 32.1.2-jre to 32.1.3-jre. - [Release notes](https://github.com/google/guava/releases) - [Commits](https://github.com/google/guava/commits) --- updated-dependencies: - dependency-name: com.google.guava:guava dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps `jackson.version` from 2.15.2 to 2.15.3. Updates `com.fasterxml.jackson.core:jackson-core` from 2.15.2 to 2.15.3 - [Release notes](https://github.com/FasterXML/jackson-core/releases) - [Commits](FasterXML/jackson-core@jackson-core-2.15.2...jackson-core-2.15.3) Updates `com.fasterxml.jackson.core:jackson-databind` from 2.15.2 to 2.15.3 - [Commits](https://github.com/FasterXML/jackson/commits) Updates `com.fasterxml.jackson.core:jackson-annotations` from 2.15.2 to 2.15.3 - [Commits](https://github.com/FasterXML/jackson/commits) Updates `com.fasterxml.jackson.datatype:jackson-datatype-jsr310` from 2.15.2 to 2.15.3 Updates `com.fasterxml.jackson.dataformat:jackson-dataformat-xml` from 2.15.2 to 2.15.3 - [Commits](FasterXML/jackson-dataformat-xml@jackson-dataformat-xml-2.15.2...jackson-dataformat-xml-2.15.3) Updates `com.fasterxml.jackson.dataformat:jackson-dataformat-yaml` from 2.15.2 to 2.15.3 - [Commits](FasterXML/jackson-dataformats-text@jackson-dataformats-text-2.15.2...jackson-dataformats-text-2.15.3) Updates `com.fasterxml.jackson.dataformat:jackson-module-jaxb-annotations` from 2.15.2 to 2.15.3 Updates `com.fasterxml.jackson.module:jackson-module-jaxb-annotations` from 2.15.2 to 2.15.3 - [Commits](FasterXML/jackson-modules-base@jackson-modules-base-2.15.2...jackson-modules-base-2.15.3) --- updated-dependencies: - dependency-name: com.fasterxml.jackson.core:jackson-core dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: com.fasterxml.jackson.core:jackson-databind dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: com.fasterxml.jackson.core:jackson-annotations dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: com.fasterxml.jackson.datatype:jackson-datatype-jsr310 dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: com.fasterxml.jackson.dataformat:jackson-dataformat-xml dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: com.fasterxml.jackson.dataformat:jackson-dataformat-yaml dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: com.fasterxml.jackson.dataformat:jackson-module-jaxb-annotations dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: com.fasterxml.jackson.module:jackson-module-jaxb-annotations dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [org.apache.maven.plugins:maven-surefire-plugin](https://github.com/apache/maven-surefire) from 3.1.2 to 3.2.1. - [Release notes](https://github.com/apache/maven-surefire/releases) - [Commits](apache/maven-surefire@surefire-3.1.2...surefire-3.2.1) --- updated-dependencies: - dependency-name: org.apache.maven.plugins:maven-surefire-plugin dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [org.apache.maven.plugins:maven-dependency-plugin](https://github.com/apache/maven-dependency-plugin) from 3.6.0 to 3.6.1. - [Commits](apache/maven-dependency-plugin@maven-dependency-plugin-3.6.0...maven-dependency-plugin-3.6.1) --- updated-dependencies: - dependency-name: org.apache.maven.plugins:maven-dependency-plugin dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
No problem :D Done 👍 |
Apparently some of the unit tests need to be fixed. Could you work on that @HansBambel? |
@dadoonet Sorry, but it's been too long that dealt with Java (back in University). I don't even know where to start here. If you help me to compile/run the code/tests I can have a look. |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Thanks! |
In #1274 (comment) it was noted that the default http protocol resulted in an error
Unexpected end of file from server
.When switching to
https
the error does not occur.With this PR the default protocol of a local node's url will be
https
.The documentation was also updated to reflect that. I was not sure how to handle the text in the Note as it proposes to use HTTPS instead of HTTP, but now this is the default. Should it be omitted or kept to show the change from version 2.2?
Important
I don't know how to run the tests locally as I am not a Java developer and I also do not use elasticsearch, but opensearch (and even this not yet successfully).