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

Redact sensitive info #25159

Conversation

jlichwa
Copy link
Contributor

@jlichwa jlichwa commented Aug 3, 2023

Data Plane API - Pull Request

API Info: The Basics

Most of the information about your service should be captured in the issue that serves as your API Spec engagement record.

  • Link to API Spec engagement record issue:

Is this review for (select one):

  • a private preview
  • a public preview
  • GA release

Change Scope

This section will help us focus on the specific parts of your API that are new or have been modified.
Please share a link to the design document for the new APIs, a link to the previous API Spec document (if applicable), and the root paths that have been updated.

  • Design Document:
  • Previous API Spec Doc:
  • Updated paths:

❔Got questions? Need additional info?? We are here to help!

Contact us!

The Azure API Review Board is dedicated to helping you create amazing APIs. You can read about our mission and learn more about our process on our wiki.

Click here for links to tools, specs, guidelines & other good stuff

Tooling

Guidelines & Specifications

Helpful Links

@jlichwa jlichwa requested a review from heaths as a code owner August 3, 2023 21:43
@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Aug 3, 2023

Swagger Validation Report

️️✔️BreakingChange succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️Breaking Change(Cross-Version) succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️LintDiff succeeded [Detail] [Expand]
Validation passes for LintDiff.
️️✔️Avocado succeeded [Detail] [Expand]
Validation passes for Avocado.
️️✔️SwaggerAPIView succeeded [Detail] [Expand]
️️✔️TypeSpecAPIView succeeded [Detail] [Expand]
️️✔️ModelValidation succeeded [Detail] [Expand]
Validation passes for ModelValidation.
️️✔️SemanticValidation succeeded [Detail] [Expand]
Validation passes for SemanticValidation.
️️✔️PrettierCheck succeeded [Detail] [Expand]
Validation passes for PrettierCheck.
️️✔️SpellCheck succeeded [Detail] [Expand]
Validation passes for SpellCheck.
️️✔️PR Summary succeeded [Detail] [Expand]
Validation passes for Summary.
️️✔️Automated merging requirements met succeeded [Detail] [Expand]
Posted by Swagger Pipeline | How to fix these errors?

@openapi-workflow-bot
Copy link

Hi, @jlichwa! Thank you for your pull request. To help get your PR merged:

  • Ensure you reviewed the checklists in the PR description.
  • Know that PR assignee is the person auto-assigned and responsible for your current PR review and approval.
  • For convenient view of the API changes made by this PR, refer to the URLs provided in the table in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.
  • @openapi-pipeline-app
    Copy link

    openapi-pipeline-app bot commented Aug 3, 2023

    Swagger Generation Artifacts

    ️️✔️ApiDocPreview succeeded [Detail] [Expand]
    ️️✔️SDK Breaking Change Tracking succeeded [Detail] [Expand]

    Breaking Changes Tracking

    Posted by Swagger Pipeline | How to fix these errors?

    @openapi-pipeline-app
    Copy link

    Swagger pipeline started successfully. If there is ApiView generated, it will be updated in this comment.

    @heaths heaths merged commit 9fdddb9 into release-keyvault-Microsoft.KeyVault-7.5-preview.1 Aug 3, 2023
    @heaths heaths deleted the KeyVault-7.5-preview-fixes branch August 3, 2023 21:48
    konrad-jamrozik pushed a commit that referenced this pull request Aug 12, 2023
    * Adds base for updating Microsoft.KeyVault from version stable/7.4 to version 7.5-preview.1
    
    * Updates readme
    
    * Updates API version in new specs and examples
    
    * Adding New readonly property hsmPlatform to the Key Attributes (#23769)
    
    * Trustedmode backuprestore (#23780)
    
    * Trusted mode backup, restore, selective restore swagger specifications
    
    * Rename trusted mode to managed identity mode
    
    * Incorporate Heath's PR suggestions.
    
    * Description change
    
    * add enums
    
    * Incorporate heath's suggestion
    
    * reword description
    
    * rename
    
    * Redact sensitive info (#25159)
    
    * Fix example base64 encodings (#25273)
    
    ---------
    
    Co-authored-by: Heath Stewart <[email protected]>
    Co-authored-by: Kapil Jain (MSFT) <[email protected]>
    Co-authored-by: vasanthrajams <[email protected]>
    Co-authored-by: Heath Stewart <[email protected]>
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    3 participants