Failed At is way out about sidekiq-failures

[mhfs]

Sounds feasible. Would you be willing to take a look on a possible solution? Do you see that happening to other dates in sidekiq like in the retries tab? Don't have the time to look into that for a few weeks but would be happy to merge a PR.

I'll try and have a look this weekend during RailsCamp AU.

[coolzilj]

@Soliah Did you figure it out? I ran into the same issue.
I found a solution coolzilj/sidekiq-failures@498d297, but just can't understand why it solve this problem?

[coolzilj]

@Soliah @mhfs
I think I understand why the hell this happening now.

My system time zone is 'CST' (China Standard Time)

Time.now.getutc.iso8601
=> "2013-09-27T05:28:49Z" (Correct!)

Time.parse(Time.now.strftime("%Y/%m/%d %H:%M:%S %Z")).getutc.iso8601
=> "2013-09-27T19:29:42Z" (This is what I got in sidekiq-failures, and IT IS WRONG!)

---

In Time::parse doc

Since there are numerous conflicts among locally defined time zone abbreviations all over the world, this method is not intended to understand all of them. For example, the abbreviation “CST” is used variously as:

-06:00 in America/Chicago,
-05:00 in America/Havana,
+08:00 in Asia/Harbin,
+09:30 in Australia/Darwin,
+10:30 in Australia/Adelaide,
etc.
Based on this fact, this method only understands the time zone abbreviations described in RFC 822 and the system time zone, in the order named. (i.e. a definition in RFC 822 overrides the system time zone definition.) The system time zone is taken from Time.local(year, 1, 1).zone and Time.local(year, 7, 1).zone. If the extracted time zone abbreviation does not match any of them, it is ignored and the given time is regarded as a local time.

In RFC 822 doc

zone = /  "CST" / "CDT"                ;  Central:  - 6/ - 5

CST means Central Standard Time (North America),
so Time.parse(Time.now.strftime("%Y/%m/%d %H:%M:%S %Z")).getutc.iso8601 this method would treat
'CST' (China Standard Time) as CST (Central Standard Time (North America)).
So what I got is totally wrong.

My solution is just use Time.parse(Time.now.strftime("%Y/%m/%d %H:%M:%S")).getutc.iso8601 ,
then Time::parse would use the local time.

refs:

• rfc822
• Time::parse
• CST

[layerssss]

nice!

[mhfs]

I decided to keep everything UTC as Sidekiq seems to do. Thanks for the help.