atharva-0026 opened a new pull request, #72981: URL: https://github.com/apache/airflow/pull/72981
Closes #68382 `Connection.port` had no validation at all, so you could set it to something like `999999` or `-5` and it would just silently save. Neither of those can ever be a real port, so this adds a check that raises a clear error instead of quietly storing garbage. Valid ports are 0-65535. Added a `@validates` hook on the port column that enforces that range, and still lets `port=None` through since not every connection needs one. Added tests for the obvious cases: too high, negative, a normal valid port, `None`, and the two boundary values (0 and 65535). Ran the full model test suite locally and nothing else broke. -- 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]
