airbytehq / airbytehq/PyAirbyte

Evaluate adding workspace validation to CloudSource and CloudDestination

未關閉
#930 0 則留言 0 個 reaction 已指派 0 人 在 GitHub 檢視
主要語言
Python
星號
344
分支
77
平均合併
1 天 11 小時
30 天內合併 PR
35

描述

## Summary

Following the addition of workspace validation to `CloudConnection` in https://github.com/airbytehq/PyAirbyte/pull/929, we should evaluate whether similar validation should be added to `CloudSource` and `CloudDestination` classes.

## Background

PR #929 added:
- `check_is_valid()` method to `CloudConnection` to verify the connection exists and belongs to the expected workspace
- `AirbyteWorkspaceMismatchError` exception (generic, designed to work with connections, sources, and destinations)
- Updated `_fetch_connection_info()` with `force_refresh` and `verify` parameters

The exception was intentionally designed to be generic (with `resource_type` and `resource_id` fields) to support future use with sources and destinations.

## Proposed Work

1. Evaluate if `CloudSource` and `CloudDestination` classes would benefit from similar `check_is_valid()` methods
2. If so, implement workspace validation using the existing `AirbyteWorkspaceMismatchError` exception
3. Update `_fetch_connector_info()` methods with similar `force_refresh` and `verify` parameters

## Timeline

Evaluate after ~1 month of usage to gather feedback on the `CloudConnection` implementation.

---
Requested by @aaronsteers

貢獻指南

開啟貢獻指南

評估

這個 Issue 還沒有評估資料。

把新 issue 寄到你的電子郵件信箱

精選適合新手參與的 GitHub issue 摘要。