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.

Sign up for GitHub

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

Jump to bottom

[Improve][Manager] Eliminate NullPointerException on user management #4965

Closed
2 tasks done
woofyzhao opened this issue Jul 11, 2022 · 0 comments · Fixed by #4971
Closed
2 tasks done

[Improve][Manager] Eliminate NullPointerException on user management #4965

woofyzhao opened this issue Jul 11, 2022 · 0 comments · Fixed by #4971
Assignees
@woofyzhao
Labels
component/manager type/bug Something is wrong
Milestone
1.3.0

Comments

@woofyzhao
Copy link
Contributor

Description

image

The admin user is special in that it is pre inserted by script and does not have encrypted fields like secret_key or public_key, etc. Thus the np should be avoided when some fields are null.

InLong Component

InLong Manager

Are you willing to submit PR?

  • Yes, I am willing to submit a PR!

Code of Conduct
@woofyzhao woofyzhao mentioned this issue Jul 11, 2022
Merged
@healchow healchow changed the title [Improve][Manager] Eliminate np exception on user management. [Improve][Manager] Eliminate NullPointerException on user management Jul 11, 2022
@healchow healchow added this to the 1.3.0 milestone Jul 11, 2022
@healchow healchow added component/manager type/bug Something is wrong and removed type/improve labels Jul 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@woofyzhao woofyzhao

Labels
component/manager type/bug Something is wrong
Projects
None yet
Milestone
1.3.0
Development

Successfully merging a pull request may close this issue.

[INLONG-4965][Manager] Fix np exception on user query api woofyzhao/incubator-inlong
2 participants
@woofyzhao @healchow