Skip to content

Commit

Permalink
[Microsoft.Azure.ServiceBus] Fix event source messages 40 and 41 (#22009
Browse files Browse the repository at this point in the history
)

* Fix typo: `AmqpSendAuthenticanToken` → `AmqpSendAuthenticationToken`
* Fix claims: `System.String[]` → `Manage,Listen`
  • Loading branch information
0xced authored Jun 21, 2021
1 parent e33bbdd commit 834ddd4
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -423,17 +423,17 @@ public void AmqpSendAuthenticationTokenStart(Uri address, string audience, strin
{
if (this.IsEnabled())
{
this.AmqpSendAuthenticationTokenStart(address.ToString(), audience, resource, claims.ToString());
this.AmqpSendAuthenticationTokenStart(address.ToString(), audience, resource, string.Join(",", claims));
}
}

[Event(40, Level = EventLevel.Verbose, Message = "AmqpSendAuthenticanToken started. Address: {0}, Audience: {1}, Resource: {2}, Claims: {3}")]
[Event(40, Level = EventLevel.Verbose, Message = "AmqpSendAuthenticationToken started. Address: {0}, Audience: {1}, Resource: {2}, Claims: {3}")]
void AmqpSendAuthenticationTokenStart(string address, string audience, string resource, string claims)
{
this.WriteEvent(40, address, audience, resource, claims);
}

[Event(41, Level = EventLevel.Verbose, Message = "AmqpSendAuthenticanToken done.")]
[Event(41, Level = EventLevel.Verbose, Message = "AmqpSendAuthenticationToken done.")]
public void AmqpSendAuthenticationTokenStop()
{
if (this.IsEnabled())
Expand Down

0 comments on commit 834ddd4

Please sign in to comment.