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

[KeyVault] Preview #4 - release testing #5444

Closed
9 tasks
sophiajt opened this issue Oct 8, 2019 · 3 comments · Fixed by #5447
Closed
9 tasks

[KeyVault] Preview #4 - release testing #5444

sophiajt opened this issue Oct 8, 2019 · 3 comments · Fixed by #5447
Labels
Client This issue points to a problem in the data-plane of the library. KeyVault

Comments

@sophiajt
Copy link
Contributor

sophiajt commented Oct 8, 2019

Keys

  • Line 40 helloWorld.ts - Missing ! on version
const updatedKey = await client.updateKey(keyName, result.properties.version, { enabled: false });

Secrets

  • Line 26 helloWorld.ts - Missing ! on version
  const updatedSecret = await client.updateSecretProperties(secretName, result.properties.version, {
    enabled: false
  });
  • Line 47 backupAndRestore.ts - Missing ! on backupResult
await writeFile("secret_backup.dat", backupResult);

Certificates

  • Line 38 helloWorld.ts - Missing ! on version
certificateWithPolicy.properties.version
  • Line 44 issuers.ts - 'certificate' declared but never read
const certificate = await client.getCertificate("MyCertificate", "");
  • issuers.ts - error when running: "Issuer not found"
  • backupAndRestore.ts - error when running: "Certificate is currently being deleted." Are the delays sufficient here?
  • contacts.ts - error when running: "Contacts not found"
  • operations.ts - error when running "The 'version' cannot be empty." Seems to be erroring on getCertificate
@sophiajt sophiajt added Client This issue points to a problem in the data-plane of the library. KeyVault labels Oct 8, 2019
@triage-new-issues triage-new-issues bot removed the triage label Oct 8, 2019
@sadasant
Copy link
Contributor

sadasant commented Oct 8, 2019

Oh this is specifically about the samples.

@sadasant
Copy link
Contributor

sadasant commented Oct 8, 2019

Notes:

  • Our samples are being built on CI too. I wonder why we're not catching these.
  • "Issuer not found" and "Contacts not found" <-- Are these errors? We're catching the errors in the samples to show that the resources have been deleted.

@sadasant
Copy link
Contributor

sadasant commented Oct 8, 2019

I created some config files called tsconfig.samples.json and changed the samples build script. I was able to catch and fix all of the feedback here and more! 🌞

@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Client This issue points to a problem in the data-plane of the library. KeyVault
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants