Fix errors when From is null in telemetry #3436
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes: #3395
If
TelemetryLoggerMiddleware
is used and thenCreateConversationAsync()
is called, there are cases whereActivity.From
will be null, which causes things like this to throw:Changes:
Adds a few null-conditional operators, where appropriate.
Testing:
Added one test which calls
CreateConversationAsync()
while ensuring theActivity.From
is null.