Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

QueryAsync doesn't propagate Exception #39

Open
1 task done
tkouba opened this issue Jan 2, 2025 · 0 comments
Open
1 task done

QueryAsync doesn't propagate Exception #39

tkouba opened this issue Jan 2, 2025 · 0 comments
Labels
bug Something isn't working

Comments

@tkouba
Copy link

tkouba commented Jan 2, 2025

Prerequisites

  • Put an X between the brackets on this line if you have done all of the following:
    • Checked that this issue isn't already filed
    • Checked that this issue isn't already fixed in a newer version

Describe the bug

Method public async Task<WeatherForecast?> QueryAsync(WeatherForecastOptions options) doesn't propagate Exception.

To Reproduce

See source code

public async Task<WeatherForecast?> QueryAsync(WeatherForecastOptions options)
{
try
{
return await GetWeatherForecastAsync(options);
}
catch (Exception)
{
return null;
}
}

Expected behavior

Throw Exception instead of returning null.

Version

2.0.0.

@tkouba tkouba added the bug Something isn't working label Jan 2, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant