Skip to content

Commit 1eb232b

Browse files
committed
fixed s3 validation on account update
1 parent e16c526 commit 1eb232b

File tree

3 files changed

+8
-5
lines changed

3 files changed

+8
-5
lines changed

addon_imps/storage/s3.py

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -130,3 +130,10 @@ def list_buckets(self):
130130
item_name=bucket["Name"] + "/",
131131
item_type=storage.ItemType.FOLDER,
132132
)
133+
134+
async def get_external_account_id(self, auth_result_extras: dict[str, str]) -> str:
135+
try:
136+
self.client.list_buckets()
137+
return ""
138+
except BotoExceptions.ClientError:
139+
raise ValidationError("Fail to validate access key and secret key")

addon_service/external_service/models.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ def service_type(self):
8686
return ServiceTypes(self.int_service_type)
8787

8888
@property
89-
def configurable_api_root(self):
89+
def configurable_api_root(self) -> bool:
9090
return ServiceTypes.HOSTED in self.service_type
9191

9292
@property

addon_service/external_service/storage/views.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,10 +12,6 @@
1212
list=extend_schema(
1313
description="Get the list of all available external storage services"
1414
),
15-
get=extend_schema(
16-
description="Get particular external storage service",
17-
),
18-
get_wb_credentials=extend_schema(exclude=True),
1915
)
2016
class ExternalStorageServiceViewSet(ReadOnlyModelViewSet):
2117
queryset = ExternalStorageService.objects.all().select_related(

0 commit comments

Comments
 (0)