New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
馃悰 Source Jira - better handling of invalid domain in check #31896
Conversation
The latest updates on your projects. Learn more about Vercel for Git 鈫楋笌 1 Ignored Deployment
|
Current dependencies on/for this PR: This comment was auto-generated by Graphite. |
Before Merging a Connector Pull RequestWow! What a great pull request you have here! 馃帀 To merge this PR, ensure the following has been done/considered for each connector added or updated:
If the checklist is complete, but the CI check is failing,
|
What
We've gotten paged a couple times due to an unhandled exception raised when a user inputs an invalid domain while configuring Jira. Specifically, this happens when the user inputs the domain with protocol (e.g.
https://airbyte.atlassian.net
rather than the expectedairbyte.atlassian.net
).Previously invalid domains were attempted to be handled as a config error, but there's an edge case in there with how invalid urls are handled and raised.
(see sentry issue)
How
Fix by checking if response is None before accessing any fields in it. Also, handle the InvalidURL error so it raises the same config error. The messging for the error has also been tweaked to hint at protocol inclusion.
Then, updated any tests as needed.