Skip to content

Update Vault URL - #2678

Open
Danilo Raspa (nilo-ms) wants to merge 4 commits into
devfrom
danilo/update-vault-url
Open

Update Vault URL#2678
Danilo Raspa (nilo-ms) wants to merge 4 commits into
devfrom
danilo/update-vault-url

Conversation

@nilo-ms

Copy link
Copy Markdown
Contributor

Update keyvault URL. Use newly created vault

Copilot AI review requested due to automatic review settings June 18, 2025 15:39
@nilo-ms
Danilo Raspa (nilo-ms) requested a review from a team as a code owner June 18, 2025 15:39
@github-actions

Copy link
Copy Markdown

❌ Work item link check failed. Description does not contain AB#{ID}.

Click here to Learn more.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the build automation vault endpoint to point at the newly created mobile build vault.

  • Changed the BUILD_AUTOMATION_VAULT_BASE_PATH constant to the new URL.
Comments suppressed due to low confidence (1)

keyvault/src/main/java/com/microsoft/identity/internal/test/keyvault/ApiClient.java:76

  • [nitpick] The constant name BUILD_AUTOMATION_VAULT_BASE_PATH no longer reflects the new 'mobilebuildvault' URL. Consider renaming it to something like MOBILE_BUILD_VAULT_BASE_PATH or MOBILE_BUILD_AUTOMATION_VAULT_BASE_PATH for clarity.
    public static final String BUILD_AUTOMATION_VAULT_BASE_PATH = "https://mobilebuildvault.vault.azure.net";

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.

4 participants