Skip to content

Commit

Permalink
[Identity] Hide dev credential timeout API for 1.8.0 release (#31788)
Browse files Browse the repository at this point in the history
* [Identity] Hide dev credential timeout API for 1.8.0 release

* update API spec

* update changelog for release
  • Loading branch information
schaabs authored Oct 13, 2022
1 parent 8b07647 commit acf795f
Show file tree
Hide file tree
Showing 6 changed files with 5 additions and 24 deletions.
17 changes: 1 addition & 16 deletions sdk/identity/Azure.Identity/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,29 +1,14 @@
# Release History

## 1.8.0-beta.1 (Unreleased)
## 1.8.0-beta.1 (2022-10-13)

### Features Added
- Credentials that are implemented via launching a sub-process to acquire tokens now have configurable timeouts. This addresses scenarios where these proceses can take longer than the current default timeout values. (A community contribution, courtesy of _[reynaldoburgos](https://github.com/reynaldoburgos)_). The affected credentials and their associated options are:
- `AzureCliCredential` and `AzureCliCredentialOptions.CliProcessTimeout`
- `AzurePowerShellCredential` and `AzurePowerShellCredentialOptions.PowerShellProcessTimeout`
- `VisualStudioCredential` and `VisualStudioCredentialOptions.VisualStudioProcessTimeout`
- `DefaultAzureCredential` and `DefaultAzureCredentialOptions.DeveloperCredentialTimeout` Note: this option applies to all developer credentials above when using `DefaultAzureCredential`.
- Reintroduced `ManagedIdentityCredential` token caching support from 1.7.0-beta.1
- `EnvironmentCredential` updated to support specifying a certificate password via the `AZURE_CLIENT_CERTIFICATE_PASSWORD` environment variable

### Breaking Changes
- Excluded `VisualStudioCodeCredential` from `DefaultAzureCredential` token chain by default as SDK authentication via Visual Studio Code is broken due to issue [#27263](https://github.com/Azure/azure-sdk-for-net/issues/27263). The `VisualStudioCodeCredential` will be re-enabled in the `DefaultAzureCredential` flow once a fix is in place. Issue [#30525](https://github.com/Azure/azure-sdk-for-net/issues/30525) tracks this. In the meantime Visual Studio Code users can authenticate their development environment using the [Azure CLI](https://learn.microsoft.com/cli/azure/).

### Bugs Fixed

### Other Changes

### Acknowledgments

Thank you to our developer community members who helped to make Azure Identity better with their contributions to this release:

- _[reynaldoburgos](https://github.com/reynaldoburgos)_

## 1.7.0 (2022-09-19)

### Features Added
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,6 @@ public partial class AzureCliCredentialOptions : Azure.Identity.TokenCredentialO
{
public AzureCliCredentialOptions() { }
public System.Collections.Generic.IList<string> AdditionallyAllowedTenants { get { throw null; } }
public System.TimeSpan? CliProcessTimeout { get { throw null; } set { } }
public string TenantId { get { throw null; } set { } }
}
public partial class AzurePowerShellCredential : Azure.Core.TokenCredential
Expand All @@ -74,7 +73,6 @@ public partial class AzurePowerShellCredentialOptions : Azure.Identity.TokenCred
{
public AzurePowerShellCredentialOptions() { }
public System.Collections.Generic.IList<string> AdditionallyAllowedTenants { get { throw null; } }
public System.TimeSpan? PowerShellProcessTimeout { get { throw null; } set { } }
public string TenantId { get { throw null; } set { } }
}
public partial class ChainedTokenCredential : Azure.Core.TokenCredential
Expand Down Expand Up @@ -149,7 +147,6 @@ public partial class DefaultAzureCredentialOptions : Azure.Identity.TokenCredent
{
public DefaultAzureCredentialOptions() { }
public System.Collections.Generic.IList<string> AdditionallyAllowedTenants { get { throw null; } }
public System.TimeSpan? DeveloperCredentialTimeout { get { throw null; } set { } }
public bool ExcludeAzureCliCredential { get { throw null; } set { } }
public bool ExcludeAzurePowerShellCredential { get { throw null; } set { } }
public bool ExcludeEnvironmentCredential { get { throw null; } set { } }
Expand Down Expand Up @@ -381,6 +378,5 @@ public partial class VisualStudioCredentialOptions : Azure.Identity.TokenCredent
public VisualStudioCredentialOptions() { }
public System.Collections.Generic.IList<string> AdditionallyAllowedTenants { get { throw null; } }
public string TenantId { get { throw null; } set { } }
public System.TimeSpan? VisualStudioProcessTimeout { get { throw null; } set { } }
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,6 @@ public class AzureCliCredentialOptions : TokenCredentialOptions
/// <summary>
/// The Cli process timeout.
/// </summary>
public TimeSpan? CliProcessTimeout { get; set; }
internal TimeSpan? CliProcessTimeout { get; set; }
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,6 @@ public class AzurePowerShellCredentialOptions : TokenCredentialOptions
/// <summary>
/// The Powershell process timeout.
/// </summary>
public TimeSpan? PowerShellProcessTimeout { get; set; } = TimeSpan.FromSeconds(10);
internal TimeSpan? PowerShellProcessTimeout { get; set; } = TimeSpan.FromSeconds(10);
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -198,7 +198,7 @@ public string VisualStudioCodeTenantId
/// <summary>
/// Specifies timeout for Developer credentials. e.g. Visual Studio, Azure CLI, Azure Powershell.
/// </summary>
public TimeSpan? DeveloperCredentialTimeout { get; set; } = TimeSpan.FromSeconds(30);
internal TimeSpan? DeveloperCredentialTimeout { get; set; } = TimeSpan.FromSeconds(30);

/// <summary>
/// Specifies whether the <see cref="EnvironmentCredential"/> will be excluded from the authentication flow. Setting to true disables reading
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,6 @@ public string TenantId
/// <summary>
/// The VisualStudio process timeout.
/// </summary>
public TimeSpan? VisualStudioProcessTimeout { get; set; }
internal TimeSpan? VisualStudioProcessTimeout { get; set; }
}
}

0 comments on commit acf795f

Please sign in to comment.