Andyohart/managed identity #532

4gust · 2024-11-08T11:43:31Z

No description provided.

* Readme suggestion update Co-authored-by: Bogdan Gavril <[email protected]>

https://github.com/AzureAD/microsoft-authentication-library-for-go into andyohart/managed-identity-basic-class-implementation

…-class-implementation Implement Initial API Classes

Added a simple version of getting token and printing it reformatting code.

Added tests and implementation for SAMI IMDS

Reverted changes in the test app

Formatting changes

Added method for UAMI

Updated the some code and cleaned up some comments and print statement

Updated the key for the resource Co-authored-by: Charles Lowell <[email protected]>

Updated the token from url function to a reaquest based function

…icrosoft-authentication-library-for-go into acquire-token-for-mise

Updated test to fail not only return error

Added tests for failure and success for SAMI

added context to request

Updated the tests to check for errors more correctly

Update some test and used mock.Client some refactoring for comments

Co-authored-by: Charles Lowell <[email protected]>

…icrosoft-authentication-library-for-go into acquire-token-for-mise

Added Region auto enable in confidential client and its test

Removed Integration tests from github actions

The field isn't used anywhere. Removing it simplifies the work on introducing authority abstraction. We might re-add it once we need it for anything.

…t readability Previously, the test results were printed as #0 #1 #2, which made it hard to navigate the failed test cases. Now, the test cases are annotated by their respective names. The names also make it more clear what is being unit-tested.

…ONCall Parsing the url, then setting qv.Encode() manually to u.RawQuery feel like an overcomplicated approach of creating url for http request. Use the native constructor, which enforces ctx to be passed into the request, and do simple sprintf for url + query.

This enables adding the same const for dSTS later in the process.

This allows reusing the function for dSTS flow.

sonarqubecloud · 2024-11-08T11:43:51Z

Quality Gate failed

Failed conditions
1 Security Hotspot

See analysis details on SonarCloud

AndyOHart and others added 30 commits August 21, 2024 08:30

Changes for running

995fa3c

* Adds .md file for managed identitys public api

0993a3f

Update docs/managedidentity_public_api.md

1053165

* Readme suggestion update Co-authored-by: Bogdan Gavril <[email protected]>

Makes DefaultToIMDS the default 0 value

aadf13d

Merge branch 'andyohart/managed-identity-basic-class-implementation' of

2fe36df

https://github.com/AzureAD/microsoft-authentication-library-for-go into andyohart/managed-identity-basic-class-implementation

Merge pull request #499 from AzureAD/andyohart/managed-identity-basic…

e82f85d

…-class-implementation Implement Initial API Classes

Initial system assgined for acquire token

1d2f3e8

Added a simple version of getting token.

63e6bed

Added a simple version of getting token and printing it reformatting code.

added IMDB for SAMI

69a039c

Added tests and implementation for SAMI IMDS

Reverted the test app to original state

7c94182

Reverted changes in the test app

Formatting changes

2646418

Formatting changes

Added methods for UAMI

4db1c7e

Added method for UAMI

Updated and cleaned up MI for SAMI

3bf0383

Updated the some code and cleaned up some comments and print statement

Update apps/managedidentity/managedidentity.go

8c3fed1

Updated the key for the resource Co-authored-by: Charles Lowell <[email protected]>

Resolved some comments.

5eb2919

Updated the token from url function to a reaquest based function

Merge branch 'acquire-token-for-mise' of https://github.com/AzureAD/m…

29583da

…icrosoft-authentication-library-for-go into acquire-token-for-mise

Updated test

64e4705

Updated test to fail not only return error

Updated the Identity method for feedback

a7e760a

Added tests for failure and success for SAMI

Passed context to http request

df2ad5a

added context to request

Updated service errors handling and tests

287963e

Updated the tests to check for errors more correctly

Updated tests to use mock

df9faf1

Update some test and used mock.Client some refactoring for comments

small update

5395b9a

Added a withStatusCode method in mock

6a72df2

Update apps/internal/mock/mock.go

b293a60

Co-authored-by: Charles Lowell <[email protected]>

Updated the method usage for WithHTTPStatusCode

c2b9127

Update apps/managedidentity/managedidentity_test.go

e451611

Co-authored-by: Charles Lowell <[email protected]>

Update apps/managedidentity/managedidentity_test.go

9912ee9

Co-authored-by: Charles Lowell <[email protected]>

Removed typed data from test

7f147d4

Merge branch 'acquire-token-for-mise' of https://github.com/AzureAD/m…

a2b0a2a

…icrosoft-authentication-library-for-go into acquire-token-for-mise

Updated test to return json error

82b1155

4gust and others added 28 commits November 8, 2024 11:40

testing the ado build on the personal branch

b5bff8e

updated the build path

b7f69c1

updating path

3f86c34

updating the cert creation script

c4e46cf

update path

3bf9f81

comment some tests

c10e633

uploading working tests

f15746b

Added README for running integration tests

e897a26

Skipping 2 tests

b060c7f

Wrap ResolveEndpoints error

f04d4b4

Update build_test.yaml for Azure Pipelines

ee88303

Added Region auto enable

68cc4fe

Added Region auto enable in confidential client and its test

Separated test

9d5e86c

Updated variableName

d88efcf

Update go.yml to remove Integration tests

9158019

Removed Integration tests from github actions

refactor: remove UserRealmURIPrefix from authority.Info

2adf0dd

The field isn't used anywhere. Removing it simplifies the work on introducing authority abstraction. We might re-add it once we need it for anything.

test(oauth): add WithTenant test-cases to increase method test coverage

0975c01

refactor(oauth): rename aad instance discovery endpoint const

80cb6c4

This enables adding the same const for dSTS later in the process.

refactor(oauth): make WithTenant extensible with authority types

e39ce1c

refactor(authority): use authority.ADFS instead of re-defined ADFS const

c717825

refactor(confidential): fakeClient accepts authority as param

a4fe72c

This allows reusing the function for dSTS flow.

feat(oauth): add support for dSTS authority type

17acbeb

Fix invalid authority uri

995c080

If authority segments <3, throw

46b3a48

Fix WithTenantID("adfs") regression (#529)

b54495a

Create release.md

9b700f7

4gust closed this Nov 8, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Andyohart/managed identity #532

Andyohart/managed identity #532

4gust commented Nov 8, 2024

sonarqubecloud bot commented Nov 8, 2024

Andyohart/managed identity #532

Andyohart/managed identity #532

Conversation

4gust commented Nov 8, 2024

sonarqubecloud bot commented Nov 8, 2024

Quality Gate failed