Skip to content

Commit 6d9ffa5

Browse files
authored
fix(secret): set project_id as optional in datasources (#2738)
* fix(secret): set project_id as optional in datasources * fix: set project_id to ptr in API creation helpers
1 parent c3a2c31 commit 6d9ffa5

5 files changed

+581
-740
lines changed

internal/services/secret/helpers.go

+12-10
Original file line numberDiff line numberDiff line change
@@ -33,37 +33,39 @@ func newAPIWithRegion(d *schema.ResourceData, m interface{}) (*secret.API, scw.R
3333
}
3434

3535
// newAPIWithRegionAndDefault returns a new Secret API and the region for a Create request
36-
func newAPIWithRegionProjectIDAndDefault(d *schema.ResourceData, m interface{}, defaultRegion scw.Region) (*secret.API, scw.Region, string, error) {
36+
func newAPIWithRegionOptionalProjectIDAndDefault(d *schema.ResourceData, m interface{}, defaultRegion scw.Region) (*secret.API, scw.Region, *string, error) {
3737
api := secret.NewAPI(meta.ExtractScwClient(m))
3838

3939
region, err := meta.ExtractRegionWithDefault(d, m, defaultRegion)
4040
if err != nil {
41-
return nil, "", "", err
41+
return nil, "", nil, err
4242
}
4343

44+
var projectIDPtr *string
4445
projectID, _, err := meta.ExtractProjectID(d, m)
45-
if err != nil {
46-
return nil, "", "", err
46+
if err == nil {
47+
projectIDPtr = &projectID
4748
}
4849

49-
return api, region, projectID, nil
50+
return api, region, projectIDPtr, nil
5051
}
5152

5253
// newAPIWithRegionAndProjectID returns a new Secret API, with region and projectID
53-
func newAPIWithRegionAndProjectID(d *schema.ResourceData, m interface{}) (*secret.API, scw.Region, string, error) {
54+
func newAPIWithRegionAndProjectID(d *schema.ResourceData, m interface{}) (*secret.API, scw.Region, *string, error) {
5455
api := secret.NewAPI(meta.ExtractScwClient(m))
5556

5657
region, err := meta.ExtractRegion(d, m)
5758
if err != nil {
58-
return nil, "", "", err
59+
return nil, "", nil, err
5960
}
6061

62+
var projectIDPtr *string
6163
projectID, _, err := meta.ExtractProjectID(d, m)
62-
if err != nil {
63-
return nil, "", "", err
64+
if err == nil {
65+
projectIDPtr = &projectID
6466
}
6567

66-
return api, region, projectID, nil
68+
return api, region, projectIDPtr, nil
6769
}
6870

6971
// NewAPIWithRegionAndID returns a Secret API with locality and ID extracted from the state

internal/services/secret/secret_data_source.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ func DataSourceSecretRead(ctx context.Context, d *schema.ResourceData, m interfa
5555
request := &secret.ListSecretsRequest{
5656
Region: region,
5757
Name: types.ExpandStringPtr(secretName),
58-
ProjectID: types.ExpandStringPtr(projectID),
58+
ProjectID: projectID,
5959
OrganizationID: types.ExpandStringPtr(d.Get("organization_id")),
6060
Path: types.ExpandStringPtr(d.Get("path")),
6161
}

0 commit comments

Comments
 (0)