Change config_set_blah() family of functions to call scitokens_config_set_blah()

To cut down on code duplication, the config_set_blah() family of APIs have been
changed to immediately call the scitokens-prefixed family of APIs under the hood.

Author: jhiemstrawisc

src/scitokens.cpp

@@ -956,42 +956,7 @@ 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) {
-    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;
-    }
+    return scitokens_config_set_int(key, value, err_msg);
 }
 
 int scitokens_config_set_int(const char *key, int value, char **err_msg) {
@@ -1034,28 +999,7 @@ int scitokens_config_set_int(const char *key, int value, char **err_msg) {
 }
 
 int 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;
-    }
+    return scitokens_config_get_int(key, err_msg);
 }
 
 int scitokens_config_get_int(const char *key, char **err_msg) {
@@ -1084,31 +1028,7 @@ int scitokens_config_get_int(const char *key, char **err_msg) {
 }
 
 int 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;
+    return scitokens_config_set_str(key, value, err_msg);
 }
 
 int scitokens_config_set_str(const char *key, const char *value,
@@ -1141,25 +1061,7 @@ int scitokens_config_set_str(const char *key, const char *value,
 }
 
 int 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;
+    return scitokens_config_get_str(key, output, err_msg);
 }
 
 int scitokens_config_get_str(const char *key, char **output, char **err_msg) {