Skip to content

Commit

Permalink
WebAPI dynamic call should not enumerate byte[] arguments
Browse files Browse the repository at this point in the history
  • Loading branch information
azuisleet committed May 19, 2019
1 parent 4195d47 commit fa128f1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion SteamKit2/SteamKit2/Steam/WebAPI/WebAPI.cs
Original file line number Diff line number Diff line change
Expand Up @@ -396,7 +396,7 @@ public override bool TryInvokeMember( InvokeMemberBinder binder, object[] args,
continue;
}
// flatten lists
else if ( argValue is IEnumerable && !( argValue is string ) )
else if ( argValue is IEnumerable && !( argValue is string || argValue is byte[] ) )

This comment has been minimized.

Copy link
@JustArchi

JustArchi May 27, 2019

Contributor

Perhaps it'd be better to change byte[] to IEnumerable<byte>, just in case somebody uses stuff like LINQ or other types of collections?

{
int index = 0;
var enumerable = argValue as IEnumerable;
Expand Down

0 comments on commit fa128f1

Please sign in to comment.