Skip to content

Commit b633139

Browse files
authored
Apply upstream ES GeoIP factory interface changes. (#273)
* Apply upstream ES GeoIP factory interface changes. * Keep setting ES treeish main in main branch.
1 parent 2773101 commit b633139

File tree

5 files changed

+13
-6
lines changed

5 files changed

+13
-6
lines changed

gradle.properties

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
11
LOGSTASH_PATH=../../logstash
2-
ELASTICSEARCH_TREEISH=9.0
2+
ELASTICSEARCH_TREEISH=main

src/main/java/co/elastic/logstash/filters/elasticintegration/IngestPipelineFactory.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
import co.elastic.logstash.filters.elasticintegration.ingest.PipelineProcessor;
1010
import org.apache.logging.log4j.LogManager;
1111
import org.apache.logging.log4j.Logger;
12+
import org.elasticsearch.cluster.metadata.ProjectId;
1213
import org.elasticsearch.ingest.Pipeline;
1314
import org.elasticsearch.ingest.PipelineConfiguration;
1415
import org.elasticsearch.ingest.Processor;
@@ -46,7 +47,7 @@ public IngestPipelineFactory withProcessors(final Map<String, Processor.Factory>
4647

4748
public Optional<IngestPipeline> create(final PipelineConfiguration pipelineConfiguration) {
4849
try {
49-
final Pipeline pipeline = Pipeline.create(pipelineConfiguration.getId(), pipelineConfiguration.getConfig(false), processorFactories, scriptService);
50+
final Pipeline pipeline = Pipeline.create(pipelineConfiguration.getId(), pipelineConfiguration.getConfig(false), processorFactories, scriptService, ProjectId.DEFAULT);
5051
final IngestPipeline ingestPipeline = new IngestPipeline(pipelineConfiguration, pipeline);
5152
LOGGER.debug(() -> String.format("successfully created ingest pipeline `%s` from pipeline configuration", pipelineConfiguration.getId()));
5253
return Optional.of(ingestPipeline);

src/main/java/co/elastic/logstash/filters/elasticintegration/geoip/GeoIpProcessorFactory.java

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
*/
77
package co.elastic.logstash.filters.elasticintegration.geoip;
88

9+
import org.elasticsearch.cluster.metadata.ProjectId;
910
import org.elasticsearch.ingest.Processor;
1011
import org.elasticsearch.ingest.geoip.GeoIpProcessor;
1112

@@ -22,7 +23,9 @@ public GeoIpProcessorFactory(final IpDatabaseProvider ipDatabaseProvider) {
2223
public Processor create(Map<String, Processor.Factory> processorFactories,
2324
String tag,
2425
String description,
25-
Map<String, Object> config) throws Exception {
26-
return new GeoIpProcessor.Factory("geoip", this.ipDatabaseProvider).create(processorFactories, tag, description, config);
26+
Map<String, Object> config,
27+
ProjectId projectId) throws Exception {
28+
return new GeoIpProcessor.Factory("geoip", this.ipDatabaseProvider)
29+
.create(processorFactories, tag, description, config, projectId);
2730
}
2831
}

src/main/java/co/elastic/logstash/filters/elasticintegration/ingest/PipelineProcessor.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
import co.elastic.logstash.filters.elasticintegration.IngestPipelineResolver;
1111
import org.apache.logging.log4j.LogManager;
1212
import org.apache.logging.log4j.Logger;
13+
import org.elasticsearch.cluster.metadata.ProjectId;
1314
import org.elasticsearch.ingest.AbstractProcessor;
1415
import org.elasticsearch.ingest.ConfigurationUtils;
1516
import org.elasticsearch.ingest.IngestDocument;
@@ -91,7 +92,8 @@ public Factory(IngestPipelineResolver pipelineProvider, ScriptService scriptServ
9192
public Processor create(Map<String, Processor.Factory> registry,
9293
String processorTag,
9394
String description,
94-
Map<String, Object> config) throws Exception {
95+
Map<String, Object> config,
96+
ProjectId projectId) throws Exception {
9597
String pipeline = ConfigurationUtils.readStringProperty(TYPE, processorTag, config, "name");
9698
TemplateScript.Factory pipelineTemplate = ConfigurationUtils.compileTemplate(TYPE, processorTag, "name", pipeline, scriptService);
9799
boolean ignoreMissingPipeline = ConfigurationUtils.readBooleanProperty(TYPE, processorTag, config, "ignore_missing_pipeline", false);

src/main/java/co/elastic/logstash/filters/elasticintegration/ingest/SetSecurityUserProcessor.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
*/
77
package co.elastic.logstash.filters.elasticintegration.ingest;
88

9+
import org.elasticsearch.cluster.metadata.ProjectId;
910
import org.elasticsearch.ingest.AbstractProcessor;
1011
import org.elasticsearch.ingest.IngestDocument;
1112
import org.elasticsearch.ingest.Processor;
@@ -36,7 +37,7 @@ public static final class Factory implements Processor.Factory {
3637

3738
@Override
3839
public SetSecurityUserProcessor create(Map<String, Processor.Factory> registry, String processorTag,
39-
String description, Map<String, Object> config) {
40+
String description, Map<String, Object> config, ProjectId projectId) {
4041
String[] supportedConfigs = {"field", "properties"};
4142
for (String cfg : supportedConfigs) {
4243
config.remove(cfg);

0 commit comments

Comments
 (0)