Skip to content
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

SubscriptionCheckpoint event type not filtered. #4079

Open
ylorph opened this issue Nov 23, 2023 · 1 comment
Open

SubscriptionCheckpoint event type not filtered. #4079

ylorph opened this issue Nov 23, 2023 · 1 comment
Labels
kind/bug Issues which are a software defect linear subsystem/persistent subs Issues relating to persistent subscriptions

Comments

@ylorph
Copy link
Contributor

ylorph commented Nov 23, 2023

Describe the bug

Presistent subscription checkpoint event type SubscriptionCheckpoint are not taken into account when using EventTypeFilter.ExcludeSystemEvents() because the type is not prefixed with $

To Reproduce
N/A

Expected behavior
when using EventTypeFilter.ExcludeSystemEvents() those event type should be filtered out

Actual behavior

SubscriptionCheckpoint are pushed to consumers when excluding system events.

Config/Logs/Screenshots
N/A

EventStore details

  • EventStore server version: All
  • Operating system: N/A
  • EventStore client version (if applicable): Any client

Additional context

Could be done as additional behavior on the filter system as well as adding the correct prfix to the event type in the databse

https://discuss.eventstore.com/t/subscription-returning-system-events/5037/2

latest version of code appending the event at the moment of report

DB-543

@ylorph ylorph added kind/bug Issues which are a software defect subsystem/persistent subs Issues relating to persistent subscriptions linear labels Nov 23, 2023
@ylorph
Copy link
Contributor Author

ylorph commented Apr 8, 2024

Good point, would a filter based on both the event type & the stream name work ?

https://github.com/EventStore/EventStore/blob/a3125bc11cebecd342f6421829201a9eeb8c90d6/src/EventStore.Core/Services/PersistentSubscription/PersistentSubscriptionCheckpointWriter.cs#L18C30-L18C89
something like ;

eventType=="SubscriptionCheckpoint" 
&& streamName.startwith( "$persistentsubscription-") 
&& streaName.endsWith("-checkpoint")

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Issues which are a software defect linear subsystem/persistent subs Issues relating to persistent subscriptions
Projects
None yet
Development

No branches or pull requests

1 participant