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

Improve region support and add region telemetry #388

Merged
merged 5 commits into from
Jun 8, 2021

Conversation

Avery-Dunn
Copy link
Collaborator

@Avery-Dunn Avery-Dunn commented May 14, 2021

Improves support for Azure regions and updates telemetry to follow the most recent schema, as per this region design doc and this telemetry design doc.

Additionally, updates the oauth2-oidc-sdk dependency to fix an issue with the transitive json-smart dependency mentioned in #394

@Avery-Dunn Avery-Dunn requested review from SomkaPe and sangonzal May 14, 2021 21:03
Copy link
Contributor

@sangonzal sangonzal left a comment

Choose a reason for hiding this comment

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

Just a question about keeping CurrentRequest and LastRequest in sync. Otherwise looks good to me.

Copy link
Contributor

@sangonzal sangonzal 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 to me

…tion-library-for-java into avdunn/improved-region-support

� Conflicts:
�	src/main/java/com/microsoft/aad/msal4j/AadInstanceDiscoveryProvider.java
@Avery-Dunn Avery-Dunn requested a review from SomkaPe June 4, 2021 22:04
@Avery-Dunn Avery-Dunn merged commit 01bb679 into dev Jun 8, 2021
Avery-Dunn added a commit that referenced this pull request Jun 16, 2021
* Pesomka/b2b stage1 (#356)

multi cloud accounts

* Pesomka/key store provider (#358)

general purpose API should not be tied to specific security provider

* Instance aware support (#362)

 Instance aware support for interactive requests

* Update telemetry to reduce size of payload (#361)

* Update telemetry implementation

* Bump guava from 26.0-jre to 29.0-jre (#366)

Bumps [guava](https://github.com/google/guava) from 26.0-jre to 29.0-jre.

* Bump guava from 26.0-jre to 29.0-jre in /src/samples/msal-obo-sample (#367)

Bumps [guava](https://github.com/google/guava) from 26.0-jre to 29.0-jre.

* survey added to README.md

Loyalty developer survey added to README.md
Link placed in tab and new feedback section.

* Update oauth2-oidc-sdk dependency (#373)

Upgrade oauth2-oidc-sdk dependency

* Add default cache lookup to client credential flow (#368)

* Add default cache lookup to client credential flow

* Bump commons-io from 2.6 to 2.7 (#376)

Bumps commons-io from 2.6 to 2.7.

* Sagonzal/update client credentials (#377)

* Update obo flow to attempt cache lookup by default

* 1.10.0 release

* fix for issue with common scopes override (#385)

fix for issue with common scopes override

* Add ccs routing headers. Add option to pass in header map

* Rename IApiParameters to IAcquireTokenParameters

* PR Feedback

* Add CCS routing information to /authorize query parameters

* Update log levels for operations in OBO and ClientCredential supliers

* Update how non 200 http response are handled. Update to only throw MsalExceptions

* Update unit tests

* Retrigger build

* Update unit test

* Bump httpclient from 4.5.9 to 4.5.13 (#401)

Bumps httpclient from 4.5.9 to 4.5.13.

---
updated-dependencies:
- dependency-name: org.apache.httpcomponents:httpclient
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Improve region support and add region telemetry (#388)

* Improve region support and add region telemetry

* Finish implementation of most recent telemetry design

* Update oauth dependency

* Address code review comments

* integration test for cross cloud b2b (#396)

integration test for cross cloud b2b

* Version and changelog updates for release 1.10.1 (#405)

* Fix merge conflicts

Co-authored-by: SomkaPe <pesomka@microsoft.com>
Co-authored-by: Santiago Gonzalez <35743865+sangonzal@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: ShannonCanTech <shfost@microsoft.com>
Co-authored-by: Santiago Gonzalez <sagonzal@microsoft.com>
@Avery-Dunn Avery-Dunn deleted the avdunn/improved-region-support branch January 19, 2022 22:13
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.

3 participants