infohash commented on PR #37202:
URL: https://github.com/apache/airflow/pull/37202#issuecomment-1933613022

   No, they don't have to which is why backward compatibility is there.
   
   Until now, there was only one way:
   
   ```python
   $ export AIRFLOW_CONN_APPRISE_DEFAULT='{"extra": {"config": "{\"path\": 
\"https://hooks.slack.com/services/T1JJ3T3L2/A1BRTD4JD/TIiajkdnlazkcOXrIdevi7F\";,
 \"tags\": \"alert\"}"}}'
   ```
   
   But now, you can also set its env var like this:
   
   ```python
   $ export AIRFLOW_CONN_APPRISE_DEFAULT='{"extra": {"config": {"path": 
"https://hooks.slack.com/services/T1JJ3T3L2/A1BRTD4JD/TIiajkdnlazkcOXrIdevi7F";, 
"tags": "alert"}}}'
   ```
   
   It should have been like this from the start because 
`airfow.models.Connection` accepts JSON string in env var so there was never 
any need to escape nested JSON string.


-- 
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

Reply via email to