Skip to content

Conversation

@mattmundell
Copy link
Contributor

What

Move add_users out of manage_sql.c.

Why

Reducing size of manage_sql.c. Better per-file organisation of resources.

References

Follows /pull/2709.

Testing

Created group and role with users, looks OK.

o m m '<create_role><name>r test 1</name><users>testuser, m_test</users></create_role>'
o m m '<create_group><name>test 1</name><users>m, m_test</users></create_group>'

manage_user_exists was removed in f74e5f1
manage_user_uuid was removed in f74e5f1
@mattmundell mattmundell requested review from a team as code owners January 12, 2026 15:39
@greenbonebot greenbonebot enabled auto-merge (rebase) January 12, 2026 15:39
@greenbonebot greenbonebot merged commit 49b1c2a into main Jan 13, 2026
29 of 30 checks passed
@greenbonebot greenbonebot deleted the user-files-3 branch January 13, 2026 09:59
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.

4 participants