Add comments

Author: rmi22186

src/store.ts

@@ -765,8 +765,15 @@ export function processBaseUrls(
 function processCustomBaseUrls(config: SDKInitConfig): Dictionary<string> {
     const defaultBaseUrls: Dictionary<string> = Constants.DefaultBaseUrls;
     const CNAMEUrlPaths: Dictionary<string> = Constants.CNAMEUrlPaths;
+
+    // newBaseUrls are default if the customer is not using a CNAME
+    // If a customer passes either config.domain or config.v3SecureServiceUrl,
+    // config.identityUrl, etc, the customer is using a CNAME.
+    // config.domain will take priority if a customer passes both.
     const newBaseUrls: Dictionary<string> = {};
-    // If config.domain exists, the customer is using a CNAME, and we append the url paths to the domain
+    // If config.domain exists, the customer is using a CNAME.  We append the url paths to the provided domain.
+    // This flag is set on the Rokt/MP snippet (starting at version 2.6), meaning config.domain will alwys be empty
+    // if a customer is using a snippet prior to 2.6.
     if (!isEmpty(config.domain)) {
         for (let pathKey in CNAMEUrlPaths) {
             newBaseUrls[pathKey] = `${config.domain}${CNAMEUrlPaths[pathKey]}`;

test/src/tests-store.ts

@@ -1543,6 +1543,8 @@ describe('Store', () => {
             });
 
             it('should append url paths to domain when config.domain is set', () => {
+                // This example assumes only the domain is set, and not any of the 
+                // configurable URLs
                 const config = {
                     domain: 'custom.domain.com'
                 };
@@ -1566,6 +1568,9 @@ describe('Store', () => {
             });
 
             it('should prioritize domain over custom baseUrls when both are set', () => {
+                // If both the domain and other configurable URLs are set, then 
+                // we use the domain.  A customer should not be passing in both, as
+                // that would be an implementation error.
                 const config = {
                     domain: 'custom.domain.com',
                     v1SecureServiceUrl: 'foo.customer.mp.com/v1/JS/',