Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

DI registrations create separate backing stores when tracing is disabled

Đang mở
  • Có gì không ổn?
#610 2 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 7 ngày

@nmummau đang làm issue này rồi.

Từ ngày 14/10/2026.

  • #622 của @nmummau — đang mở

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
15/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Đình trệ
Công nghệ
csharp
Lĩnh vực
backend

Hướng nghiên cứu

The issue names StoreRegistrationTests.cs under src/Extensions/test/Eventuous.Tests.DependencyInjection and describes the affected AddEventStore, AddEventReaderWriter, AddEventReader, and AddEventWriter APIs. Start by reviewing the existing regression tests and tracing how registrations resolve shared stores with diagnostics enabled and disabled. Done means the listed host, factory, scope, and diagnostics cases pass; the issue says an implementation is already committed locally, so check whether that work is still available before starting.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Related test issue

Discovered while implementing #608 — event-store dependency injection coverage.

Problem

With EventuousDiagnostics disabled, store registration methods register implementations or factories separately for each interface. Each interface therefore gets its own singleton rather than sharing one backing instance.

A successful append through IEventWriter can be followed by StreamNotFound when the same stream is read through IEventReader. This is reproducible with the stateful InMemoryEventStore; it is not a claim that persistent providers lose stored data. Factories can also run more than once, and standalone reader/writer registrations can ignore an already registered concrete store.

Affected APIs include the generic and factory overloads of AddEventStore<T>, AddEventReaderWriter<T>, and separately composed AddEventReader<T> / AddEventWriter<T>.

Reproduction

  1. Disable diagnostics, preserving the previous setting for restoration afterward.
  2. Build a host with services.AddEventStore<InMemoryEventStore>().
  3. Resolve IEventWriter and IEventReader from that host.
  4. Append an event to a new stream through the writer.
  5. Read that stream through the reader.

Actual: the read throws StreamNotFound because the interfaces resolve to different stores.

Expected: all interfaces registered for the same implementation share one backing singleton. Changing tracing configuration must not change which events are visible.

Fix and acceptance criteria

  • Register the concrete implementation or factory once, then resolve interface aliases from that singleton.
  • Verify append/read behavior through a real host for store, combined reader/writer, and separately registered reader/writer APIs.
  • Cover generic and factory overloads with tracing enabled and disabled.
  • Factories are invoked once for the shared instance, and interface resolutions remain singleton across scopes.
  • Standalone reader/writer registrations reuse a pre-registered concrete instance without invoking a replacement factory.
  • Tests restore the process-wide diagnostics setting and cannot race with other tests that observe it.

Existing regression tests and implementation

StoreRegistrationTests.cs in src/Extensions/test/Eventuous.Tests.DependencyInjection contains:

  • WrittenEventsAreVisibleThroughEveryRegisteredInterface
  • StandaloneRegistrationUsesTheExistingBackingStore
  • The tracing-disabled cases of RepeatedRegistrationKeepsOneServiceAndOneFactory

These scenarios failed before the production fix. The complete DI project passes all 31 tests on .NET 10 after the fixes for this issue and the companion registration issue.

An implementation is already committed locally on fix/issue-608-store-registrations, commit df3dbe48. It also fixes duplicate registrations and overridden services; this draft does not imply the fix has been published or merged.

Ngôn ngữ chính
C#
Star
512
Fork
99
Merge trung bình
5 ngày 6 giờ
Pull request đã merge (30 ngày)
5

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của Eventuous/eventuous

Tất cả issue của Eventuous/eventuous

Issue tương tự

Thêm issue về C#

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.