gforeman02 commented on code in PR #8462:
URL: https://github.com/apache/nifi/pull/8462#discussion_r1571431883
##########
nifi-nar-bundles/nifi-standard-services/nifi-lookup-services-bundle/nifi-lookup-services/src/main/java/org/apache/nifi/lookup/RestLookupService.java:
##########
@@ -125,11 +127,30 @@ public class RestLookupService extends
AbstractControllerService implements Reco
.identifiesControllerService(SSLContextService.class)
.build();
+ public static final PropertyDescriptor AUTHENTICATION_STRATEGY = new
PropertyDescriptor.Builder()
+ .name("rest-lookup-authentication-strategy")
+ .displayName("Authentication Strategy")
+ .description("Authentication strategy to use with REST service.")
+ .required(true)
+ .allowableValues(AuthenticationStrategy.class)
+ .defaultValue(AuthenticationStrategy.NONE)
+ .build();
+
+ public static final PropertyDescriptor OAUTH2_ACCESS_TOKEN_PROVIDER = new
PropertyDescriptor.Builder()
+ .name("rest-lookup-oauth2-access-token-provider")
+ .displayName("OAuth2 Access Token Provider")
+ .description("Enables managed retrieval of OAuth2 Bearer Token applied
to HTTP requests using the Authorization Header.")
+ .identifiesControllerService(OAuth2AccessTokenProvider.class)
+ .required(false)
Review Comment:
@exceptionfactory I assume this will also apply to the basic auth username
and password fields as well now that they also depend on the selection.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
