Skip to content

Commit

Permalink
Add default constructor to fix deserialization error
Browse files Browse the repository at this point in the history
  • Loading branch information
jradhakrishnan committed Aug 22, 2023
1 parent 58e7532 commit c6b81ed
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 8 deletions.
16 changes: 9 additions & 7 deletions src/main/java/ai/asserts/aws/account/AWSAccount.java
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@
import lombok.Builder;
import lombok.EqualsAndHashCode;
import lombok.Getter;
import lombok.NoArgsConstructor;
import lombok.Setter;
import lombok.ToString;
import lombok.experimental.SuperBuilder;
Expand All @@ -21,22 +22,23 @@
@EqualsAndHashCode
@ToString
@SuperBuilder
@NoArgsConstructor
public class AWSAccount {
@Setter
private String tenant;
// Use different json field name to match property from API Response
@JsonProperty("accountID")
private final String accountId;
private final String name;
private String accountId;
private String name;
@ToString.Exclude
private final String accessId;
private String accessId;
@ToString.Exclude
private final String secretKey;
private String secretKey;
// Use different json field name to match property from API Response
@JsonProperty("assumeRoleARN")
private final String assumeRole;
private final String externalId;
private final boolean paused;
private String assumeRole;
private String externalId;
private boolean paused;
@Builder.Default
private final Set<String> regions = new TreeSet<>();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
import lombok.AllArgsConstructor;
import lombok.EqualsAndHashCode;
import lombok.Getter;
import lombok.NoArgsConstructor;
import lombok.ToString;
import lombok.experimental.SuperBuilder;
import lombok.extern.slf4j.Slf4j;
Expand Down Expand Up @@ -97,8 +98,8 @@ private Set<AWSAccount> getAccountsInternal() {
AWSAccount ac = new AWSAccount(tenantName, accountId, null, null, null,
regions);
accountRegions.putIfAbsent(ac.getAccountId(), ac);
log.info("Scraping AWS Accounts {}", accountRegions);
}
log.info("Scraping AWS Accounts {}", accountRegions);
return Sets.newHashSet(accountRegions.values());
}

Expand All @@ -111,6 +112,7 @@ String getTenantName() {
@Getter
@SuperBuilder
@ToString
@NoArgsConstructor
public static class CloudwatchConfigs {
List<AWSAccount> cloudWatchConfigs;
}
Expand Down

0 comments on commit c6b81ed

Please sign in to comment.