achristianson commented on a change in pull request #4156: NIFI-7273: Add flow
metrics REST endpoint with for Prometheus scraping
URL: https://github.com/apache/nifi/pull/4156#discussion_r396531678
##########
File path:
nifi-nar-bundles/nifi-prometheus-bundle/nifi-prometheus-reporting-task/src/main/java/org/apache/nifi/reporting/prometheus/PrometheusRecordSink.java
##########
@@ -62,13 +63,22 @@
private volatile Map<String, Gauge> gauges;
private static final CollectorRegistry RECORD_REGISTRY = new
CollectorRegistry();
+ public static final PropertyDescriptor SSL_CONTEXT = new
PropertyDescriptor.Builder()
+ .name("prometheus-reporting-task-ssl-context")
+ .displayName("SSL Context Service")
+ .description("The SSL Context Service to use in order to secure
the server. If specified, the server will"
+ + "accept only HTTPS requests; otherwise, the server will
accept only HTTP requests")
+ .required(false)
+ .identifiesControllerService(RestrictedSSLContextService.class)
+ .build();
Review comment:
Fair enough... changing all that would be well out of scope for this PR. I
think it's something to look into in the future (changing all of NiFi to be
secure by default), but not a worry for now.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
