doc: add report_user_by_email.py to examples #242
Conversation
| # Retrieve user info from API: | ||
| email_address = get_next_arg('E-mail address of user to retrieve: ') | ||
| req_params = {"email": email_address} | ||
| # There is no get_user_by_email in the duo_client_python library, so we use the generic api_call |
There was a problem hiding this comment.
Maybe we could add a function in there? I don't know if this a common thing that people might want.
There was a problem hiding this comment.
Adding a new method to the library specifically for this use case should be rather trivial. I do not have any data on generalized API usage for user searches via email address to offer an opinion on whether a separate method is warranted. Perhaps adding a new optional parameter to the get_users() method to specify the email address could be an alternate possible update to the library. However, adding a get_user_by_email() method would maintain consistency with other methods available.
There was a problem hiding this comment.
I vote let's add a get_user_by_email method - I agree it's easy and consistent. Do you want to handle that Mark?
There was a problem hiding this comment.
New PR submitted.
Description
Added new example script for retrieving user by email address via Duo Admin API
Motivation and Context
This was done in response to issue #231
How Has This Been Tested?
Tested via manual execution for both positive and negative result requests
Types of Changes