Github user rvesse commented on a diff in the pull request:
https://github.com/apache/jena/pull/492#discussion_r235395467
--- Diff:
jena-fuseki2/jena-fuseki-core/src/main/java/org/apache/jena/fuseki/build/FusekiBuilder.java
---
@@ -70,16 +69,42 @@ public static void populateStdServices(DataService
dataService, boolean allowUpd
/** Add an operation to a {@link DataService} with a given endpoint
name */
public static void addServiceEP(DataService dataService, Operation
operation, String endpointName) {
- dataService.addEndpoint(operation, endpointName) ;
+ dataService.addEndpoint(operation, endpointName) ;
+ }
+
+ /** Add an operation to a {@link DataService} with a given endpoint
name */
+ public static void addServiceEP(DataService dataService, Operation
operation, String endpointName, AuthPolicy requestAuth) {
+ dataService.addEndpoint(operation, endpointName, requestAuth) ;
}
public static void addServiceEP(DataService dataService, Operation
operation, Resource svc, Property property) {
String p = "<"+property.getURI()+">" ;
ResultSet rs = FusekiBuildLib.query("SELECT * { ?svc " + p + "
?ep}", svc.getModel(), "svc", svc) ;
for ( ; rs.hasNext() ; ) {
QuerySolution soln = rs.next() ;
- String epName = soln.getLiteral("ep").getLexicalForm() ;
- addServiceEP(dataService, operation, epName);
+ AuthPolicy requestAuth = null;
--- End diff --
Should this be getting populated with a non-null value here? Or is the
intention to set to `null` by default and then potentially load in the auth
policy later elsewhere?
---
