Correctly allow base64 prefixes for "s/" (#4958)

Fixes #4957
deephaven · Dec 18, 2023 · 8930861 · 8930861
1 parent 48a56d9
commit 8930861
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/web/client-api/src/main/java/io/deephaven/web/client/api/console/JsVariableDefinition.java b/web/client-api/src/main/java/io/deephaven/web/client/api/console/JsVariableDefinition.java
@@ -28,8 +28,8 @@ public class JsVariableDefinition {
  private final String applicationName;
 
  public JsVariableDefinition(String type, String title, String id, String description) {
- // base64('s/') ==> 'cy8'
- if (!id.startsWith("cy8")) {
+ // base64('s/' + str) starts with 'cy8' or 'cy9'
+ if (!id.startsWith("cy")) {
  throw new IllegalArgumentException("Cannot create a VariableDefinition from a non-scope ticket");
  }
  this.type = type;