Skip to content

Fix "The Castle Windsor Implementation doesn't work with exception handlers #741" issue #742

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.

Sign up for GitHub

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

Merged
jbogard merged 4 commits into from
May 18, 2022
Merged

Fix "The Castle Windsor Implementation doesn't work with exception handlers #741" issue #742

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
5a251d0
Fix MediatR examples exception handlers
alexandruchirita4192 May 17, 2022
7a1525b
Fix MediatR examples exception handlers overrides
alexandruchirita4192 May 17, 2022
add1900
Explain some important code in examples
alexandruchirita4192 May 17, 2022
19211b3
Castle Windsor test streams too and fix all checks
alexandruchirita4192 May 17, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Fix MediatR examples exception handlers
  • Loading branch information
@alexandruchirita4192
alexandruchirita4192 authored May 17, 2022
commit 5a251d070a8b7355bd469769239ae502a0b6a47e
22 changes: 21 additions & 1 deletion samples/MediatR.Examples/ExceptionHandler/ExceptionsHandlers.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,12 @@ protected override async Task Handle(PingResource request,
RequestExceptionHandlerState<Pong> state,
CancellationToken cancellationToken)
{
// Exception type name required because it is checked later in messages
await _writer.WriteLineAsync($"Handling {exception.GetType().FullName}");

// Exception handler type name required because it is checked later in messages
await _writer.WriteLineAsync($"---- Exception Handler: '{typeof(CommonExceptionHandler).FullName}'").ConfigureAwait(false);

state.SetHandled(new Pong());
}
}
Expand All @@ -33,7 +38,12 @@ public async Task Handle(PingResource request,
RequestExceptionHandlerState<Pong> state,
CancellationToken cancellationToken)
{
// Exception type name required because it is checked later in messages
await _writer.WriteLineAsync($"Handling {exception.GetType().FullName}");

// Exception handler type name required because it is checked later in messages
await _writer.WriteLineAsync($"---- Exception Handler: '{typeof(ConnectionExceptionHandler).FullName}'").ConfigureAwait(false);

state.SetHandled(new Pong());
}
}
Expand All @@ -49,7 +59,12 @@ public async Task Handle(PingResource request,
RequestExceptionHandlerState<Pong> state,
CancellationToken cancellationToken)
{
// Exception type name required because it is checked later in messages
await _writer.WriteLineAsync($"Handling {exception.GetType().FullName}");

// Exception handler type name required because it is checked later in messages
await _writer.WriteLineAsync($"---- Exception Handler: '{typeof(AccessDeniedExceptionHandler).FullName}'").ConfigureAwait(false);

state.SetHandled(new Pong());
}
}
Expand All @@ -65,7 +80,12 @@ public virtual async Task Handle(PingNewResource request,
RequestExceptionHandlerState<Pong> state,
CancellationToken cancellationToken)
{
// Exception type name required because it is checked later in messages
await _writer.WriteLineAsync($"Handling {exception.GetType().FullName}");

// Exception handler type name required because it is checked later in messages
await _writer.WriteLineAsync($"---- Exception Handler: '{typeof(ServerExceptionHandler).FullName}'").ConfigureAwait(false);

state.SetHandled(new Pong());
}
}
}