[Request] logins configure roles too

[DorBreger]

Is your feature request related to a problem? Please describe.
when using the login module I would like to add roles too.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Have another parameter of type list with all roles that will be granted to the login
Describe alternatives you've considered

Additional context

[lowlydba]

Are you thinking server roles or database roles?

[DorBreger]

Server

[lowlydba]

That seems like a good fit! It may make sense to also add a module for defining server roles as well to complement this (#251)

[lowlydba]

So my schedule is going to be crazy for a while, and I thought I'd have time to work on this but likely will not. I did start getting the initial logic down, if you want to use it as a head start on making a PR.

https://github.com/lowlydba/lowlydba.sqlserver/compare/add-roles-to-login?expand=1

[DorBreger]

So my schedule is going to be crazy for a while, and I thought I'd have time to work on this but likely will not. I did start getting the initial logic down, if you want to use it as a head start on making a PR.

https://github.com/lowlydba/lowlydba.sqlserver/compare/add-roles-to-login?expand=1

Sure. I'm quite busy as well, but I will try to tackle this when I have time, it seems interesting, thanks for the help!

[DorBreger]

seems like I need to be a collaborator for this link to work

[lowlydba]

Ah, sorry about that. Does this work? main...add-roles-to-login