kinesisvideoarchivedmedia.ListFragments is missing a paginator

[keep-cloud-tr]

Documentation

• I've gone though the API reference
• I've checked AWS Forums and StackOverflow for answers
• I've searched for previous similar issues and didn't find any solution

Describe the bug

There is no paginator for kinesisvideoarchivedmedia ListFragments. Fragments can easily exceed the default page size, so this is needed.

It is present in the v1 sdk: https://github.com/aws/aws-sdk-go/blob/895cfb31afb235ab5461432d181680152cafe67d/service/kinesisvideoarchivedmedia/api.go#L934

It was also present in the v2 sdk at one time, but was removed for some reason:

aws-sdk-go-v2/service/kinesisvideoarchivedmedia/api_op_ListFragments.go

Line 186 in 5448b45

func NewListFragmentsPaginator(client ListFragmentsAPIClient, params *ListFragmentsInput, optFns ...func(*ListFragmentsPaginatorOptions)) *ListFragmentsPaginator {

Similar to: aws/aws-sdk#257

Expected behavior

kinesisvideoarchivedmedia.NewListFragmentsPaginator to be defined

Current behavior

kinesisvideoarchivedmedia.NewListFragmentsPaginator is not defined

Steps to Reproduce

go doc github.com/aws/aws-sdk-go-v2/service/kinesisvideoarchivedmedia | grep -i fragment | grep -i pagi

Possible Solution

Re-implement kinesisvideoarchivedmedia.NewListFragmentsPaginator.

AWS Go SDK version used

github.com/aws/aws-sdk-go-v2 v1.13.0

Compiler and Version used

go version go1.17.6 linux/amd64

Operating System and version

Linux

[keep-cloud-tr]

Would it be sufficient to just add a paginated trait to the smithy model?

diff --git a/codegen/sdk-codegen/aws-models/kinesis-video-archived-media.json b/codegen/sdk-codegen/aws-models/kinesis-video-archived-media.json
index 7c144a7b8a..5c663e41a2 100644
--- a/codegen/sdk-codegen/aws-models/kinesis-video-archived-media.json
+++ b/codegen/sdk-codegen/aws-models/kinesis-video-archived-media.json
@@ -1241,6 +1241,12 @@
                     "method": "POST",
                     "uri": "/listFragments",
                     "code": 200
+                },
+                "smithy.api#paginated": {
+                    "inputToken": "NextToken",
+                    "outputToken": "NextToken",
+                    "items": "FragmentList",
+                    "pageSize": "MaxResults"
                 }
             }
         },

[lucix-aws]

The described paginator is present today, I'm assuming this was fixed upstream in the model. Closing.

[github-actions]

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.