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.

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

Jump to bottom

EncryptDecrypt is depricated #196

Closed
mathias-nickl opened this issue Jan 22, 2018 · 3 comments
Closed

EncryptDecrypt is depricated #196

mathias-nickl opened this issue Jan 22, 2018 · 3 comments
Assignees
@prmathur-microsoft
Labels
bug fix checked in Fix checked into main or preview, but not yet released.

Comments

@mathias-nickl
Copy link

azure-iot-sdk-java/provisioning/security/tpm-provider/src/main/java/com/microsoft/azure/sdk/iot/provisioning/security/hsm/SecurityProviderTPMHsm.java

Line 337 in 2b2eed3

EncryptDecryptResponse edResp = tpm.EncryptDecrypt(hSymKey, (byte)1, TPM_ALG_ID.CFB, iv, encUriData.buffer);

Dear Provisioning team,

TPM command EncryptDecrypt (0x164) is depricated and not supported on newer hardware. It is recommended to use EncryptDecrypt2 instead.

Best

MN
@prmathur-microsoft prmathur-microsoft self-assigned this Jan 22, 2018
@prmathur-microsoft
Copy link
Member

We will work on fixing this issue.

prmathur-microsoft added a commit that referenced this issue Jan 29, 2018
@prmathur-microsoft
Remove deprecated version of EncryptDecrypt (Fix Github issue #196)
9706731
prmathur-microsoft added a commit that referenced this issue Jan 31, 2018
@prmathur-microsoft
Remove deprecated version of EncryptDecrypt (Fix Github issue #196)
699b8e6
@prmathur-microsoft
Copy link
Member

This issue is now fixed and in master. We will be releasing a package for the same on 2/9.

@prmathur-microsoft prmathur-microsoft added the fix checked in Fix checked into main or preview, but not yet released. label Feb 1, 2018
@prmathur-microsoft
Copy link
Member

New maven packages with a fix for this issue are released. Feel free to reopen this issue if you still see any problems. Closing this issue for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@prmathur-microsoft prmathur-microsoft

Labels
bug fix checked in Fix checked into main or preview, but not yet released.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mathias-nickl @prmathur-microsoft