Skip to content

Commit

Permalink
MINOR: fix model dump dict key names (#19484) (#19485)
Browse files Browse the repository at this point in the history
Co-authored-by: Abdallah Serghine <[email protected]>
  • Loading branch information
KylixSerg and Abdallah Serghine authored Jan 23, 2025
1 parent 06759c5 commit a631d22
Showing 1 changed file with 7 additions and 6 deletions.
13 changes: 7 additions & 6 deletions ingestion/src/metadata/clients/aws_client.py
Original file line number Diff line number Diff line change
Expand Up @@ -58,12 +58,13 @@ class AWSAssumeRoleCredentialResponse(BaseModel):


class AWSAssumeRoleCredentialWrapper(BaseModel):
accessKeyId: str = Field()
secretAccessKey: CustomSecretStr = Field()
sessionToken: Optional[str] = Field(
default=None,
)
expiryTime: Optional[str] = Field()
accessKeyId: str = Field(alias="access_key")
secretAccessKey: CustomSecretStr = Field(alias="secret_key")
sessionToken: Optional[str] = Field(default=None, alias="token")
expiryTime: Optional[str] = Field(alias="expiry_time")

class Config:
populate_by_name = True


AWSAssumeRoleCredentialFormat = TypeVar(
Expand Down

0 comments on commit a631d22

Please sign in to comment.