Skip to content

Protect casting against nil conversion#1552

Merged
annakhm merged 1 commit intomasterfrom
protect-empty-block
Mar 3, 2025
Merged

Protect casting against nil conversion#1552
annakhm merged 1 commit intomasterfrom
protect-empty-block

Conversation

@annakhm
Copy link
Copy Markdown
Contributor

@annakhm annakhm commented Feb 24, 2025

Empty block is represeted in schema by slice of size 1 with nil value. Wherever empty block is allowed (no required attributes), we should protect against nil values in slice.

@annakhm annakhm force-pushed the protect-empty-block branch from ebcc887 to 67d4bc4 Compare February 25, 2025 00:17
@annakhm
Copy link
Copy Markdown
Contributor Author

annakhm commented Feb 25, 2025

/test-all

@annakhm
Copy link
Copy Markdown
Contributor Author

annakhm commented Mar 1, 2025

/test-all

@annakhm annakhm requested review from ksamoray and salv-orlando March 1, 2025 01:28
@ksamoray
Copy link
Copy Markdown
Contributor

ksamoray commented Mar 3, 2025

/test-all

1 similar comment
@ksamoray
Copy link
Copy Markdown
Contributor

ksamoray commented Mar 3, 2025

/test-all

Empty block is represeted in schema by slice of size 1 with
nil value. Wherever empty block is allowed (no required attributes),
we should protect against nil values in slice.

Signed-off-by: Anna Khmelnitsky <akhmelnitsky@vmware.com>
@annakhm annakhm force-pushed the protect-empty-block branch from 4a7b002 to b5c4fb6 Compare March 3, 2025 17:34
@annakhm
Copy link
Copy Markdown
Contributor Author

annakhm commented Mar 3, 2025

/test-all

@annakhm annakhm merged commit 26ba328 into master Mar 3, 2025
12 checks passed
@annakhm annakhm deleted the protect-empty-block branch March 3, 2025 23:17
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.

NSXT Terraform provider crashes when trying to add nsxt_policy_intrusion_service_profile

3 participants