Aws-sdk-js: ServiceConfigurationOptions.endpoint type should allow AWS.Endpoint

Created on 27 Aug 2019  路  6Comments  路  Source: aws/aws-sdk-js

Confirm by changing [ ] to [x] below to ensure that it's a bug:

Describe the bug
ServiceConfigurationOptions.endpoint is defined as type string, but the AWS.Endpoint documentation suggests an AWS.Endpoint object is valid as well:

var ep = new AWS.Endpoint('awsproxy.example.com');
var s3 = new AWS.S3({endpoint: ep});

SDK version number
v2.518.0

Expected behavior
Seems the type should be:

endpoint?: string | Endpoint;
closing-soon feature-request

Most helpful comment

I think this issue is still valid, though it has been a year and my PR was never reviewed.

All 6 comments

Hey @kyle-johnson,

Thank-you for reaching out, it can be marked as a feature request, you are welcomed to open a PR about this.

Hi @ajredniwja, I opened a PR for this here: #2923

Hey @esilkensen,

Thank-you for the PR, the team will review the PR and make a decision on merging it or not.

Greetings! We鈥檙e closing this issue because it has been open a long time and hasn鈥檛 been updated in a while and may not be getting the attention it deserves. We encourage you to check if this is still an issue in the latest release and if you find that this is still a problem, please feel free to comment or open a new issue.

I think this issue is still valid, though it has been a year and my PR was never reviewed.

Apologies for the delay, I merged the PR @esilkensen.

Was this page helpful?
0 / 5 - 0 ratings