-
Notifications
You must be signed in to change notification settings - Fork 910
Expose the metadata key on client interfaces #2327
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
ac9e0c0
to
92fc8d3
Compare
"globalEndpoint": "json-service.amazonaws.com", | ||
"protocol": "rest-json", | ||
"serviceAbbreviation": "Json Service", | ||
"serviceFullName": "Some Service That Uses Json Protocol", | ||
"serviceId":"Json Service", | ||
"serviceId": "Json Service", | ||
"signingName": "json-service", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added so it tests when singing name != endpoint prefix
92fc8d3
to
6fd77f9
Compare
Digging into this more it seems there is a
on the sync interface (it is missing from the async interface). This is basically what I needed the ID for in the first place, but on order to make what I need work (dynamic service info in the IDEs), I basically need to find a way to update
Is there any thoughts on how to make the The endpoints.json parser currently lives in The other more work for us approach would be to roll our own ServiceMetadata system, but I would like to leverage the endpoints.json loader given how complicated the resolve logic is. In that case, I would still need the ID to be exposed. |
6fd77f9
to
43f5366
Compare
Sorry for delay. This pr should be good to go once we get another review from the team, as this is a change to the public apis. About the extra feature request mentioned in the comment, we can implement a new utillity method to read the service metadata directly from the json file so you can get "dynamic service info". We've created a new feature request task in our backlog, will implement it once it's prioritized. |
Kudos, SonarCloud Quality Gate passed! |
@all-contributors please add @abrooksv for code |
I've put up a pull request to add @abrooksv! 🎉 |
…09ffbb424 Pull request: release <- staging/e77e56f0-b289-43b0-8a65-92c09ffbb424
Description
Add a new static field on the client interfaces with metadata ID (aka API endpoint prefix)
Motivation and Context
#2301
Testing
Screenshots (if appropriate)
Types of changes
Checklist
mvn install
succeedsLicense