You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "topherinternational (via GitHub)" <gi...@apache.org> on 2024/11/07 22:49:32 UTC

[PR] Fix PostgresHook bug when getting AWS Redshift Serverless credentials [airflow]

topherinternational opened a new pull request, #43807:
URL: https://github.com/apache/airflow/pull/43807

   #43669 used the wrong keys for retrieving user and password from the boto3 redshift-serverless client response, leading to KeyError exceptions. This change uses the correct camel-cased `dbUser` and `dbPassword` keys; the boto3 redshift client response returns the Pascal-cased `DbUser` and `DbPassword` and these were mistakenly used in the serverless code block.
   
   related: #43669 
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix PostgresHook bug when getting AWS Redshift Serverless credentials [airflow]

Posted by "shahar1 (via GitHub)" <gi...@apache.org>.
shahar1 merged PR #43807:
URL: https://github.com/apache/airflow/pull/43807


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix PostgresHook bug when getting AWS Redshift Serverless credentials [airflow]

Posted by "topherinternational (via GitHub)" <gi...@apache.org>.
topherinternational commented on PR #43807:
URL: https://github.com/apache/airflow/pull/43807#issuecomment-2464650812

   > Nice catch!
   
   Thanks, the "catch" was when I shadowed this code into my org's Airflow instance and got the key error, so I found it the hard way 😅.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org