Skip to content

Conversation

@jaidisido
Copy link
Contributor

Description of changes:

  • Datetime method needs to be UTC timezone aware, otherwise datetime.datetime(year=1970, month=1, day=1) will return a different timestamp value (potentially negative) depending on the user's timezone
  • Set security token explicitly in Redshift test file

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@jaidisido jaidisido self-assigned this Mar 4, 2021
@jaidisido jaidisido requested a review from igorborgest March 4, 2021 18:11
@jaidisido jaidisido added the enhancement New feature or request label Mar 4, 2021
@jaidisido jaidisido added this to the 2.6.0 milestone Mar 4, 2021
@jaidisido jaidisido merged commit 7600bdf into main Mar 4, 2021
@jaidisido jaidisido deleted the cwe-redshift-tests branch March 4, 2021 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants