Skip to content

Conversation

@singh733
Copy link
Contributor

@singh733 singh733 commented Nov 3, 2022

HttpSys request header dictionary does not remove headers when Remove() is called # 43230

Fixes # 43230

@ghost ghost added the community-contribution Indicates that the PR has been added by a community member label Nov 3, 2022
@ghost
Copy link

ghost commented Nov 3, 2022

Thanks for your PR, @singh733. Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

@Tratcher
Copy link
Member

Also I made the changes in .tt file (which automatically reflected in the generated file)

The auto generation is working? I thought it was broken/disabled years ago. #5869. How did you make it run?

@singh733
Copy link
Contributor Author

Also I made the changes in .tt file (which automatically reflected in the generated file)

The auto generation is working? I thought it was broken/disabled years ago. #5869. How did you make it run?

By adding TextTemplatingFileGenerator in the Custom tool it works.

image

When we make any change and save it then the following pop up came. It automatically update the generated file

image

@ladeak
Copy link
Contributor

ladeak commented Nov 18, 2022

Hi, I am looking at #45156 touching the same files.

To confirm the goal of this change: after this change _flag0 and _flag1 will indicate if the value of the header is read from the native memory or set by the user. When the user deletes the value, _flag0 and _flag1 would remain set, but an empty string would be returned?

@Tratcher
Copy link
Member

Hi, I am looking at #45156 touching the same files.

To confirm the goal of this change: after this change _flag0 and _flag1 will indicate if the value of the header is read from the native memory or set by the user. When the user deletes the value, _flag0 and _flag1 would remain set, but an empty string would be returned?

A null value, but otherwise yes.

// Tracks if individual fields have been read from native or set directly.
// Once read or set, their presence in the collection is marked by if their StringValues is null or not.
private UInt32 _flag0, _flag1;

@singh733
Copy link
Contributor Author

@Tratcher I have done all the changes as Suggested. Please let me know if you need any change.

@Tratcher Tratcher merged commit fddab79 into dotnet:main Nov 22, 2022
@ghost ghost added this to the 8.0-preview1 milestone Nov 22, 2022
@Tratcher
Copy link
Member

Thanks

@amcasey amcasey added area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions and removed area-runtime labels Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants