Deprecate usage of bare env vars and config sources

Deprecate usage of bare environment variables and config sources in configuration files.
  - Use `${env:VAR}` or `${VAR}` instead of `$VAR`.
  - Use `${uri:selector}` instead of `${uri:selector}`, e.g. `${file:/path/to/file}` instead of `$file:/path/to/file`.

Detect such usages and print a deprecation warning. It's ok to not use logger because it's not initiated at this point yet. We use `log.Print` at the stage in other places as well.

CHANGELOG.md

@@ -8,6 +8,12 @@
   in v0.42.0 to support a bug causing double expansion. $$ is treated as an escape sequence representing a literal
   $ character ([#5134](https://github.com/signalfx/splunk-otel-collector/pull/5134))
 
+### 🚩Deprecations 🚩
+
+- (Splunk) Deprecate usage of bare environment variables and config sources in configuration files. 
+  - Use `${env:VAR}` or `${VAR}` instead of `$VAR`.
+  - Use `${uri:selector}` instead of `$uri:selector`, e.g. `${file:/path/to/file}` instead of `$file:/path/to/file`.
+
 ### 💡 Enhancements 💡
 
 - (Splunk) `discovery`: Enable discovery mode for SQL Server receiver ([#5109](https://github.com/signalfx/splunk-otel-collector/pull/5109))

internal/configsource/source.go

@@ -17,9 +17,11 @@ package configsource
 import (
 	"context"
 	"fmt"
+	"log"
 	"net/url"
 	"os"
 	"strings"
+	"sync"
 
 	"github.com/knadh/koanf/maps"
 	"github.com/spf13/cast"
@@ -334,22 +336,33 @@ func resolveStringValue(ctx context.Context, configSources map[string]ConfigSour
 			var expandableContent, cfgSrcName string
 			w := 0 // number of bytes consumed on this pass
 
+			var deprecatedFormUsed bool
 			switch {
 			case s[j+1] == '{':
 				expandableContent, w, cfgSrcName = getBracketedExpandableContent(s, j+1)
 			default:
-				// TODO: To be deprecated removed to align with the upstream behavior
+				deprecatedFormUsed = true
 				expandableContent, w, cfgSrcName = getBareExpandableContent(s, j+1)
 			}
 
 			// At this point expandableContent contains a string to be expanded, evaluate and expand it.
 			switch {
 			case cfgSrcName == "":
 				// Not a config source, expand as os.ExpandEnv
-				// TODO: Align with confmap.strictlyTypedInput feature gate
+				if deprecatedFormUsed {
+					printDeprecationWarningOnce(fmt.Sprintf(
+						"[WARNING] Variable substitution using $VAR has been deprecated in favor of ${VAR} and "+
+							"${env:VAR}. Please update $%s in your configuration.\n", expandableContent))
+				}
 				buf = osExpandEnv(buf, expandableContent, w)
 
 			default:
+				if deprecatedFormUsed {
+					printDeprecationWarningOnce(fmt.Sprintf(
+						"[WARNING] Config source expansion formatted as $uri:selector has been deprecated, "+
+							"use ${uri:selector[?params]} instead. Please replace $%s with ${%s} in your configuration.\n",
+						expandableContent, expandableContent))
+				}
 				// A config source, retrieve and apply results.
 				retrieved, closeFunc, err := retrieveConfigSourceData(ctx, configSources, confmapProviders, cfgSrcName, expandableContent, watcher)
 				if err != nil {
@@ -705,3 +718,12 @@ func MergeCloseFuncs(closeFuncs []confmap.CloseFunc) confmap.CloseFunc {
 		return errs
 	}
 }
+
+var deprecationWarningsPrinted = &sync.Map{}
+
+func printDeprecationWarningOnce(msg string) {
+	if _, ok := deprecationWarningsPrinted.Load(msg); !ok {
+		log.Print(msg)
+		deprecationWarningsPrinted.Store(msg, struct{}{})
+	}
+}