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

Updates Server endpoints to v4 #135

Merged
merged 13 commits into from
Feb 27, 2019
Merged

Updates Server endpoints to v4 #135

merged 13 commits into from
Feb 27, 2019

Conversation

icemann92
Copy link
Contributor

  • Updates SDK to use latest server v4 endpoints.
  • As part of v4 endpoints,
    • Adds support for aud to be an array
    • Updates iss validation (includes full path)
    • Adds validation for azp
    • token version in headers
  • Updated test cases

aal80 and others added 12 commits June 14, 2018 16:33
* notify oauth server on logout

* temporary server path

* temporary server path

* temporary server path

* restore server path

* restore server path

* add log messge

* change var names

* change message severity
Development->master, various fixes and new Tokyo region
@coveralls
Copy link

coveralls commented Feb 25, 2019

Coverage Status

Coverage decreased (-1.7%) to 74.286% when pulling 8d3fa18 on v4-iss2 into dde3e51 on development.

@@ -24,5 +24,4 @@
String getLocale();
Copy link
Contributor

Choose a reason for hiding this comment

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

Gender can be removed as well

return issuer.split("/")[2];
}

private static String suffixFromRegion(String region) {
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we will want to keep come sort of conversion from our old bluemix regions to new formats. One popular library we integrate with still uses the old branding

https://github.com/ibm-bluemix-mobile-services/bms-clientsdk-android-core/blob/master/lib/src/main/java/com/ibm/mobilefirstplatform/clientsdk/android/core/api/BMSClient.java

@ibm-cloud-security ibm-cloud-security deleted a comment Feb 26, 2019
@icemann92 icemann92 merged commit d028c5e into development Feb 27, 2019
gtaban added a commit that referenced this pull request Apr 5, 2019
* Update login widget section in readme (#133)

* Updates Server endpoints to v4 (#135)

* Changes SDK to use server v4 endpoints. Updates test cases

* 1. Adds version and AZP validation 2. Updates iss validation

* Issue 2439 (#138)

Removed azp check
rotembr added a commit that referenced this pull request Aug 21, 2019
* Update login widget section in readme (#133)

* Updates Server endpoints to v4 (#135)

* Changes SDK to use server v4 endpoints. Updates test cases

* 1. Adds version and AZP validation 2. Updates iss validation

* Issue 2439 (#138)

Removed azp check

* update version to 6 (#140)

* Logout wrapper (#144)

* added a wrapper logout function in AppID class which calls AppIDAuthorizationManager's logout

* updated readme with the new AppID's logout wrapper
rotembr added a commit that referenced this pull request Aug 22, 2019
* Update login widget section in readme (#133)

* Updates Server endpoints to v4 (#135)

* Changes SDK to use server v4 endpoints. Updates test cases

* 1. Adds version and AZP validation 2. Updates iss validation

* Issue 2439 (#138)

Removed azp check

* update version to 6 (#140)

* Logout wrapper (#144)

* added a wrapper logout function in AppID class which calls AppIDAuthorizationManager's logout

* updated readme with the new AppID's logout wrapper

* added slack link (#147)
rotembr added a commit that referenced this pull request Aug 25, 2019
* Update login widget section in readme (#133)

* Updates Server endpoints to v4 (#135)

* Changes SDK to use server v4 endpoints. Updates test cases

* 1. Adds version and AZP validation 2. Updates iss validation

* Issue 2439 (#138)

Removed azp check

* update version to 6 (#140)

* Logout wrapper (#144)

* added a wrapper logout function in AppID class which calls AppIDAuthorizationManager's logout

* updated readme with the new AppID's logout wrapper

* added slack link (#147)

* add some description to logout (#149)

* update (#151)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants