Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Encoding is in CharacterSet for HTML #346

Merged
merged 1 commit into from
Nov 25, 2019

Conversation

rboy1
Copy link
Contributor

@rboy1 rboy1 commented Nov 20, 2019

When the content is HTML the encoding is provided in CharacterSet, the ConentEncoding applies to the whole body and typically is used to indicate compressed content and not really encoding of the characters.

https://stackoverflow.com/questions/5938007/what-is-the-difference-between-content-type-charset-x-and-content-encoding-x

When the content is HTML the encoding is provided in CharacterSet, the ConentEncoding applies to the whole body and typically is used to indicate compressed content and not really encoding of the characters.

https://stackoverflow.com/questions/5938007/what-is-the-difference-between-content-type-charset-x-and-content-encoding-x
@JonathanMagnan JonathanMagnan self-assigned this Nov 25, 2019
@JonathanMagnan JonathanMagnan merged commit 127d21a into zzzprojects:master Nov 25, 2019
@JonathanMagnan
Copy link
Member

Hello @rboy1 ,

Thank you for your pull, it has been released in the v1.11.17

We love the change but we are not sure yet of the side impact. So depending on what will be reported, we might eventually need to change your code to enable it as an option.

Best Regards,

Jonathan


Performance Libraries
context.BulkInsert(list, options => options.BatchSize = 1000);
Entity Framework ExtensionsEntity Framework ClassicBulk OperationsDapper Plus

Runtime Evaluation
Eval.Execute("x + y", new {x = 1, y = 2}); // return 3
C# Eval FunctionSQL Eval Function

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants