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

Back-port fix for producing correct UTC DateTime values by default, to ASP .NET 3.1.x #27618

Closed
augustoproiete opened this issue Nov 7, 2020 · 2 comments
Labels
area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates ✔️ Resolution: Answered Resolved because the question asked by the original author has been answered. Status: Resolved

Comments

@augustoproiete
Copy link

This is a follow-up from issue #11584 which was fixed by PR #24893 but seems it will be available in only in .NET 5.

As it stands today and described in more detail in #11584, sending a UTC time string such as 2020-11-07T02:30:04.0576719Z to an ASP .NET Core 3.1 API converts it to a DateTime with DateTimeKind.Local instead of DateTimeKind.Utc causing the API to think a conversion to UTC is necessary, leading to bad data and bugs in applications.

I completely understand the policy of not back-porting non-critical bug fixes to earlier releases which makes sense, but I am advocating that this is indeed a very critical bug that should be back-ported to (at least) ASP .NET 3.1.x. given that being able to correctly parse a UTC time string is fundamental to most (if not all) applications 😅.

If back-porting is not an option, then properly documenting it on docs.microsoft.com with a code sample/template would be the next best thing.

/cc @pranavkm

@javiercn javiercn added the area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates label Nov 9, 2020
@captainsafia
Copy link
Member

@augustoproiete While we won't be back-porting this bug-fix, you can use the code in #11584 (comment) as a reference for how to implement a fix in existing 3.1 applications.

@captainsafia captainsafia added the ✔️ Resolution: Answered Resolved because the question asked by the original author has been answered. label Nov 9, 2020
@ghost ghost added the Status: Resolved label Nov 9, 2020
@ghost
Copy link

ghost commented Nov 10, 2020

This issue has been resolved and has not had any activity for 1 day. It will be closed for housekeeping purposes.

See our Issue Management Policies for more information.

@ghost ghost closed this as completed Nov 10, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Dec 10, 2020
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates ✔️ Resolution: Answered Resolved because the question asked by the original author has been answered. Status: Resolved
Projects
None yet
Development

No branches or pull requests

3 participants