Actix-web: Why chrono is removed?

Created on 28 Mar 2020  路  7Comments  路  Source: actix/actix-web

789

But CookieBuilder still using time, instead of chrono.

My case:

  • I have chrono::NaiveDateTime from my business-logic layer, and I want to set cookie expiration time.

Most helpful comment

I don't have an opinion on this, I didn't remove it, @fafhrd91 did. I'm just answering your question. You asked why it was removed, the answer to your question is in the comment I linked. We cannot ask for more details surrounding this unfortunately, since the original author is sadly no longer with us.

All 7 comments

@JohnTitor What about to migrate to chrono in 3.0.0? I can help with it

It is explained in the PR you linked why it is removed: https://github.com/actix/actix-web/pull/789#issuecomment-485041718

I don't have an opinion on this, I didn't remove it, @fafhrd91 did. I'm just answering your question. You asked why it was removed, the answer to your question is in the comment I linked. We cannot ask for more details surrounding this unfortunately, since the original author is sadly no longer with us.

can this be closed? @sergeysove

What about to add support for chrono?

@sergeysova Could you make a PR to add it?

Was this page helpful?
0 / 5 - 0 ratings

Related issues

gh67uyyghj picture gh67uyyghj  路  3Comments

sebzim4500 picture sebzim4500  路  3Comments

alex picture alex  路  4Comments

Eilie picture Eilie  路  5Comments

icommit picture icommit  路  3Comments