-
Notifications
You must be signed in to change notification settings - Fork 19
enforce event to topic map if the endpoint specifies #1242
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
Changes from 4 commits
b69595e
f37bd85
f18f73a
1525dec
9985343
e57a596
b1f05b0
e0f043a
6b7d152
f8e7b7b
0682423
de97db0
9acea2a
651478d
37d0f31
4be01be
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -28,6 +28,16 @@ protected TopicTopology(TopologyOptions options, IValidateOptions<TopologyOption | |
|
||
internal TopologyOptions Options { get; } | ||
|
||
/// <summary> | ||
/// Determines if an exception should be thrown when attempting to publish an event not mapped in PublishedEventToTopicsMap | ||
/// </summary> | ||
public bool ThrowIfUnmappedEventTypes | ||
|
||
{ | ||
get => Options.ThrowIfUnmappedEventTypes; | ||
set => Options.ThrowIfUnmappedEventTypes = value; | ||
} | ||
|
||
|
||
/// <summary> | ||
/// Creates an instance of the topology object based on serializable state. | ||
/// </summary> | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -43,4 +43,9 @@ public Dictionary<string, string> QueueNameToSubscriptionNameMap | |
get; | ||
init => field = value ?? []; | ||
} = []; | ||
|
||
/// <summary> | ||
/// Determines if an exception should be thrown when attempting to publish an event not mapped in PublishedEventToTopicsMap | ||
/// </summary> | ||
public bool ThrowIfUnmappedEventTypes { get; set; } = false; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same comment as above. I think this is the time when we need to introduce the There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Alternative name suggestions: ThrowForUnmappedEventTypes There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. discussed in call: having a new subclass of |
||
} |
Uh oh!
There was an error while loading. Please reload this page.