Skip to content

Commit

Permalink
Merge pull request #934 from jfrog/fix-check-license-toggle
Browse files Browse the repository at this point in the history
Revert back to deprecated GetOkExists func
  • Loading branch information
alexhung authored Apr 12, 2024
2 parents 9f83e9c + 4561264 commit 8d054b5
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
6 changes: 6 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,9 @@
## 10.6.1 (Apr 12, 2024)

BUG FIXES:

* provider: Fix `check_license` attribute not process correctly. Issue: [#930](https://github.com/jfrog/terraform-provider-artifactory/issues/930) PR: [#934](https://github.com/jfrog/terraform-provider-artifactory/pull/934)

## 10.6.0 (Apr 12, 2024)

FEATURES:
Expand Down
2 changes: 1 addition & 1 deletion pkg/artifactory/provider/sdkv2.go
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ func providerConfigure(ctx context.Context, d *schema.ResourceData, terraformVer
// Due to migration from SDK v2 to plugin framework, we have to remove defaults from the provider configuration.
// https://discuss.hashicorp.com/t/muxing-upgraded-tfsdk-and-framework-provider-with-default-provider-configuration/43945
checkLicense := true
if v, ok := d.GetOk("check_license"); ok {
if v, ok := d.GetOkExists("check_license"); ok {
checkLicense = v.(bool)
}
if checkLicense {
Expand Down

0 comments on commit 8d054b5

Please sign in to comment.