[BUG] Config parsing with type conversion and env var replacement mangles input

[th3fallen]

with this config.yaml

server:
  http:
    addr: 0.0.0.0:8000
    timeout: 100s
  grpc:
    addr: 0.0.0.0:9000
    timeout: 100s
data:
  database:
    driver: mysql
    source: ${DB_USERNAME:root}:${DB_PASSWORD:password}@tcp(${DB_HOST:mysql.default.svc.cluster.local:3306})/${DB_NAME:remoteconfig}?parseTime=True&loc=Local
    debug: ${DB_DEBUG:false}

and this config options

c := config.New(
		config.WithResolveActualTypes(true),
		config.WithSource(
			file.NewSource(flagconf),
			env.NewSource(),
		),
	)

What happened:

With the above inputs, when i enable WithResolveActualTypes the value returned from data.database.source is root

What you expected to happen:

data.database.source should return root:password@tcp(mysql.default.svc.cluster.local:3306)/remoteconfig?parseTime=True&loc=Local

How to reproduce it (as minimally and precisely as possible):

Anything else we need to know?:

The error appears to stem from https://github.com/go-kratos/kratos/blob/main/config/options.go#L189

Environment:

• Kratos version (use kratos -v): kratos version v2.8.4
• Go version (use go version):go version go1.24.2 darwin/arm64
• OS (e.g: cat /etc/os-release):OSX Latest

[dosubot]

Hello @th3fallen! 👋 I'm Dosu, and I'm here to help you solve bugs, answer questions, and support your contributions while you wait for a human maintainer. Let's dive into that issue you posted!

The issue you're experiencing with the WithResolveActualTypes function in Kratos seems to be related to how placeholders are resolved and converted to their actual types. When WithResolveActualTypes is enabled, the function processes placeholders and attempts to convert values to their actual types. However, it appears that the resolution process is not handling the full connection string as expected, resulting in only the root part being returned.

Here are a few things to consider:

1. Environment Variables: Ensure that all the environment variables (DB_USERNAME, DB_PASSWORD, DB_HOST, DB_NAME, DB_DEBUG) are correctly set in your environment. If any of these are missing or not set, the default values specified in the placeholders will be used.

2. Type Conversion: The WithResolveActualTypes function converts values to their actual types if enableConvertToType is set to true. This might be affecting how the placeholders are resolved, especially if the conversion logic is not handling complex strings like database connection strings properly.

3. Resolver Logic: The resolver function in options.go is responsible for processing the configuration map and resolving placeholders. It uses the expand function to substitute placeholders and convert values. If the logic within expand or the resolver is not correctly handling nested or complex strings, it could lead to incomplete resolution.

To address this, you might want to review the implementation of the expand function and the resolver logic in options.go to ensure they are correctly handling the resolution and conversion of complex strings like your database connection string ^[1].

To continue talking to Dosu, mention @dosu.

---

^{Help Dosu learn! Give it feedback: Great Response | Irrelevant Answer | Incorrect Sources | Too Verbose | Hallucination | Bug Report | Other}