Skip to content

Add integration points for ConfigurationOptions customizers#1809

Merged
NickCraver merged 8 commits intoStackExchange:mainfrom
philon-msft:main
Jul 26, 2021
Merged

Add integration points for ConfigurationOptions customizers#1809
NickCraver merged 8 commits intoStackExchange:mainfrom
philon-msft:main

Conversation

@philon-msft
Copy link
Copy Markdown
Collaborator

Enables the use of extensions that customize ConfigurationOptions settings to tune behavior or add new capabilities. As proposed in #1801

Adds:

  • New overloads of the ConnectionMultiplexer.Connect() and ConnectAsync() methods, that take an additional Action parameter to customize the ConfigurationOptions
  • A new ConfigurationOptions.Apply() method to apply the same type of customizer directly to a ConfigurationOptions instance, before connecting

philon-msft and others added 6 commits July 1, 2021 11:38
Prefix each entry with a timestamp that looks like:

2021-07-01T18:18:47.9263352Z: cachename.redis.cache.windows.net:6380/Interactive: Connecting...
2021-07-01T18:18:47.9463333Z: cachename.redis.cache.windows.net:6380: BeginConnectAsync
2021-07-01T18:18:47.9483351Z: 1 unique nodes specified
This also shortens the format to HH:mm:ss.ffff, which is all we need for debugging and it's less to grok. Also prevents double timestamp logging in tests.
Nick Craver added 2 commits July 25, 2021 16:12
Also re-arranges `ConnectAsync` to be under the overloads like `Connect`.

I don't think we want `null` to go boom here - no real win case for the user, so let's just make it handle null with no fuss.
@NickCraver NickCraver merged commit 56ac9a2 into StackExchange:main Jul 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Proposal: Extensibility point to support automated application of configuration options

3 participants