Re: [PR] JWTAuthPluginIntegrationTest - dynamically generate the self-signed certificate using the localhost loopback address [solr]

2023-11-28 Thread via GitHub
jdyer1 commented on PR #2099: URL: https://github.com/apache/solr/pull/2099#issuecomment-1830832210 @janhoy On the plus side, all checks are now passing. On the minus side, I went ahead and mapped 127.0.0.1 to "localhost.localdomain" on my /etc/hosts file, and sadly the test still fails w

Re: [PR] JWTAuthPluginIntegrationTest - dynamically generate the self-signed certificate using the localhost loopback address [solr]

2023-11-28 Thread via GitHub
janhoy commented on PR #2099: URL: https://github.com/apache/solr/pull/2099#issuecomment-1830366870 The other test failure looks like some Java security manager thing; ``` 2> 3803 INFO (SUITE-JWTAuthPluginIntegrationTest-seed#[F234B11BA9F49CDF]-worker) [n: c: s: r: x: t:] o.a.s

Re: [PR] JWTAuthPluginIntegrationTest - dynamically generate the self-signed certificate using the localhost loopback address [solr]

2023-11-28 Thread via GitHub
janhoy commented on PR #2099: URL: https://github.com/apache/solr/pull/2099#issuecomment-1830360446 Regarding the `usedUndeclaredArtifacts` you can add a special entry in grade file for it, look in dev-docs/ for details. -- This is an automated message from the Apache Git Service. To resp

Re: [PR] JWTAuthPluginIntegrationTest - dynamically generate the self-signed certificate using the localhost loopback address [solr]

2023-11-28 Thread via GitHub
janhoy commented on code in PR #2099: URL: https://github.com/apache/solr/pull/2099#discussion_r1408138396 ## solr/modules/jwt-auth/src/test/org/apache/solr/security/jwt/JWTAuthPluginIntegrationTest.java: ## @@ -116,6 +122,12 @@ public static void afterClass() throws Exception {

Re: [PR] JWTAuthPluginIntegrationTest - dynamically generate the self-signed certificate using the localhost loopback address [solr]

2023-11-28 Thread via GitHub
jdyer1 commented on PR #2099: URL: https://github.com/apache/solr/pull/2099#issuecomment-1830290767 @epugh Thank you for your review. I expected IntelliJ to automatically use the project's formatting preferences but it does not seem to be the case. But `gradlew tidy` did the trick.

Re: [PR] JWTAuthPluginIntegrationTest - dynamically generate the self-signed certificate using the localhost loopback address [solr]

2023-11-28 Thread via GitHub
epugh commented on code in PR #2099: URL: https://github.com/apache/solr/pull/2099#discussion_r1407862073 ## solr/modules/jwt-auth/src/test/org/apache/solr/security/jwt/JWTAuthPluginIntegrationTest.java: ## @@ -25,16 +25,14 @@ import java.io.InputStreamReader; import java.io.O

Re: [PR] JWTAuthPluginIntegrationTest - dynamically generate the self-signed certificate using the localhost loopback address [solr]

2023-11-28 Thread via GitHub
janhoy commented on code in PR #2099: URL: https://github.com/apache/solr/pull/2099#discussion_r1407459703 ## solr/modules/jwt-auth/src/test/org/apache/solr/security/jwt/JWTAuthPluginIntegrationTest.java: ## @@ -116,6 +122,12 @@ public static void afterClass() throws Exception {

Re: [PR] JWTAuthPluginIntegrationTest - dynamically generate the self-signed certificate using the localhost loopback address [solr]

2023-11-27 Thread via GitHub
jdyer1 commented on PR #2099: URL: https://github.com/apache/solr/pull/2099#issuecomment-1828823519 This is an attempt to help the frequent failures with JWTAuthPluginIntegrationTest. I see a past comment on SOLR-15484 suggested generating the certificate based on the loopback address rath

[PR] JWTAuthPluginIntegrationTest - dynamically generate the self-signed certificate using the localhost loopback address [solr]

2023-11-27 Thread via GitHub
jdyer1 opened a new pull request, #2099: URL: https://github.com/apache/solr/pull/2099 https://issues.apache.org/jira/browse/SOLR-15484 # Description Please provide a short description of the changes you're making with this pull request. # Solution Ple