-
Notifications
You must be signed in to change notification settings - Fork 9.6k
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 panic when import id is empty string #34625
fix panic when import id is empty string #34625
Conversation
f9b321a
to
14a8009
Compare
5614e92
to
e0cb633
Compare
e0cb633
to
a281646
Compare
a281646
to
f56b19a
Compare
We can only detect this during evaluation since the import id is an hcl expression. Fixes #33505
f56b19a
to
2b7c7b2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! I think this can be backported as well.
Reminder for the merging maintainer: if this is a user-visible change, please update the changelog on the appropriate release branch. |
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. |
We can only detect this during evaluation since the import id is an hcl expression
Fixes #33505
Target Release
1.7.x
Draft CHANGELOG entry
BUG FIXES