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

   ### Apache Airflow version
   
   main (development)
   
   ### If "Other Airflow 2 version" selected, which one?
   
   _No response_
   
   ### What happened?
   
   When creating or updating a connection via the rest api. We do not run any 
validation against `extra`. While from the UI we we do enforce that extra is 
json with 
   ```
   The Extra connection field contained an invalid value for Conn ID: test.
   
   If connection parameters need to be added to Extra, please make sure they 
are in the form of a single, valid JSON object.
   
   
   The following Extra parameters were not added to the connection:
   {invalid json}
   ```
   
   ### What you think should happen instead?
   
   We should apply the same validation in the REST API code and throw an error 
if `extra` is not valid json
   
   ### How to reproduce
   
   Make a connection with the REST API
   
   ### Operating System
   
   all
   
   ### Versions of Apache Airflow Providers
   
   _No response_
   
   ### Deployment
   
   Official Apache Airflow Helm Chart
   
   ### Deployment details
   
   _No response_
   
   ### Anything else?
   
   _No response_
   
   ### Are you willing to submit PR?
   
   - [ ] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of 
Conduct](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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to