potiuk commented on PR #32867: URL: https://github.com/apache/airflow/pull/32867#issuecomment-1892849628
> The issue was with the description text shown in the error message, not the regex itself - so in theory can I can use the old one but then perhaps the error message gets rather long due to not being able to show the concise message using special characters... What do you think @potiuk? Well ... in this case... we can also just escape AND shorten the connection id. It's not THAT important to show connection_id fully - we know what connection id we were just updating so showing all of it in the error message is probably good enough. I have a feeling (now after looking at it and reallising that it's just error message) that sanitizing the id at entry in this case is far too invasive. -- 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]
