WIP

getvariant · Jun 23, 2023 · 58730d3 · 58730d3
1 parent cd0ec4c
commit 58730d3
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 9 deletions.
diff --git a/docs/com/variant/extapi/std/demo/UserQualifyingHook.html b/docs/com/variant/extapi/std/demo/UserQualifyingHook.html
@@ -100,7 +100,7 @@ <h2 title="Class UserQualifyingHook" class="title">Class UserQualifyingHook</h2>
 <li>java.lang.Object</li>
 <li>
 <ul class="inheritance">
-<li>com.variant.extapi.std.demo.UserQualifyingHook</li>
+<li>com.variant.extapi.std.demo.UserQualificationHook</li>
 </ul>
 </li>
 </ul>

diff --git a/...t/extapi/std/demo/UserQualifyingHook.java → ...xtapi/std/demo/UserQualificationHook.java b/...t/extapi/std/demo/UserQualifyingHook.java → ...xtapi/std/demo/UserQualificationHook.java
@@ -6,7 +6,6 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-import com.typesafe.config.Config;
 import com.variant.server.api.Session;
 import com.variant.server.api.lifecycle.LifecycleHook;
 import com.variant.server.api.lifecycle.VariationQualificationLifecycleEvent;
@@ -20,15 +19,14 @@
  * </code>
  * 
  */
-public class UserQualifyingHook implements LifecycleHook<VariationQualificationLifecycleEvent> {
+public class UserQualificationHook implements LifecycleHook<VariationQualificationLifecycleEvent> {
 
-	private static final Logger LOG = LoggerFactory.getLogger(UserQualifyingHook.class);
+	private static final Logger LOG = LoggerFactory.getLogger(UserQualificationHook.class);
 
 	private final String[] blackList;
 
-	public UserQualifyingHook(String init) {
-		blackList = new String[0];
-		//blackList = config.getList("blackList").stream().map(e -> e.unwrapped()).toArray(String[]::new);
+	public UserQualificationHook(String init) {
+		blackList = init.split(",");
 	}
 
 	@Override
@@ -41,8 +39,8 @@ public Optional<VariationQualificationLifecycleEvent.PostResult> post(VariationQ
 
 		Session ssn = event.getSession();
 		String user = ssn.getAttributes().get("user");
-
-		boolean blacklisted = Arrays.stream(blackList).filter(d -> user.equals(d)).findFirst().isPresent();
+		if (user == null) throw new RuntimeException("Session attribute 'user' must be set");
+		boolean blacklisted = Arrays.asList(blackList).contains(user);
 
 		if (blacklisted)
 			LOG.info("Disqualified blacklisted user [" + user + "] from variation " + event.getVariation().getName() + "]");