ModuleAPI SSO auth callbacks

[yasinishyn]

This PR implements a callaback that is executed on any SSO auth (registration and logins). This introduces an ability to add custom callbacks using ModuleAPI.

Reasoning:

• Password auth provider callbacks - will not work with SSO type logins;
• account_validity_callbacks.on_user_registration runs only on registration and is called before external_ids are set removing the ability to get the remote_user_id inside the ModuleAPI callback'
• There is not general on_login callback (which can be another interesting solution);

Pull Request Checklist

• Pull request is based on the develop branch
• Pull request includes a changelog file. The entry should:
  • Be a short description of your change which makes sense to users. "Fixed a bug that prevented receiving messages from other servers." instead of "Moved X method from EventStore to EventWorkerStore.".
  • Use markdown where necessary, mostly for code blocks.
  • End with either a period (.) or an exclamation mark (!).
  • Start with a capital letter.
  • Feel free to credit yourself, by adding a sentence "Contributed by @github_username." or "Contributed by [Your Name]." to the end of the entry.
• Pull request includes a sign off
• Code style is correct
  (run the linters)

Signed-off-by: Andrii Yasynyshyn yasinishyn.a.n@gmail.com

[clokep]

Looks like this is failing lint.

[yasinishyn]

Linters are fixed. @clokep could you please take a look again?

[clokep]

I kicked off the build again, this is in our review queue. 👍

[yasinishyn]

I kicked off the build again, this is in our review queue. 👍

Thank you! :)

[MatMaul]

@anoadragon453 what do you think of on_sso_login vs having a more generic on_login as proposed ?

On my side I think I would favor just going for a generic on_login, someone will probably need it someday and then we would have to keep both for not much gain.

[anoadragon453]

A generic on_user_login sounds reasonable to me. I found an issue that relates to this and wrote up what I think would be a reasonable design at #10487 (comment).

[yasinishyn]

@MatMaul @anoadragon453
Please let me know if you need my help here.
Will be happy to implement a more generic on_user_login callback.

[MatMaul]

@yasinishyn that would be nice, thank you 🙂

[clokep]

(Removing from the review queue since it sounds like changes are planned?)

[yasinishyn]

I am sorry for the late update on this.
Please see a more generic implementation of the callback added in the last comment.

[clokep]

There's a few linting errors.

[erikjohnston]

Makes sense to me! Thank you