Skip to content

Update: Readme/Integration test case with key alias extension use cases #15

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.

Sign up for GitHub

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update: Readme/Integration test case with key alias extension use cases #15

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
848da2a
Updated Readme with key alias extension use cases
Jan 27, 2022
bf1d8f6
Added key alias extension for integration test case
Jan 31, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
50 changes: 50 additions & 0 deletions test/integration/key-protect.v2.test.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -763,4 +763,54 @@ describe('key protect v2 integration', () => {
done();
});
});
describe('key alias extensions', () => {
it('checkKeyaliasExtension', async done => {
let response;
const samplePlaintext = 'dGhpcyBpcyBhIGJhc2U2NCBzdHJpbmcK';
try {
// create a key alias
const keyAlias = 'nodejsAlias';
const createKeyAliasParams = Object.assign({}, options);
createKeyAliasParams.id = keyId;
createKeyAliasParams.alias = keyAlias;
response = await keyProtectClient.createKeyAlias(createKeyAliasParams);
expect(response).toBeDefined();
expect(response.status).toEqual(201);

// wrap using key alias
const wrapKeyParams = Object.assign({}, options);
wrapKeyParams.id = createKeyAliasParams.alias;
Copy link
Member

@dinesh-venkatraman-g dinesh-venkatraman-g Apr 7, 2022

Choose a reason for hiding this comment

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

We should not be setting alias to id.. instead, we should be able to set alias in input and perform the key action

wrapKeyParams.keyActionWrapBody = {
plaintext: samplePlaintext,
};
response = await keyProtectClient.wrapKey(wrapKeyParams);
const ciphertextResult = response.result.ciphertext;
expect(response).toBeDefined();
expect(response.status).toEqual(200);

// un-wrap using key alias
const unwrapKeyParams = Object.assign({}, options);
unwrapKeyParams.id = createKeyAliasParams.alias;
Copy link
Member

@dinesh-venkatraman-g dinesh-venkatraman-g Apr 7, 2022

Choose a reason for hiding this comment

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

We should not be setting alias to id.. instead, we should be able to set alias in input and perform the key action

unwrapKeyParams.keyActionUnwrapBody = {
ciphertext: ciphertextResult,
};
response = await keyProtectClient.unwrapKey(unwrapKeyParams);
const plaintextResult = response.result.plaintext;
expect(response).toBeDefined();
expect(plaintextResult).toEqual(samplePlaintext);
expect(response.status).toEqual(200);

// delete a key using key alias
const deleteKeyParams = Object.assign({}, options);
deleteKeyParams.id = createKeyAliasParams.alias;
Copy link
Member

@dinesh-venkatraman-g dinesh-venkatraman-g Apr 7, 2022

Choose a reason for hiding this comment

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

We should not be setting alias to id.. instead, we should be able to set alias in input and perform the key action

deleteKeyParams.prefer = 'return=representation';
response = await keyProtectClient.deleteKey(deleteKeyParams);
expect(response).toBeDefined();
expect(response.status).toEqual(200);
} catch (err) {
done(err);
}
done();
});
});
});