Skip to content

Commit 8889ba5

Browse files
committed
Remove animal sniffer from REST client sniffer
Animal sniffer is no longer needed, we can remove it for sniffer like we did for the low-level REST client with elastic#29646
1 parent 5e4d0b4 commit 8889ba5

File tree

2 files changed

+0
-5
lines changed

2 files changed

+0
-5
lines changed

client/sniffer/build.gradle

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@
2020
import org.elasticsearch.gradle.precommit.PrecommitTasks
2121

2222
apply plugin: 'elasticsearch.build'
23-
apply plugin: 'ru.vyarus.animalsniffer'
2423
apply plugin: 'nebula.maven-base-publish'
2524
apply plugin: 'nebula.maven-scm'
2625

@@ -52,8 +51,6 @@ dependencies {
5251
testCompile "org.hamcrest:hamcrest-all:${versions.hamcrest}"
5352
testCompile "org.elasticsearch:securemock:${versions.securemock}"
5453
testCompile "org.elasticsearch:mocksocket:${versions.mocksocket}"
55-
testCompile "org.codehaus.mojo:animal-sniffer-annotations:1.15"
56-
signature "org.codehaus.mojo.signature:java17:1.0@signature"
5754
}
5855

5956
forbiddenApisMain {

client/sniffer/src/test/java/org/elasticsearch/client/sniff/ElasticsearchHostsSnifferTests.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,8 +60,6 @@
6060
import static org.junit.Assert.assertThat;
6161
import static org.junit.Assert.fail;
6262

63-
//animal-sniffer doesn't like our usage of com.sun.net.httpserver.* classes
64-
@IgnoreJRERequirement
6563
public class ElasticsearchHostsSnifferTests extends RestClientTestCase {
6664

6765
private int sniffRequestTimeout;

0 commit comments

Comments
 (0)