Skip to content

Commit d83c4fa

Browse files
authored
fix(dns): API returns 403 instead of 404 on missing record (#1808)
1 parent 621c60b commit d83c4fa

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

scaleway/resource_domain_record.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -321,7 +321,7 @@ func resourceScalewayDomainRecordRead(ctx context.Context, d *schema.ResourceDat
321321
ID: &recordID,
322322
}, scw.WithAllPages(), scw.WithContext(ctx))
323323
if err != nil {
324-
if is404Error(err) {
324+
if is404Error(err) || is403Error(err) {
325325
d.SetId("")
326326
return nil
327327
}
@@ -351,7 +351,7 @@ func resourceScalewayDomainRecordRead(ctx context.Context, d *schema.ResourceDat
351351
ID: &idRecord,
352352
}, scw.WithAllPages(), scw.WithContext(ctx))
353353
if err != nil {
354-
if is404Error(err) {
354+
if is404Error(err) || is403Error(err) {
355355
d.SetId("")
356356
return nil
357357
}
@@ -370,7 +370,7 @@ func resourceScalewayDomainRecordRead(ctx context.Context, d *schema.ResourceDat
370370

371371
res, err := waitForDNSZone(ctx, domainAPI, dnsZone, d.Timeout(schema.TimeoutRead))
372372
if err != nil {
373-
if is404Error(err) {
373+
if is404Error(err) || is403Error(err) {
374374
d.SetId("")
375375
return nil
376376
}
@@ -505,7 +505,7 @@ func resourceScalewayDomainRecordDelete(ctx context.Context, d *schema.ResourceD
505505

506506
_, err = waitForDNSZone(ctx, domainAPI, d.Get("dns_zone").(string), d.Timeout(schema.TimeoutDelete))
507507
if err != nil && !ErrCodeEquals(err, domain.ErrCodeNoSuchDNSZone) {
508-
if is404Error(err) {
508+
if is404Error(err) || is403Error(err) {
509509
return nil
510510
}
511511
return diag.FromErr(err)
@@ -517,7 +517,7 @@ func resourceScalewayDomainRecordDelete(ctx context.Context, d *schema.ResourceD
517517
DNSZone: d.Get("dns_zone").(string),
518518
})
519519
if err != nil {
520-
if is404Error(err) {
520+
if is404Error(err) || is403Error(err) {
521521
return nil
522522
}
523523
return diag.FromErr(err)

0 commit comments

Comments
 (0)