loadDiscoveryDocumentAndLogin() should be renamed to loadDiscoveryDocumentAndStartImplicitFlow()

Current method name loadDiscoveryDocumentAndLogin() is quite confusing, because it does not actually call something similar to the tryLogin() method (like loadDiscoveryDocumentAndTryLogin() does), but instead it does something completely different - ie, it redirects the user to the auth server.

The similarity in the names of loadDiscoveryDocumentAndLogin() and loadDiscoveryDocumentAndTryLogin() is very confusing, because they actually do completely different stuff.

It would be great to make the names more clear.

[jeroenheijmans]

I think, with a year past, no real interaction or others chiming in here, and the level of breaking change this would be, we could close this issue. Don’t think we’ll be making this change/addition on any short term…

[Maximaximum]

The name change could be done in the major branch. The new version does not have to be released right away, it would be ok to do it whenever there are enough reasonable changes, but at least the name changes will be in the next major release.

Also, I don’t understand what’s the problem to mark some method names as deprecated. It will not force the old users to change a single letter in their code, but will allow the new users to use meaningful names.

Otherwise… well, to be honest, the current state of the library in terms of clearness and understandability (and documentation as well) is not excellent (at least). It can just stay this way, then one day probably a better library appears, and everyone will simply move to using that one. Or, it can be improved.