Add duplicate config APIs with prefixed names

Unfortunately, the choice of names for the config_set_blah() family of APIs is
quite generic and poses the potential to cause C symbol collisions when SciTokens
is used by large projects that dynamically load lots of libraries (a la HTCondor).
A reasonable plan to address this, since those APIs are in use and changing their
names would be API-breaking, seems to be adding a second set of prefixed APIs, with
the intention of deprecating the old functions at the next major release.

Author: jhiemstrawisc

src/scitokens.cpp

@@ -994,6 +994,45 @@ int config_set_int(const char *key, int value, char **err_msg) {
     }
 }
 
+int scitokens_config_set_int(const char *key, int value, char **err_msg) {
+    if (!key) {
+        if (err_msg) {
+            *err_msg = strdup("A key must be provided.");
+        }
+        return -1;
+    }
+
+    std::string _key = key;
+    if (_key == "keycache.update_interval_s") {
+        if (value < 0) {
+            if (err_msg) {
+                *err_msg = strdup("Update interval must be positive.");
+            }
+            return -1;
+        }
+        configurer::Configuration::set_next_update_delta(value);
+        return 0;
+    }
+
+    else if (_key == "keycache.expiration_interval_s") {
+        if (value < 0) {
+            if (err_msg) {
+                *err_msg = strdup("Expiry interval must be positive.");
+            }
+            return -1;
+        }
+        configurer::Configuration::set_expiry_delta(value);
+        return 0;
+    }
+
+    else {
+        if (err_msg) {
+            *err_msg = strdup("Key not recognized.");
+        }
+        return -1;
+    }
+}
+
 int config_get_int(const char *key, char **err_msg) {
     if (!key) {
         if (err_msg) {
@@ -1019,6 +1058,31 @@ int config_get_int(const char *key, char **err_msg) {
     }
 }
 
+int scitokens_config_get_int(const char *key, char **err_msg) {
+    if (!key) {
+        if (err_msg) {
+            *err_msg = strdup("A key must be provided.");
+        }
+        return -1;
+    }
+
+    std::string _key = key;
+    if (_key == "keycache.update_interval_s") {
+        return configurer::Configuration::get_next_update_delta();
+    }
+
+    else if (_key == "keycache.expiration_interval_s") {
+        return configurer::Configuration::get_expiry_delta();
+    }
+
+    else {
+        if (err_msg) {
+            *err_msg = strdup("Key not recognized.");
+        }
+        return -1;
+    }
+}
+
 int config_set_str(const char *key, const char *value, char **err_msg) {
     if (!key) {
         if (err_msg) {
@@ -1047,6 +1111,34 @@ int config_set_str(const char *key, const char *value, char **err_msg) {
     return 0;
 }
 
+int scitokens_config_set_str(const char *key, const char *value, char **err_msg) {
+    if (!key) {
+        if (err_msg) {
+            *err_msg = strdup("A key must be provided.");
+        }
+        return -1;
+    }
+
+    std::string _key = key;
+    if (_key == "keycache.cache_home") {
+        auto rp = configurer::Configuration::set_cache_home(value);
+        if (!rp.first) { // There was an error, pass rp.second to err_msg
+            if (err_msg) {
+                *err_msg = strdup(rp.second.c_str());
+            }
+            return -1;
+        }
+    }
+
+    else {
+        if (err_msg) {
+            *err_msg = strdup("Key not recognized.");
+        }
+        return -1;
+    }
+    return 0;
+}
+
 int config_get_str(const char *key, char **output, char **err_msg) {
     if (!key) {
         if (err_msg) {
@@ -1068,3 +1160,25 @@ int config_get_str(const char *key, char **output, char **err_msg) {
     }
     return 0;
 }
+
+int scitokens_config_get_str(const char *key, char **output, char **err_msg) {
+    if (!key) {
+        if (err_msg) {
+            *err_msg = strdup("A key must be provided.");
+        }
+        return -1;
+    }
+
+    std::string _key = key;
+    if (_key == "keycache.cache_home") {
+        *output = strdup(configurer::Configuration::get_cache_home().c_str());
+    }
+
+    else {
+        if (err_msg) {
+            *err_msg = strdup("Key not recognized.");
+        }
+        return -1;
+    }
+    return 0;
+}

src/scitokens.h

@@ -302,6 +302,9 @@ int keycache_set_jwks(const char *issuer, const char *jwks, char **err_msg);
  */
 int config_set_int(const char *key, int value, char **err_msg);
 
+// Prefixed version of the same API to avoid potential symbol collisions
+int scitokens_config_set_int(const char *key, int value, char **err_msg);
+
 /**
  * Get current scitokens int parameters.
  * Returns the value associated with the supplied input key on success, and -1
@@ -310,19 +313,28 @@ int config_set_int(const char *key, int value, char **err_msg);
  */
 int config_get_int(const char *key, char **err_msg);
 
+// Prefixed version of the same API to avoid potential symbol collisions
+int scitokens_config_get_int(const char *key, char **err_msg);
+
 /**
  * Set current scitokens str parameters.
  * Returns 0 on success, nonzero on failure
  */
 int config_set_str(const char *key, const char *value, char **err_msg);
 
+// Prefixed version of the same API to avoid potential symbol collisions
+int scitokens_config_set_str(const char *key, const char *value, char **err_msg);
+
 /**
  * Get current scitokens str parameters.
  * Returns 0 on success, nonzero on failure, and populates the value associated
  * with the input key to output.
  */
 int config_get_str(const char *key, char **output, char **err_msg);
 
+// Prefixed version of the same API to avoid potential symbol collisions
+int scitokens_config_get_str(const char *key, char **output, char **err_msg);
+
 #ifdef __cplusplus
 }
 #endif