Skip to content
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

feat: allow IAM clientid/secret to be configured #14

Merged
merged 1 commit into from
Apr 19, 2019
Merged

Conversation

padamstx
Copy link
Member

Fixes: https://github.ibm.com/arf/planning-sdk-squad/issues/851

This PR includes these changes:

  1. allow the clientid/secret values to be set by the user and then used during interactions with the IAM token server. These values can be set in the following ways:
  • via the Options object passed to the IamTokenManagerV1 ctor
  • by calling the new IamTokenManager.setIamAuthorizationInfo() function
  1. the default IAM token server URL was changed from "iam.bluemix.net" to "iam.cloud.ibm.com"

@padamstx padamstx requested review from dpopp07 and esminerva April 18, 2019 21:20
@padamstx padamstx self-assigned this Apr 18, 2019
@codecov
Copy link

codecov bot commented Apr 18, 2019

Codecov Report

Merging #14 into master will increase coverage by 0.19%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #14      +/-   ##
==========================================
+ Coverage   95.38%   95.57%   +0.19%     
==========================================
  Files           7        7              
  Lines         325      339      +14     
  Branches       54       55       +1     
==========================================
+ Hits          310      324      +14     
  Misses         13       13              
  Partials        2        2
Impacted Files Coverage Δ
iam-token-manager/v1.ts 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f7e65d8...c6c8be4. Read the comment docs.

Copy link
Member

@dpopp07 dpopp07 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes looks great and the tests are very thorough, so thanks!

I hate to do this to you, but there are a couple style issues that I requested changes on. Want to keep all the code consistent 😅 I guess we're not running the linter in Travis, I'll make that change later

iam-token-manager/v1.ts Outdated Show resolved Hide resolved
iam-token-manager/v1.ts Show resolved Hide resolved
iam-token-manager/v1.ts Outdated Show resolved Hide resolved
Copy link
Member

@dpopp07 dpopp07 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! 👍

@dpopp07 dpopp07 merged commit ff8f2e7 into master Apr 19, 2019
@dpopp07 dpopp07 deleted the issue-851 branch April 19, 2019 16:13
dpopp07 pushed a commit that referenced this pull request Apr 19, 2019
# [0.2.0](v0.1.2...v0.2.0) (2019-04-19)

### Features

* allow IAM clientid/secret to be configured ([#14](#14)) ([ff8f2e7](ff8f2e7))
@dpopp07
Copy link
Member

dpopp07 commented Apr 19, 2019

🎉 This PR is included in version 0.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

JurajNyiri pushed a commit to JurajNyiri/node-sdk-core that referenced this pull request Aug 22, 2024
Generated SDK source code using:
- Generator version 3.12.2
- Specification version 0.0.26
- Automation (cloudant-sdks) version 536c0f1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants