shashbha14 commented on PR #66593:
URL: https://github.com/apache/airflow/pull/66593#issuecomment-4408510904
> I'd like to reproduce this issue locally. Do you happen to have a screen
recording showing the behavior before and after the fix?
I don't have a screen recording, but here are the steps to reproduce locally:
1. Directly insert a connection with invalid JSON in the extra field via the
DB:
UPDATE connection SET extra = 'invalid json' WHERE conn_id = 'your_conn';
2. Open the Airflow UI → Admin → Connections
3. Click the Edit button on that connection
4. Without the fix: the form fails to render
5. With the fix: the form opens and shows the raw string
The root cause is visible in the code — JSON.parse on line 85 has no
try-catch, while the same file already handles this at lines 103-110.
--
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]
