-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Key Vault Certificates #6467
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
Closed
Closed
Key Vault Certificates #6467
Changes from 1 commit
Commits
Show all changes
166 commits
Select commit
Hold shift + click to select a range
84e325f
working test case
samvaity 3c4baeb
adding more impl
samvaity ec9a925
crud working
samvaity 4f35316
working until contacts
samvaity e121e4d
all working
samvaity e53155c
working all
samvaity 45559a2
passes initial tests
iscai-msft f26938d
Merge branch 'master' into keyvault-certs
iscai-msft bb2bf3f
added some changes after design meeting, crud issuers passes
iscai-msft 955da45
added list to crud issuers and passes, merge certs not passing
iscai-msft c87f6dd
fixed CeritificatePolicy wrapper class, all tests besides merge certi…
iscai-msft 5a6c05a
hello world sample works
iscai-msft 3a0d425
list operations sample passing
iscai-msft 7c592f6
all sync samples working
iscai-msft 8aa6eb8
get_pending_csr test passes
iscai-msft 0acec0c
Merge branch 'keyvault-certs' of github.com:iscai-msft/azure-sdk-for-…
iscai-msft 6bfa74e
switched import certificates parameter to bytes
iscai-msft 2485f02
modified wrapped models, added docstrings
iscai-msft 9c6fd42
uncommented tests
iscai-msft c5825ad
added readme for certificates
iscai-msft d8e59e8
deleted pip and lock file
iscai-msft aaf9220
deleted copied secret aio implementation
iscai-msft 86746f9
pulled origin master key recordings
iscai-msft 835ad10
Merge branch 'master' of github.com:Azure/azure-sdk-for-python into k…
iscai-msft 043de50
added _shared folder
iscai-msft 40528c5
added test recordings for secrets and keys for CI
iscai-msft 3f57c36
passes hello word sample test
iscai-msft d73cf66
adding samples tests, waiting to hear from keyvault team about questions
iscai-msft 6119001
changed default enabled value to True
iscai-msft e6e1aa3
implemented async client
iscai-msft 715151a
added get_pending_csr async implementation
iscai-msft 8b08df8
added distributed_trace_async decorator
iscai-msft 328c3fa
Merge branch 'master' of github.com:Azure/azure-sdk-for-python into k…
iscai-msft 3a43cb4
got rid of config in client
iscai-msft f59da85
made lists not async
iscai-msft 91a6206
list testing passes
iscai-msft 2cc9966
got rid of unnecessary variables
iscai-msft b73ab20
fixed import statements
iscai-msft f4d0061
fixed merge conflict
iscai-msft 2a71649
Merge branch 'master' of github.com:Azure/azure-sdk-for-python into c…
iscai-msft b40c0ad
fixed transport error, still have contacts error
iscai-msft 1e0a445
all pass but merge and async request in playback
iscai-msft 5908047
Merge remote-tracking branch 'origin/master' into keyvault-certs
iscai-msft 0105b74
complying with latest core changes
iscai-msft b07585e
Merge branch 'keyvault-certs' of github.com:iscai-msft/azure-sdk-for-…
iscai-msft edb73e4
fixed import statement
iscai-msft 12ca6cd
moved _to_certificate_policy_bundle and ActionType enum
iscai-msft 4a164ee
implemented charles' comments
iscai-msft 2bd021d
using master recordings for keys and secrets
iscai-msft 70c0945
Merge branch 'keyvault-certs' of github.com:iscai-msft/azure-sdk-for-…
iscai-msft 6d6095d
implemented charles' comments
iscai-msft 25e231c
added distributed trace decorators
iscai-msft cd579a2
Merge branch 'keyvault-certs' of github.com:iscai-msft/azure-sdk-for-…
iscai-msft ba62e63
samples running in asyncio loop, not passing
iscai-msft 95e4ffa
integrated core changes
iscai-msft 7949b24
included async client exposure
iscai-msft 4df00d6
samples running
iscai-msft e96ba8c
fixed import statement
iscai-msft a61674c
fixed import statements
iscai-msft 77cf453
Merge branch 'keyvault-certs' of github.com:iscai-msft/azure-sdk-for-…
iscai-msft b9c7595
removed unnecessary async samples from sync pr
iscai-msft 2b0d4ce
working on charles' latest comments
iscai-msft 67229b2
Merge branch 'master' of github.com:Azure/azure-sdk-for-python into k…
iscai-msft fc9341d
still working on comments
iscai-msft b3f64ec
Merge branch 'keyvault-certs' of github.com:iscai-msft/azure-sdk-for-…
iscai-msft 82e03b8
fixed docstring
iscai-msft b0e01df
async_request_cancellation_and_deletion now passing in playback
iscai-msft 8d6f30e
Merge branch 'keyvault-certs' of github.com:iscai-msft/azure-sdk-for-…
iscai-msft da7537d
cleaned up async request test
iscai-msft 4c759fb
Merge branch 'keyvault-certs' of github.com:iscai-msft/azure-sdk-for-…
iscai-msft 86a061f
Merge branch 'master' of github.com:Azure/azure-sdk-for-python into c…
iscai-msft 74722da
async request test works in playback, added challenged based auth
iscai-msft 1c58b5c
Merge branch 'master' of github.com:Azure/azure-sdk-for-python into k…
iscai-msft 2e9525f
added challenge based auth code
iscai-msft be3a83c
Merge branch 'keyvault-certs' of github.com:iscai-msft/azure-sdk-for-…
iscai-msft e6a519d
renamed client and models file
iscai-msft db85bbd
Merge branch 'keyvault-certs' of github.com:iscai-msft/azure-sdk-for-…
iscai-msft d2676bf
renamed async client file
iscai-msft dd1271a
fixed imports
iscai-msft b5198bc
removed keyvaultid module
iscai-msft b9e3f32
Merge branch 'keyvault-certs' of github.com:iscai-msft/azure-sdk-for-…
iscai-msft 8b37c4e
removed references to keyvaultid module
iscai-msft ddb76a2
added check if none for admin details
iscai-msft bcfc179
Merge branch 'keyvault-certs' of github.com:iscai-msft/azure-sdk-for-…
iscai-msft 45d2d6d
added examples to docstrings
iscai-msft fd06b29
Merge branch 'keyvault-certs' of github.com:iscai-msft/azure-sdk-for-…
iscai-msft 16b6e11
fixed get certificate example
iscai-msft 4d9aed4
Merge branch 'keyvault-certs' of github.com:iscai-msft/azure-sdk-for-…
iscai-msft 63371e5
added examples in docstrings
iscai-msft 918b556
removed samples testing
iscai-msft 43c8682
Merge branch 'keyvault-certs' of github.com:iscai-msft/azure-sdk-for-…
iscai-msft 37b72d0
added challenge auth testing
iscai-msft 3cbd714
Merge branch 'keyvault-certs' of github.com:iscai-msft/azure-sdk-for-…
iscai-msft 3777fc6
added challenge auth testing
iscai-msft 8000a27
fixed type hinting in certs client to work with python 2.x
iscai-msft e8f2a5a
began pylint errors
iscai-msft 13d85a7
Merge branch 'keyvault-certs' of github.com:iscai-msft/azure-sdk-for-…
iscai-msft 05bb7f3
pylint on aio done except exceptions
iscai-msft ce7fe78
pylinting
iscai-msft 56c5a48
pylinted samples
iscai-msft 6ff1bdb
passing pylint
iscai-msft 73d3fbf
Merge branch 'keyvault-certs' of github.com:iscai-msft/azure-sdk-for-…
iscai-msft b46c728
rewriting merge test
iscai-msft 4283000
Merge branch 'master' of github.com:Azure/azure-sdk-for-python into k…
iscai-msft e6cc5f3
switched san_dns names
iscai-msft 69ddc9f
Merge branch 'keyvault-certs' of github.com:iscai-msft/azure-sdk-for-…
iscai-msft e5abe2f
changing san_dns_names
iscai-msft c58b315
updating recordings with new san DNS names
iscai-msft 5444cfe
Merge branch 'keyvault-certs' of github.com:iscai-msft/azure-sdk-for-…
iscai-msft 5ce7a42
Merge branch 'master' of github.com:Azure/azure-sdk-for-python into k…
iscai-msft e971659
used pre-hashed certificate bytes for merge_certificate, not working
iscai-msft 22bfb69
Merge branch 'master' of github.com:Azure/azure-sdk-for-python into k…
iscai-msft e3103a7
clean after Krista's pylint changes
iscai-msft 3f56d26
Merge branch 'keyvault-certs' of github.com:iscai-msft/azure-sdk-for-…
iscai-msft 2987092
async clean after Krista's pylint changes
iscai-msft 91951af
Merge branch 'master' of github.com:Azure/azure-sdk-for-python into k…
iscai-msft 783199e
trying to add backup restore test
iscai-msft 1f92178
added backup restore test, passing
iscai-msft 425db1c
Merge branch 'keyvault-certs' of github.com:iscai-msft/azure-sdk-for-…
iscai-msft 1dcb7ef
added backup restore test
iscai-msft 546676f
got rid of unnecessary waiting and print statements
iscai-msft 7c85af6
fixed small errors in readme
iscai-msft 981d2ca
fixed samples commenting
iscai-msft f3a3ed4
Merge branch 'master' of github.com:Azure/azure-sdk-for-python into c…
iscai-msft b17354d
fixing mypy issues
iscai-msft c114701
added new seed testing
iscai-msft 88ef59b
working on merge certificate test
iscai-msft 46263b5
got rid of not before and expires
iscai-msft 5cc2a75
got rid of not before and expires
iscai-msft 4b5722c
added Charles' comments from other pylint pr
iscai-msft deefd92
added SecretContentType enum for CertificatePolicy
iscai-msft cfc8444
added KeyUsageType enum
iscai-msft ad4db88
added default policy, tests passing
iscai-msft b98e47c
before switching to shell commands
iscai-msft fdbce63
using shell commands
iscai-msft 359ebbf
Revert "using shell commands"
iscai-msft 7120f72
merge certificate passing!!!!
iscai-msft f63d97a
added seeding, all tests passing
iscai-msft dccd338
Merge branch 'certs-async' of github.com:iscai-msft/azure-sdk-for-pyt…
iscai-msft 6b14fe1
fixed weird conflict error
iscai-msft 17b49ae
fixing dev requirements and test to pass ci
iscai-msft 59bda52
removed itertools to be compatible with python 2.7
iscai-msft b3709ab
Merge branch 'master' of github.com:Azure/azure-sdk-for-python into c…
iscai-msft 6aaee8b
reset changes to mgmt plane test
iscai-msft 2a5516c
changed default certificate policy
iscai-msft 3fe5db7
getting rid of erroneous certificate policy model change
iscai-msft 9fb3745
implemented lro for create certificate, all tests passing
iscai-msft c2d584e
implemented Charles' suggestions
iscai-msft c6ae595
implemented Charles' comments
iscai-msft b7d3d24
moved sans to kwargs, added comments to models
iscai-msft c18a9a1
fixed links in samples
iscai-msft 80d189d
Merge remote-tracking branch 'origin/master' into certs-async
iscai-msft eb7922c
Merge branch 'master' of github.com:Azure/azure-sdk-for-python into c…
iscai-msft b2fb410
fixed typing error, awaited all async pollers
iscai-msft a681e89
updated lowerbound for azure core version
iscai-msft 901dcae
fixed linting errors
iscai-msft c820f00
updated readme
iscai-msft 98ff072
parsing out certificate id in poller
iscai-msft 5e4d99e
added model imports in aio init
iscai-msft 6b4a33a
fixed init files to just use models passed to methods
iscai-msft 84ced76
fixed enum conversion and import statements
iscai-msft 1840071
split get_certificate into one with version and one with policy
iscai-msft 390e692
fixed docstrings, added ResourceNotFoundError and ResourceExistError …
iscai-msft 1e9bf73
fixed linting error
iscai-msft 960a9b4
reordered logic in conftest for RUN_IDENTIFIER
iscai-msft 69808a4
reverted SecretContentType types and get_pending_csr
iscai-msft File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
hello world sample works
- Loading branch information
commit 5a6c05a4ad8bcc7f64c534876ace8adf07263307
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What happens if we create a
CertificateAttributesinstance with allNonearguments to the constructor? Is it not semantically the same as aNoneattributes?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think @chlowell mentioned this to me before, I believe calling it with all None arguments resulted in sending an object with all of the fields as None. I can't remember it exactly so I might be wrong though, I'll ask tomorrow.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm the source of this ugly pattern. Given an object whose serializable attributes all are
None, the serializer outputs an empty object. So we'd send over the wire something like{ "attributes": {} }