turbot / steampipe-plugin-okta

Use SQL to instantly query users, groups, applications and more from Okta. Open source CLI. No DB required.
https://hub.steampipe.io/plugins/turbot/okta
Apache License 2.0
8 stars 4 forks source link

Add `owner` to okta_group table #132

Open securitygeneration opened 1 month ago

securitygeneration commented 1 month ago

References https://developer.okta.com/docs/api/openapi/okta-management/management/tag/GroupOwner/

ParthaI commented 1 month ago

Hello @securitygeneration, just a heads-up: we've added support for a new table named okta_group_owner in this PR. This table will list all the owners of groups.

I couldn't test it in my environment due to a lack of permissions.

It would be great if you could try it out from the PR branch issue-132.

Steps to build the plugin locally from the branch issue-132

Thanks

securitygeneration commented 1 month ago

Hi @ParthaI, thanks!

Getting this error:

> select * from okta_group_owner

Error: okta: parsing time "2023-09-26 12:18:57.0" as "2006-01-02T15:04:05Z07:00": cannot parse " 12:18:57.0" as "T" (SQLSTATE HV000)
ParthaI commented 1 month ago

Hello @securitygeneration,

Thank you so much for testing the PR. I have addressed the above error and pushed a fix in the same branch (issue-132). Could you please pull the latest changes from the branch and try running the query by building the plugin locally again?

securitygeneration commented 1 month ago

Still getting this error after doing a git pull and make:

Error: okta: parsing time "2023-09-26 12:21:00.0" as "2006-01-02T15:04:05Z07:00": cannot parse " 12:21:00.0" as "T" (SQLSTATE HV000)

ParthaI commented 1 month ago

Could you please confirm if you have terminated all running Steampipe processes before building the plugin again? If not, please run the command pkill "steampipe" before building/querying the plugin.

securitygeneration commented 1 month ago

Still the same error

ParthaI commented 1 month ago

Hi @securitygeneration,

I apologize for the inconvenience. I have pushed some fixes to the issue-132 branch to address the timestamp parsing issue.

Could you please pull the latest changes from the issue-132 branch, rebuild the plugin locally, and run the query again? If possible, could you also share your log details from ~/.steampipe/logs/plugin-...., making sure to remove any sensitive information?

Thank you!