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

Issue622 Fix #640

Merged
merged 4 commits into from Jan 23, 2020
Merged

Issue622 Fix #640

merged 4 commits into from Jan 23, 2020

Conversation

maestretti
Copy link
Contributor

Fixing a JSON key error for #622

@claassistantio
Copy link

claassistantio commented Jan 9, 2020

CLA assistant check
All committers have signed the CLA.

shared/iam_audit.py Outdated Show resolved Hide resolved
@0xdabbad00
Copy link
Collaborator

@maestretti Are you able to sign the CLA? I know we had some issues in the past with it, but I don't know if those have been resolved yet.

@maestretti
Copy link
Contributor Author

maestretti commented Jan 9, 2020

@0xdabbad00 I signed it... email mismatch on the commit and my GitHub account which I fixed and re-checked. Now passing checks.

@maestretti
Copy link
Contributor Author

@0xdabbad00 this looks ready to go?

@0xdabbad00 0xdabbad00 merged commit 4c32a63 into duo-labs:master Jan 23, 2020
@0xdabbad00
Copy link
Collaborator

Yes, thank you for following up and for submitting the PR.

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.

None yet

3 participants