fix panic when passing nil to time.Time.In() - Githubissues

snowflakedb / gosnowflake

Go Snowflake Driver

Apache License 2.0

292 stars 122 forks source link

fix panic when passing nil to time.Time.In() #1100

Closed madisonchamberlain closed 2 months ago

madisonchamberlain commented 4 months ago

Description

SNOW-XXX Please explain the changes you made here. When time.Time.In is used, if a nil is passed in it will panic. (See docs) In the other functions in converter.go (see stringToValue timestamp_ltz) if the loc is nil a default will be used.

Bug was introduced in this pr

Checklist

[ ] Created tests which fail without the change (if possible)
[ ] Extended the README / documentation, if necessary