When you have "Turn on automated single sign-on" turned on, anonymous users cannot log into the site because they receive:
Call to undefined function drupal_get_path_alias() in \includes\common.inc
This is due to the drupal_bootstrap_full removal issue where not enough core files are loaded to handle things like drupal_goto properly. In addition once you do load /includes/path.inc, an infinite loop occurs for anonymous users on the home page because the destination variable is not set properly. This is a quick fix for those problems but far from fixing the problem. There are still several warnings about the $auth_conf object that need to be resolved but at least you can get to a login form to get back into your site with this fix.
@johnbarclay,
Perhaps drupal_bootstrap_full is not all that excessive. This could be a perpetual problem keeping up with core if we continue in this method. Thoughts?
| Comment | File | Size | Author |
|---|---|---|---|
| drupal_get_path_alias.patch | 922 bytes | haydeniv |
Comments
Comment #1
johnbarclay commentedThanks, I'll work it into ldap sso when I get to it soon, I have a bunch of uncommitted ldap sso code already. My sequence is here: http://drupal.org/node/1115704#comment-5569662
I think ldap sso needs alot of work. It needs to accommodate some models that integrate with ldap such as NTLM, but avoid being being a generic SSO module. After I finish with LDAP OG, I have a couple weeks set aside to focus on LDAP SSO.
Its not so much the load of the early bootstrap that bothers me, its that the hooks are fired at an odd time. Not all modules deal with this well.
I would say
- use the patch to keep your site working
- I created an ldap sso testing thread. If you can do any testing, add a comment there with your environment (#1441710)
- any insight you have on what types of SSOs LDAP SSO should support would be greatly appreciated in the general thread at (#1115704). Especially if you are familiar with other Drupal 7 SSO modules.
Comment #6
johnbarclay commentedthanks. I committed this. I'm holding off on the code in comment #1 and patched this against head.
Comment #7
johnbarclay commented