From 3e207c1c798122edfb821a51360aebf0b0bf6602 Mon Sep 17 00:00:00 2001 From: Vipul Kala Date: Wed, 29 Jul 2026 17:40:25 +0530 Subject: [PATCH] feat(amazon-photos): Update OAuth scopes to amazonphotos namespace --- .../auth/amazon/AmazonOAuthConfig.java | 24 ++++++++----------- .../auth/amazon/AmazonOAuthConfigTest.java | 23 +++++++----------- 2 files changed, 18 insertions(+), 29 deletions(-) diff --git a/extensions/auth/portability-auth-amazon/src/main/java/org/datatransferproject/auth/amazon/AmazonOAuthConfig.java b/extensions/auth/portability-auth-amazon/src/main/java/org/datatransferproject/auth/amazon/AmazonOAuthConfig.java index b64ea848b..79f5954b4 100644 --- a/extensions/auth/portability-auth-amazon/src/main/java/org/datatransferproject/auth/amazon/AmazonOAuthConfig.java +++ b/extensions/auth/portability-auth-amazon/src/main/java/org/datatransferproject/auth/amazon/AmazonOAuthConfig.java @@ -33,23 +33,19 @@ public String getTokenUrl() { @Override public Map> getImportScopes() { - return ImmutableMap.>builder() - .put(PHOTOS, ImmutableSet.of( - "photos::images:create", - "photos::albums:create", - "photos::albums:update")) - .put(VIDEOS, ImmutableSet.of( - "photos::videos:create", - "photos::albums:create", - "photos::albums:update")) - .build(); + return ImmutableMap.of( + PHOTOS, ImmutableSet.of( + "amazonphotos::images:create", + "amazonphotos::albums:create", + "amazonphotos::albums:update"), + VIDEOS, ImmutableSet.of( + "amazonphotos::videos:create", + "amazonphotos::albums:create", + "amazonphotos::albums:update")); } @Override public Map> getExportScopes() { - return ImmutableMap.>builder() - .put(PHOTOS, ImmutableSet.of("photos::images:read", "photos::albums:read")) - .put(VIDEOS, ImmutableSet.of("photos::videos:read", "photos::albums:read")) - .build(); + return ImmutableMap.of(); } } diff --git a/extensions/auth/portability-auth-amazon/src/test/java/org/datatransferproject/auth/amazon/AmazonOAuthConfigTest.java b/extensions/auth/portability-auth-amazon/src/test/java/org/datatransferproject/auth/amazon/AmazonOAuthConfigTest.java index b19c8f080..745482d3b 100644 --- a/extensions/auth/portability-auth-amazon/src/test/java/org/datatransferproject/auth/amazon/AmazonOAuthConfigTest.java +++ b/extensions/auth/portability-auth-amazon/src/test/java/org/datatransferproject/auth/amazon/AmazonOAuthConfigTest.java @@ -35,29 +35,22 @@ void tokenUrlIsLwaEndpoint() { void importScopesForPhotos() { assertThat(config.getImportScopes().get(PHOTOS)) .containsExactly( - "photos::images:create", - "photos::albums:create", - "photos::albums:update"); + "amazonphotos::images:create", + "amazonphotos::albums:create", + "amazonphotos::albums:update"); } @Test void importScopesForVideos() { assertThat(config.getImportScopes().get(VIDEOS)) .containsExactly( - "photos::videos:create", - "photos::albums:create", - "photos::albums:update"); + "amazonphotos::videos:create", + "amazonphotos::albums:create", + "amazonphotos::albums:update"); } @Test - void exportScopesForPhotos() { - assertThat(config.getExportScopes().get(PHOTOS)) - .containsExactly("photos::images:read", "photos::albums:read"); - } - - @Test - void exportScopesForVideos() { - assertThat(config.getExportScopes().get(VIDEOS)) - .containsExactly("photos::videos:read", "photos::albums:read"); + void exportScopesEmpty() { + assertThat(config.getExportScopes()).isEmpty(); } }