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

Fix nil pointer dereference error for okta_factor table #137

Merged
merged 2 commits into from
Aug 7, 2024

Conversation

ParthaI
Copy link
Contributor

@ParthaI ParthaI commented Aug 7, 2024

Example query results

Results
With empty row:

> select * from okta_admin.okta_factor
+--------+----+---------+-----------+-------------+---------+--------------+----------+--------+---------+----------+--------+-------+--------------------+--------+------+
| domain | id | user_id | user_name | factor_type | created | last_updated | provider | status | profile | embedded | verify | title | sp_connection_name | sp_ctx | _ctx |
+--------+----+---------+-----------+-------------+---------+--------------+----------+--------+---------+----------+--------+-------+--------------------+--------+------+
+--------+----+---------+-----------+-------------+---------+--------------+----------+--------+---------+----------+--------+-------+--------------------+--------+------+

Okta classic Account:

> select * from okta_factor
+-----------------------+----------------------+----------------------+-------------------+---------------------+---------------------------+---------------------------+----------+--------------------+---------------------------------->
| domain                | id                   | user_id              | user_name         | factor_type         | created                   | last_updated              | provider | status             | profile                          >
+-----------------------+----------------------+----------------------+-------------------+---------------------+---------------------------+---------------------------+----------+--------------------+---------------------------------->
| dev-12345678.okta.com | ost1m64qvuJk6dICv5d7 | 00u1kcfw5yDiVG80P5d7 | [email protected] | token:software:totp | 2021-08-30T18:23:16+05:30 | 2024-04-02T23:07:58+05:30 | OKTA     | ACTIVE             | {"credentialId":"[email protected]>
| dev-12345678.okta.com | mbl1m6472anhxTgOE5d7 | 00u1kcfw5yDiVG80P5d7 | [email protected] | sms                 | 2021-08-30T18:24:44+05:30 | 2021-08-30T18:24:44+05:30 | OKTA     | PENDING_ACTIVATION | {"phoneNumber":"+919776252898"}  >
| dev-12345678.okta.com | opfg6o5fqcXWpMfKY5d7 | 00u1kcfw5yDiVG80P5d7 | [email protected] | push                | 2024-04-02T23:07:34+05:30 | 2024-04-02T23:07:58+05:30 | OKTA     | ACTIVE             | {"credentialId":"[email protected]>
+-----------------------+----------------------+----------------------+-------------------+---------------------+---------------------------+---------------------------+----------+--------------------+---------------------------------->

Okta OIE Account:

> select * from okta_admin.okta_factor
+------------------------+----------------------+----------------------+-------------------+---------------------+---------------------------+---------------------------+----------+--------+--------------------------------------------->
| domain                 | id                   | user_id              | user_name         | factor_type         | created                   | last_updated              | provider | status | profile                                     >
+------------------------+----------------------+----------------------+-------------------+---------------------+---------------------------+---------------------------+----------+--------+--------------------------------------------->
| trial-1234567.okta.com | osthk5nek8Oywuf1R697 | 00uhizi5hzEfXoGB8697 | [email protected] | token:software:totp | 2024-08-07T10:21:40+05:30 | 2024-08-07T10:21:40+05:30 | OKTA     | ACTIVE | {"credentialId":"[email protected]"}        >
| trial-1234567.okta.com | opfhk5nek7COL47M5697 | 00uhizi5hzEfXoGB8697 | [email protected] | push                | 2024-08-07T10:21:40+05:30 | 2024-08-07T10:21:40+05:30 | OKTA     | ACTIVE | {"credentialId":"[email protected]","deviceT>
|                        |                      |                      |                   |                     |                           |                           |          |        | n":"10:2021-08-01"}                         >
+------------------------+----------------------+----------------------+-------------------+---------------------+---------------------------+---------------------------+----------+--------+--------------------------------------------->

@ParthaI ParthaI self-assigned this Aug 7, 2024
@ParthaI ParthaI requested a review from misraved August 7, 2024 05:05
@ParthaI ParthaI changed the title Fix nil pointer error for factor Fix nil pointer error for okta_factor Aug 7, 2024
@misraved misraved changed the title Fix nil pointer error for okta_factor Fix nil pointer dereference error for okta_factor table Aug 7, 2024
@misraved misraved merged commit bfffcc6 into main Aug 7, 2024
1 check passed
@misraved misraved deleted the fix-nil-pointer-error-for-factor branch August 7, 2024 06:03
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.

2 participants