Skip to content

Fix MasterKeyprovider.decrypt_data_key_from_list error handling #152

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

Merged
merged 2 commits into from
May 21, 2019
Merged

Fix MasterKeyprovider.decrypt_data_key_from_list error handling #152

merged 2 commits into from
May 21, 2019

Conversation

mattsb42-aws
Copy link
Member

Issue #, if available: #150

Description of changes:
MasterKeyprovider.decrypt_data_key_from_list should catch expected exceptions from both MasterKeyProvider.decrypt_data_key and MasterKey.decrypt_data_key #150

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@mattsb42-aws mattsb42-aws requested a review from a team May 20, 2019 22:18
@karlw00t
Copy link
Contributor

seems to be an error with something upstream in appveyor

building 'wrapt._wrappers' extension
  Traceback (most recent call last):
    File "<string>", line 1, in <module>
<snip>
    File "C:\projects\aws-encryption-sdk-python-qvyet\.tox\py34-local\lib\site-packages\setuptools\msvc9_support.py", line 52, in query_vcvarsall
      return unpatched['query_vcvarsall'](version, *args, **kwargs)
    File "c:\python34-x64\Lib\distutils\msvc9compiler.py", line 287, in query_vcvarsall
      raise ValueError(str(list(result.keys())))
  ValueError: ['path']

@mattsb42-aws mattsb42-aws merged commit 5efa515 into aws:master May 21, 2019
@mattsb42-aws mattsb42-aws deleted the dev-150 branch May 21, 2019 02:32
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.

2 participants