#10007 refactor: Decouple Hub error handling from Requests
Pull request opened by
edgarrmondragon
Description
Checklist
• I have read the
contribution guide
Was generative AI tooling used to co-author this PR?
• Yes (please specify the tool below)
Related Issues
• Closes #XXXX
Summary by Sourcery
Refine Meltano Hub client error handling to use domain-specific exceptions and simplify HTTP response processing.
Enhancements:
• Update Hub error types to subclass MeltanoError and use clearer, centralized error messages for plugin type, variant, and connection failures.
• Simplify Hub client HTTP status handling by mapping 4xx responses to plugin type errors and 5xx responses to connection errors without relying on requests exceptions.
• Tighten internal hub client code with minor typing and style adjustments, including improved header and URL handling.
Build:
• Remove the types-requests typing dependency from the project configuration.
Tests:
• Adjust Hub service tests and test HTTP client fixture to validate the new error messages and 5xx response handling behavior.
meltano/meltano