-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Documentation for what's new in Steeltoe v4 #337
base: v4
Are you sure you want to change the base?
Conversation
Preview link: https://docs-steeltoe-pr-337.azurewebsites.net
|
0ab5de4
to
3adf1ce
Compare
3adf1ce
to
e0e6f02
Compare
Co-authored-by: Tim Hess <[email protected]>
api/v4/welcome/whats-new.md
Outdated
|
||
- Placeholder substitution changed internally (wrapping and taking ownership of sources), should be added as late as possible | ||
- To improve performance, Config Server provider doesn't substitute placeholders by default anymore | ||
- Call `AddPlaceholderResolver()` *before* `AddConfigServer()` to substitute placeholders from local appsettings.json |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Call `AddPlaceholderResolver()` *before* `AddConfigServer()` to substitute placeholders from local appsettings.json | |
- Call `AddPlaceholderResolver()` *before* `AddConfigServer()` to substitute placeholders before calling Config Server |
Calling it before is really only useful if there's a placeholder used for the Config Server client config, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I suppose it could be something like (not actually existing properties):
{
"config-server-url": "http://${CONFIG_SERVER_HOSTNAME}:${CONFIG_SERVER_PORT}/path"
}
where CONFIG_SERVER_HOSTNAME
and CONFIG_SERVER_PORT
are environment variables.
Co-authored-by: Tim Hess <[email protected]>
Documents what's changed in the Steeltoe codebase, except for Management (will follow later).
Intentionally kept everything on a single page to enable search over all Steeltoe components.
Addresses the "welcome" part of #317.