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

   ### Description
   
   I bumped into a bug where a customer suffixed a connection id with a 
whitespace e.g. "myconn ". When referencing the connection id "myconn" (without 
whitespace), you get a connection not found error.
   
   I avoid such human errors, I suggest restricting the characters allowed for 
connection ids.
   
   Some suggestions:
   - There's an `airflow.utils.helpers.validate_key` function for validating 
the DAG id. Probably a good idea to reuse this.
   - I believe variable ids are also not validated, would be good to check 
those too.
   
   ### Use case/motivation
   
   _No response_
   
   ### Related issues
   
   _No response_
   
   ### Are you willing to submit a PR?
   
   - [X] 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