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

Add OutputFormat to IAM report #638

Merged
merged 2 commits into from Jan 12, 2020

Conversation

nimrodkor
Copy link
Contributor

Allow outputting the IAM report as a json file.

Resolves #637

@0xdabbad00
Copy link
Collaborator

I think you may have missed committing a file, as I believe all this does is provide a command-line argument that outputs a file with either a .html or .json extension, meaning it's the same file, just with different extensions, so the .json file would be an HTML file.

@nimrodkor
Copy link
Contributor Author

@0xdabbad00 The other thing is it skips the template.render(t=t) command - so it actually dumps the json into a file, instead of creating an HTML file out of it and then saving it to a file

@0xdabbad00
Copy link
Collaborator

Ah, ok. So I guess the use case is so you can feed this data into your own template somehow.

@0xdabbad00 0xdabbad00 merged commit 01ca451 into duo-labs:master Jan 12, 2020
@nimrodkor nimrodkor deleted the feature/iam_report_as_json branch January 14, 2020 20:24
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.

Support iam_report in json format
2 participants