Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: handle nil metadata #2555

Merged
merged 1 commit into from
Jul 7, 2023
Merged

fix: handle nil metadata #2555

merged 1 commit into from
Jul 7, 2023

Conversation

tim775
Copy link
Member

@tim775 tim775 commented Jul 7, 2023

from sentry

"runtime error: invalid memory address or nil pointer dereference"

], 
"stacktrace": 
"1: running [Created by errgroup.(*Group).Go @ errgroup.go:72]
   :24                                                             Stack()
   :270                                                            (*parallelRunner).run.func1.1()
   :884                                                            panic()
   :233                                                            (*TerragruntHCLProvider).LoadResources.func2()
   :299                                                            order2_func()
   :308                                                            median_func()
   :316                                                            medianAdjacent_func()
   :279                                                            choosePivot_func()
   :89                                                             pdqsort_func()
   :23                                                             Slice()
   :232                                                            (*TerragruntHCLProvider).LoadResources()
   ...

@tim775 tim775 requested a review from aliscott July 7, 2023 04:09
@tim775 tim775 self-assigned this Jul 7, 2023
@tim775 tim775 merged commit 9eee9d4 into master Jul 7, 2023
9 of 10 checks passed
@tim775 tim775 deleted the fix/handle_nil_metadata branch July 7, 2023 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants