Re: [I] Connection to Postgres failed due to special characters in a password [airflow]

2024-05-09 Thread via GitHub


Taragolis closed issue #38187: Connection to Postgres failed due to special 
characters in a password
URL: https://github.com/apache/airflow/issues/38187


-- 
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-unsubscr...@airflow.apache.org

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



Re: [I] Connection to Postgres failed due to special characters in a password [airflow]

2024-04-14 Thread via GitHub


rawwar commented on issue #38187:
URL: https://github.com/apache/airflow/issues/38187#issuecomment-2054020442

   PR to fix this issue: Here's the PR: 
https://github.com/apache/airflow/pull/38831
   @DEworkerDE , I've already tested the fix and it's working. Can you please 
test it as well?
   
   


-- 
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-unsubscr...@airflow.apache.org

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



Re: [I] Connection to Postgres failed due to special characters in a password [airflow]

2024-04-08 Thread via GitHub


rawwar commented on issue #38187:
URL: https://github.com/apache/airflow/issues/38187#issuecomment-2043038768

   @Taragolis , based on your comment 
[here](https://github.com/apache/airflow/issues/38195#issuecomment-2042644747)
   
   > As far as i remember, we recently discuss this case in slack. I guess 
better introduce sa_uri property (or some similar naming) which returns 
[sqlalchemy.engine.URL](https://docs.sqlalchemy.org/en/14/core/engines.html#sqlalchemy.engine.URL)
 into the DbApiHook without any implementation, e.g. raise NotImplementedError 
and implements it per hook which have implementation for SQAlchemy. 
   
   I guess we leave the implementation of `get_uri` dependent on the provider 
type. If so, do we really need to have `sa_uri` property with 
`NotImplementedError`? Since SqlAlchemy URI is pre-defined, we can consider 
using the connection and try to form the URI. Raise an error if it has issues. 
What do you think?


-- 
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-unsubscr...@airflow.apache.org

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



Re: [I] Connection to Postgres failed due to special characters in a password [airflow]

2024-03-15 Thread via GitHub


Taragolis commented on issue #38187:
URL: https://github.com/apache/airflow/issues/38187#issuecomment-2000492104

   That is not only one, let me create generic task for all known hooks which 
incorrectly build SA URI 


-- 
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-unsubscr...@airflow.apache.org

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



Re: [I] Connection to Postgres failed due to special characters in a password [airflow]

2024-03-15 Thread via GitHub


Taragolis commented on issue #38187:
URL: https://github.com/apache/airflow/issues/38187#issuecomment-2000486988

   Ah.. that one happen because PostgresHook build incorrect SA URI, it just 
tried to convert Airflow URI to SA URI


-- 
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-unsubscr...@airflow.apache.org

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



Re: [I] Connection to Postgres failed due to special characters in a password [airflow]

2024-03-15 Thread via GitHub


hussein-awala commented on issue #38187:
URL: https://github.com/apache/airflow/issues/38187#issuecomment-2000365821

   What do you provide as an extra dictionary for this connection? (you can 
hide the values if there is something confidential)


-- 
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-unsubscr...@airflow.apache.org

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



Re: [I] Connection to Postgres failed due to special characters in a password [airflow]

2024-03-15 Thread via GitHub


DEworkerDE commented on issue #38187:
URL: https://github.com/apache/airflow/issues/38187#issuecomment-223105

   I think it it because of special characters in at the start of the password 
(i.e. first char is special char)


-- 
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-unsubscr...@airflow.apache.org

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



[I] Connection to Postgres failed due to special characters in a password [airflow]

2024-03-15 Thread via GitHub


DEworkerDE opened a new issue, #38187:
URL: https://github.com/apache/airflow/issues/38187

   ### Apache Airflow version
   
   Other Airflow 2 version (please specify below)
   
   ### If "Other Airflow 2 version" selected, which one?
   
   2.7.1
   
   ### What happened?
   
   "Airflow UI test connection" shows that it was successful, but when task 
uses PostgresHook I got connection issue (db name= TEST_ETL for example).
   But airflow db which we use for airflow service works fine, i.e. we are 
using the same server, but different db.
   
   in UI:
   
![image](https://github.com/apache/airflow/assets/126095189/f7cce69f-24c4-4d16-9499-489e92944fc9)
   
   in task then we got error:
   sqlalchemy.exc.ProgrammingError: (psycopg2.ProgrammingError) invalid dsn: 
invalid connection option "__extra__"
   (Background on this error at: https://sqlalche.me/e/14/f405)
   [2024-03-15, 15:15:08 UTC] {taskinstance.py:1398} INFO - Marking task as 
FAILED. dag_id=data_ingestion, task_id=set_static_data, 
execution_date=20240315T151344, start_date=20240315T151431, 
end_date=20240315T151508
   [2024-03-15, 15:15:08 UTC] {standard_task_runner.py:104} ERROR - Failed to 
execute job 10 for task set_static_data ((psycopg2.ProgrammingError) invalid 
dsn: invalid connection option "__extra__"
   (Background on this error at: https://sqlalche.me/e/14/f405); 123)
   [2024-03-15, 15:15:08 UTC] {local_task_job_runner.py:228} INFO - Task exited 
with return code 1
   [2024-03-15, 15:15:08 UTC] {taskinstance.py:2776} INFO - 0 downstream tasks 
scheduled from follow-on schedule check
   
   
   passwords which were 100% invalid:
   1) "+&I>[A1FEyxze-P3"
   2) "!&*Rr1(-_=[{[A1FEyxze-P3 
   !&*Rr1(-_=[{https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
   


-- 
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-unsubscr...@airflow.apache.org.apache.org

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