Active
Project:
PHPMailer OAuth2
Version:
2.0.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
25 Jan 2023 at 03:07 UTC
Updated:
27 Oct 2025 at 04:22 UTC
Jump to comment: Most recent
@jigarius in #3316295-2: No reply address is registered for the application:
In general, while looking at the code, I had some thoughts which I'll share here (thinking aloud for the maintainers to hear).
- Maybe standardize the nomenclature a bit? Some places, say AAD, some places say Azure, some places it says MS. Maybe, a standard terminology should be used in all places?
- There are 2 controllers for Microsoft at the moment - one is the MsOauth2CallbackController and one is the MsLoginController. To keep things organized, maybe both of them could be merged into a, say, AzureOauth2CallbackController with two methods: "login()" and "callback()". This way, we can have individual controllers for other services like GoogleOauth2Controller, etc. in the future. The module might contain submodules in the future, like phpmailer_oauth2_azure and phpmailer_auth2_google to keep things organized.
Other ideas
Comments
Comment #2
imclean commentedComment #3
imclean commentedIt's unlikely I'll be doing much with this module in the near future. @jigarius or anyone else who is motivated can step in and have a go.
Comment #4
imclean commentedComment #5
imclean commented@jigarius are you still interested in maintaining this project? There are a few issues open which could do with some attention.
Comment #6
imclean commented