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
add duo intel module #1172
add duo intel module #1172
Conversation
added other realtionships, jsonof other props add groups, cleanups fix variable name doc update add logs docs test api hosts, groups test endpoints test users fix group test undo lastpass edit undo conftest change
9d92ea3
to
94e5e9d
Compare
'user_id': user['user_id'], | ||
'username': user['username'], | ||
'webauthncredentials': [ | ||
dumps(webauthncredential) |
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.
Making sure I understand: this takes the full json and make it a string?
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.
yes
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 guess this is ok for now. Eventually maybe it should be a node but I won't block on this.
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.
Added. And changed the direction of MEMBER_OF_DUO_GORUP
Co-authored-by: Alex Chantavy <achantavy@lyft.com>
9743df6
to
da9cb02
Compare
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.
Looks good to me, thank you for working on this! As a future note let's please not make 3k line PRs a regular thing :p
Adds a Duo intel module