Skip to content

fix: nil pointer evaluating interface {}.databaseURL #7726

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
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix: nil pointer evaluating interface {}.databaseURL #7726

wants to merge 1 commit into from

Conversation

@ductnn
Copy link

@ductnn ductnn commented Aug 4, 2025

No description provided.

@ductnn
fix: nil pointer evaluating interface {}.databaseURL
9801035 
Signed-off-by: ductnn <trannhuduc531998@gmail.com>
@auto-assign auto-assign bot requested a review from NGPixel August 4, 2025 16:43
@NGPixel
Copy link
Member

NGPixel commented Aug 6, 2025

You're introducing a condition where the object might exist but not the databaseURL, in which case nothing will work.

It would be better to keep it as a single condition and use something like dig to perform the check.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-work

2 participants

@ductnn @NGPixel