Fix: convert session.expires to Date first. Might be a string due to

[ardeearam]

…Redis seralization.

WHY are these changes introduced?

Expectation: verifyRequest should work out of the box.

Actual: verifyRequest redirects to the authRoute link even if the session has been successfully retrieved from the session store.

WHAT is this pull request doing?

This fixes #40

session.expires may not always be a Date object. When serialized and deserialized, it may return a string object. The comparison between session.expires and new Date() will have unexpected results.

Type of change

• Patch: Bug (non-breaking change which fixes an issue)
• Minor: New feature (non-breaking change which adds functionality)
• Major: Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

• I have added a changelog entry, prefixed by the type of change noted above
• I have added/updated tests for this change

[paulomarg]

Hey @ardeearam, thank you for your contribution! This is actually a bug in the underlying library which we are already fixing: Shopify/shopify-api-js#132.

I'm going to close this PR since that fix should also cover this!