Lea Fairbanks

Results 7 comments of Lea Fairbanks

Oh, I didn't update the tests, I'll get it tomorrow. My bad.

I generally prefer to avoid those kinds of assumptions. To me, having multiple methods, one for an absolute time, and one for a relative time, is very explicit and behaves...

I initially found the problem because I was trying to see if my tokens were expired and getting an incorrect value. The AbstractToken constructor was taking the absolute expiration time...

I went with your suggested approach, let me know what you think, thanks!

sure thing, work has gotten a bit more stressful in the last couple weeks, I will try to do something this weekend

The CI check seems to fail because it's using an outdated version of bundler. See: https://github.com/bundler/bundler/pull/3559

Sorry for the delay, I went ahead and added a test. Let me know if you like the approach!