Skip to content

Expose time provider for TimeWindowFilter#562

Merged
zhiyuanliang-ms merged 1 commit intomainfrom
zhiyuanliang/expose-timeprovider
Oct 22, 2025
Merged

Expose time provider for TimeWindowFilter#562
zhiyuanliang-ms merged 1 commit intomainfrom
zhiyuanliang/expose-timeprovider

Conversation

@zhiyuanliang-ms
Copy link
Member

Why this PR?

#558

Visible Changes

Now TimeWindowFilter.SystemClock is public.

DI usage:

builder.Services.AddSingleton<TimeProvider, CustomTimeProvider>();
builder.Services.AddFeatureManagement(); // feature manager builder will get TimeProvider service from DI container

If there is no registered TimeProvider service, SystemTimeProvider will be used.

Direct usage:

var mockedTimeWindowFilter = new TimeWindowFilter()
{
    SystemClock = mockedTimeProvider
};

@zhiyuanliang-ms zhiyuanliang-ms merged commit 7b4f387 into main Oct 22, 2025
4 checks passed
@zhiyuanliang-ms zhiyuanliang-ms deleted the zhiyuanliang/expose-timeprovider branch November 9, 2025 12:23
zhiyuanliang-ms added a commit that referenced this pull request Dec 18, 2025
* Merge pull request #556 from microsoft/zhiyuanliang/fix-snapshot-bug

Fix snapshot cache key bug

* add testcase for configuration manager (#557)

* Expose time provider for TimeWindowFilter (#562)

* Update FilterCollectionExtensions (#359)

* update

* update

* update

* update AddForFeature

* add test

* update

* version bump 4.4.0 (#569)
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.

3 participants