Repository navigation
Unexpected behavoir when upgrading to 111.3.0 #2235
Description
Hi
Upgraded a little internal tool which uses restsharp to use version 111.3.0 instead 110.2.0 and started to get the following error :
A default parameters with the same name has already been added
After quick check i found that the following code is the problem
var cli = new RestClient();
cli.AddDefaultHeader("User-Agent", "myagent");
return cli;
AddDefaultHeader throws the exception and changing the code to :
var cli = new RestClient();
cli.DefaultParameters.RemoveParameter("User-Agent", ParameterType.HttpHeader);
cli.AddDefaultHeader("User-Agent", "myAgent");
return cli;
Workarounds this problem.
Is this behavior is by design ?
Reacted by Patrik
Activity
alexeyzimarev commentedon Jul 11, 2024
I think it's a bug. By default, parameters with the same aren't allowed. You can always set
AllowMultipleDefaultParametersWithSameNametotrueto override this. But, headers should be excluded as it's normal to have multiple values for the same header.alexeyzimarev commentedon Jul 11, 2024
The real workaround is
client.DefaultParameters.ReplaceParameter(parameter)alexeyzimarev commentedon Jul 11, 2024
Also, setting the user agent is normally done by setting the options property:
hagaygo commentedon Jul 11, 2024
Not sure from you replies whether its gonna get changed.
Workarounds are fine , but as a of this moment , a code that worked fine on previous versions stopped working on newer versions.
Just wanted to report on this issue and it got acknowledged.
Thanks for your work and let me know if you need anything else on my part.
alexeyzimarev commentedon Jul 11, 2024
I just mentioned the workaround while I included the fix to the PR I was opening for the other issue related to headers.