Skip to content

Conversation

@misraved
Copy link
Contributor

@misraved misraved commented Dec 2, 2025

Example query results

Results
Add example SQL query results here (please include the input queries as well)

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR introduces an optional is_full_admin configuration parameter that allows users without full admin privileges to use the okta_user table without encountering permission errors when querying the assigned_roles column.

Key Changes:

  • Added is_full_admin configuration field that defaults to false
  • Modified listAssignedRolesForUser to conditionally fetch user roles only when is_full_admin is enabled
  • Added helper functions for retrieving boolean configuration values with environment variable fallback

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
okta/connection_config.go Adds IsFullAdmin field to the config struct to store the new configuration parameter
okta/connect.go Adds getEnvVarBool and getBoolValue helper functions to handle boolean config values with environment variable fallback, consistent with existing int32/int64 helpers
okta/table_okta_user.go Adds conditional check in listAssignedRolesForUser to skip fetching roles when is_full_admin is false; includes minor formatting alignment for Func field
docs/index.md Documents the new is_full_admin configuration parameter and OKTA_IS_FULL_ADMIN environment variable; includes minor table formatting fix
config/okta.spc Adds commented configuration example for is_full_admin with detailed description

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

3 participants