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

Native Auth SignIn, Signup and SSPR features #1947

Merged
merged 14 commits into from
Dec 6, 2023
Merged

Conversation

SaurabhMSFT
Copy link
Contributor

@SaurabhMSFT SaurabhMSFT commented Nov 9, 2023

This PR is to merge the SignIn, SIgnup and SSPR code into dev branch. This code was previously reviewed in #1897, #1919, #1949 and #1950 .

A technical overview of Native Auth is available at https://dev.azure.com/IdentityDivision/DevEx/_git/AuthLibrariesApiReview?path=/%5BAndroid%5D%20Native%20authentication/technical_overview.md&version=GBsammy/native-authentication&_a=preview

A companion PR in common repo is AzureAD/microsoft-authentication-library-common-for-android#2233

SaurabhMSFT and others added 6 commits October 3, 2023 19:35
The code in this pull request is for the base work needed to support Signup, SignIn and SSPR features for Native Auth. Any class that is used by at least two of the features has been included in this pull request.

This pull request introduces kotlin co routines libraries as dependencies for msal and its associated test project. Further mockk, mockito libraries have been added as dependencies for test projects.

A companion PR in the MSAL Common repository is
AzureAD/microsoft-authentication-library-common-for-android#2166
ciam-feature will be update to 82f5153 commit of dev branch

---------

Co-authored-by: melissaahn <[email protected]>
Co-authored-by: iamgusain <[email protected]>
Co-authored-by: Dome Pongmongkol <[email protected]>
Co-authored-by: Sowmya Malayanur <[email protected]>
ciam-feature is updated to 991c541
commit from dev branch

---------

Co-authored-by: melissaahn <[email protected]>
Co-authored-by: iamgusain <[email protected]>
Co-authored-by: Dome Pongmongkol <[email protected]>
Co-authored-by: Sowmya Malayanur <[email protected]>
Co-authored-by: Mohit <[email protected]>
Co-authored-by: tanmaymanolkar1 <[email protected]>
Co-authored-by: amoyixi <[email protected]>
Co-authored-by: yixilin <[email protected]>
Co-authored-by: fadidurah <[email protected]>
Co-authored-by: Dean Xu <[email protected]>
This PR includes the work for SignIn relates features of Native Auth

Companion PR in MSAL Common repo is
AzureAD/microsoft-authentication-library-common-for-android#2207
SaurabhMSFT and others added 5 commits November 9, 2023 23:49
This PR includes the work for Signup related features of Native Auth. No
new dependencies have been added as part of this PR.

A technical overview of Native Auth is available at
https://dev.azure.com/IdentityDivision/DevEx/_git/AuthLibrariesApiReview?path=/%5BAndroid%5D%20Native%20authentication/technical_overview.md&version=GBsammy/native-authentication&_a=preview

A companion PR in MSAL Common is
AzureAD/microsoft-authentication-library-common-for-android#2229
ciam-feature is updated to 9735ed9
commit from dev branch

---------

Co-authored-by: melissaahn <[email protected]>
Co-authored-by: iamgusain <[email protected]>
Co-authored-by: Dome Pongmongkol <[email protected]>
Co-authored-by: Sowmya Malayanur <[email protected]>
Co-authored-by: Mohit <[email protected]>
Co-authored-by: tanmaymanolkar1 <[email protected]>
Co-authored-by: amoyixi <[email protected]>
Co-authored-by: yixilin <[email protected]>
Co-authored-by: fadidurah <[email protected]>
Co-authored-by: Dean Xu <[email protected]>
Co-authored-by: Neha Goel <[email protected]>
Co-authored-by: Dome Pongmongkol <[email protected]>
Co-authored-by: pedro romero vargas <[email protected]>
This PR includes the work for self service password reset related
features of Native Auth. No new dependencies have been added as part of
this PR.

A technical overview of Native Auth is available at
https://dev.azure.com/IdentityDivision/DevEx/_git/AuthLibrariesApiReview?path=/%5BAndroid%5D%20Native%20authentication/technical_overview.md&version=GBsammy/native-authentication&_a=preview

A companion PR in MSAL is
AzureAD/microsoft-authentication-library-common-for-android#2230
@SaurabhMSFT SaurabhMSFT marked this pull request as ready for review December 1, 2023 16:19
@SaurabhMSFT SaurabhMSFT requested a review from a team as a code owner December 1, 2023 16:19
@SaurabhMSFT SaurabhMSFT changed the title Native Auth SignIn and other common features Native Auth SignIn, Signup and SSPR features Dec 1, 2023
changelog Outdated Show resolved Hide resolved
Copy link
Contributor

@fadidurah fadidurah left a comment

Choose a reason for hiding this comment

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

LGTM, looks like you need a dev update though

SaurabhMSFT added a commit to AzureAD/microsoft-authentication-library-common-for-android that referenced this pull request Dec 6, 2023
This PR is to merge the SignIn, Signup and other common code into dev
branch. This code was previously reviewed in #2166, #2207 and #2229

A technical overview of Native Auth is available at
https://dev.azure.com/IdentityDivision/DevEx/_git/AuthLibrariesApiReview?path=/%5BAndroid%5D%20Native%20authentication/technical_overview.md&version=GBsammy/native-authentication&_a=preview

A companion PR in common repo is
AzureAD/microsoft-authentication-library-for-android#1947

---------

Co-authored-by: pedro romero vargas <[email protected]>
Co-authored-by: melissaahn <[email protected]>
Co-authored-by: fadidurah <[email protected]>
Co-authored-by: Dome Pongmongkol <[email protected]>
Co-authored-by: iamgusain <[email protected]>
Co-authored-by: DongXu <[email protected]>
Co-authored-by: Sowmya Malayanur <[email protected]>
Co-authored-by: Mohit <[email protected]>
Co-authored-by: Sandesh Kumar Srivastava <[email protected]>
Co-authored-by: tanmaymanolkar1 <[email protected]>
Co-authored-by: amoyixi <[email protected]>
Co-authored-by: Yixi Lin <[email protected]>
Co-authored-by: Shahzaib <[email protected]>
@SaurabhMSFT SaurabhMSFT merged commit b2e9fb0 into dev Dec 6, 2023
9 checks passed
@SaurabhMSFT SaurabhMSFT deleted the saugautam/merge-cf branch December 6, 2023 20:36
iamgusain added a commit that referenced this pull request Jan 3, 2024
#1947 added a duplicate dependency for testUtils project. This causes
failure to run UTs in monthly pipeline, [Example
run](https://identitydivision.visualstudio.com/Engineering/_build/results?buildId=1229805&view=logs&j=f84464c4-dbde-50dc-0e4a-70f8148db754&t=f5fe7ea9-b05a-52f9-d231-7977d552c77d)

Removing the duplicate dependency in this change, as testUtils
dependency is already added for local and dist variant.
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.

3 participants