Fix store migration on empty string - Githubissues

netbirdio / netbird

Connect your devices into a secure WireGuard®-based overlay network with SSO, MFA and granular access controls.

https://netbird.io

BSD 3-Clause "New" or "Revised" License

10.74k stars 484 forks source link

Fix store migration on empty string #2149

Closed mlsmaycon closed 3 months ago

mlsmaycon commented 3 months ago

When fetching empty values from the database to check for migration our parser failed to handle null strings preventing the service from start

This uses sql.NullString to handle that and check for empty string resulted from null data

Describe your changes

Issue ticket number and link

Checklist

[x] Is it a bug fix
[ ] Is a typo/documentation fix
[ ] Is a feature enhancement
[ ] It is a refactor
[ ] Created tests that fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

sonarcloud[bot] commented 3 months ago

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud