chore: use established constants for URL and endpoints (#67)

* Use a base URL consistent with other SDKs
* don't version for release - major bugfix following

Author: typotter

build.gradle

@@ -6,7 +6,7 @@ plugins {
 }
 
 group = 'cloud.eppo'
-version = '3.5.1-SNAPSHOT'
+version = '3.5.2-SNAPSHOT'
 ext.isReleaseVersion = !version.endsWith("SNAPSHOT")
 
 java {

src/main/java/cloud/eppo/BaseEppoClient.java

@@ -27,7 +27,6 @@ public class BaseEppoClient {
       new ObjectMapper()
           .registerModule(EppoModule.eppoModule()); // TODO: is this the best place for this?
 
-  protected static final String DEFAULT_HOST = "https://fscdn.eppo.cloud";
   protected final ConfigurationRequestor requestor;
 
   private final IConfigurationStore configurationStore;
@@ -76,7 +75,7 @@ protected BaseEppoClient(
           "Unable to initialize Eppo SDK due to missing SDK name or version");
     }
     if (host == null) {
-      host = DEFAULT_HOST;
+      host = Constants.DEFAULT_BASE_URL;
     }
 
     this.assignmentCache = assignmentCache;

src/main/java/cloud/eppo/ConfigurationRequestor.java

@@ -9,8 +9,6 @@
 
 public class ConfigurationRequestor {
   private static final Logger log = LoggerFactory.getLogger(ConfigurationRequestor.class);
-  private static final String FLAG_CONFIG_PATH = "/api/flag-config/v1/config";
-  private static final String BANDIT_PARAMETER_PATH = "/api/flag-config/v1/bandits";
 
   private final EppoHttpClient client;
   private final IConfigurationStore configurationStore;
@@ -90,13 +88,13 @@ void fetchAndSaveFromRemote() {
     // Reuse the `lastConfig` as its bandits may be useful
     Configuration lastConfig = configurationStore.getConfiguration();
 
-    byte[] flagConfigurationJsonBytes = client.get(FLAG_CONFIG_PATH);
+    byte[] flagConfigurationJsonBytes = client.get(Constants.FLAG_CONFIG_ENDPOINT);
     Configuration.Builder configBuilder =
         Configuration.builder(flagConfigurationJsonBytes, expectObfuscatedConfig)
             .banditParametersFromConfig(lastConfig);
 
     if (supportBandits && configBuilder.requiresUpdatedBanditModels()) {
-      byte[] banditParametersJsonBytes = client.get(BANDIT_PARAMETER_PATH);
+      byte[] banditParametersJsonBytes = client.get(Constants.BANDIT_ENDPOINT);
       configBuilder.banditParameters(banditParametersJsonBytes);
     }
 
@@ -116,7 +114,7 @@ CompletableFuture<Void> fetchAndSaveFromRemoteAsync() {
 
     remoteFetchFuture =
         client
-            .getAsync(FLAG_CONFIG_PATH)
+            .getAsync(Constants.FLAG_CONFIG_ENDPOINT)
             .thenApply(
                 flagConfigJsonBytes -> {
                   synchronized (this) {
@@ -128,7 +126,8 @@ CompletableFuture<Void> fetchAndSaveFromRemoteAsync() {
                     if (supportBandits && configBuilder.requiresUpdatedBanditModels()) {
                       byte[] banditParametersJsonBytes;
                       try {
-                        banditParametersJsonBytes = client.getAsync(BANDIT_PARAMETER_PATH).get();
+                        banditParametersJsonBytes =
+                            client.getAsync(Constants.BANDIT_ENDPOINT).get();
                       } catch (InterruptedException | ExecutionException e) {
                         log.error("Error fetching from remote: " + e.getMessage());
                         throw new RuntimeException(e);

src/main/java/cloud/eppo/Constants.java

@@ -21,6 +21,7 @@ public class Constants {
   public static final String RAC_ENDPOINT = "/randomized_assignment/v3/config";
 
   public static final String BANDIT_ENDPOINT = "/flag-config/v1/bandits";
+  public static final String FLAG_CONFIG_ENDPOINT = "/flag-config/v1/config";
 
   /** Caching Settings */
   public static final String EXPERIMENT_CONFIGURATION_CACHE_KEY = "experiment-configuration";

src/test/java/cloud/eppo/ConfigurationRequestorTest.java

@@ -58,7 +58,7 @@ public void testInitialConfigurationDoesntClobberFetch() throws IOException {
     String fetchedFlagConfig =
         FileUtils.readFileToString(differentFlagConfigFile, StandardCharsets.UTF_8);
 
-    when(mockHttpClient.getAsync("/api/flag-config/v1/config")).thenReturn(configFetchFuture);
+    when(mockHttpClient.getAsync("/flag-config/v1/config")).thenReturn(configFetchFuture);
 
     // Set initial config and verify that no config has been set yet.
     requestor.setInitialConfiguration(initialConfigFuture);
@@ -98,7 +98,7 @@ public void testBrokenFetchDoesntClobberCache() throws IOException {
     String flagConfig = FileUtils.readFileToString(initialFlagConfigFile, StandardCharsets.UTF_8);
     CompletableFuture<byte[]> configFetchFuture = new CompletableFuture<>();
 
-    when(mockHttpClient.getAsync("/api/flag-config/v1/config")).thenReturn(configFetchFuture);
+    when(mockHttpClient.getAsync("/flag-config/v1/config")).thenReturn(configFetchFuture);
 
     // Set initial config and verify that no config has been set yet.
     requestor.setInitialConfiguration(initialConfigFuture);
@@ -135,7 +135,7 @@ public void testCacheWritesAfterBrokenFetch() throws IOException {
     String flagConfig = FileUtils.readFileToString(initialFlagConfigFile, StandardCharsets.UTF_8);
     CompletableFuture<byte[]> configFetchFuture = new CompletableFuture<>();
 
-    when(mockHttpClient.getAsync("/api/flag-config/v1/config")).thenReturn(configFetchFuture);
+    when(mockHttpClient.getAsync("/flag-config/v1/config")).thenReturn(configFetchFuture);
 
     // Set initial config and verify that no config has been set yet.
     requestor.setInitialConfiguration(initialConfigFuture);