Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hi,
please add boost support. Boost sets cookie everytime an user login, to ensure non-cached pages are serving to the user.
I solved it by adding these lines to function fboauth_login_user().
if (module_exists('boost')) {
boost_set_cookie($account->uid);
}
Comment | File | Size | Author |
---|---|---|---|
#11 | fboauth-fix-for-two-click-boost-issue-1794578-8391042.patch | 462 bytes | grn |
Comments
Comment #1
timofey CreditAttribution: timofey commentedThe above does not work for me.
Here's what's happening:
When boost is enabled, if the person logs in through fbconnect, and they are already logged into facebook, they are taken straight back to the front page, as if nothing happened, and unless they refresh the page, they are served cached pages.
If the person is not logged into facebook, when they return to the front page, it works fine and doesn't serve cached pages.
Please help! How do prevent the fbconnect logged in user from seeing cached pages?
Comment #2
timofey CreditAttribution: timofey commentedTo correct the issue, it's not only having difficulty with boost but this happens when drupal's "Cache pages for anonymous users" is enabled.
...so the question continues, how do you let drupal know that the person is logged in, after they sign in through fbconnect??
Comment #3
quicksketchI don't use Boost on any of my sites, so I probably won't investigate this issue myself.
Comment #4
ifeelgood CreditAttribution: ifeelgood commentedThe above code works, but you need to list "fboauth/*" to boost exception list.
Comment #5
quicksketchThis is a feature request, not a bug.
Comment #6
Alauddin CreditAttribution: Alauddin commentedThis code from the issue creator along with #4 works like a charm.
Here are detailed instructions if your fbOAuth login takes place after couple of clicks and or page refreshes.
1) add the code as mentioned above to fboauth/includes/fboauth.fboauth.inc file
so the login function looks like this (added the if module boost exists)
2) Now visit boost config page
/admin/config/system/boost
and add the "fboauth/*" as exception.
BOOST CACHEABILITY SETTINGS
Cache specific pages
(default checked) All pages except those listed
fboauth/*
3) Clear cache - /admin/config/development/performance
Enjoy!
It would be great if module creator added this functionality as a lot of sites use boost.
Comment #7
imoreno CreditAttribution: imoreno commented+1 for testing and integrate this one, boost is very popular and useful, and D7 might be very slow and resource consuming with large sites.
This could be a great help for large sites with a lot of anonymous users.
Please consider it,
BR
Itzhak
Comment #9
irishdan CreditAttribution: irishdan commentedagreed. would love to see this in the next release. If we create a patch and test, is it possible to get it commited to the next release?
Comment #10
vflirt CreditAttribution: vflirt commentedHi
I am reopening this issue as this is not fixed in anyway in the code.
Comment #11
grn CreditAttribution: grn commentedPatch created by following the guidelines here: https://www.drupal.org/node/707484.
Enjoy.
BR GrN.dk
Comment #12
DarrellDuane CreditAttribution: DarrellDuane commentedIf someone who uses boost can validate that this works, I'll be happy to commit it.
Comment #13
grn CreditAttribution: grn commentedWell, I am using boost on multiple sites where it works great. Examples: HelePlanter.dk and IdeaHunters.dk.
But I am the on who submitted the patch, so maybe you will need approval from someone else.
BR. GrN.dk
Comment #14
glynster CreditAttribution: glynster commentedYes works like a charm, please commit as well!
I just upgraded to the new release and realized this was not included.
Comment #15
DarrellDuane CreditAttribution: DarrellDuane commentedok!
Comment #16
DarrellDuane CreditAttribution: DarrellDuane commentedOk, this has been added to the latest development version. Sorry I didn't add the right commit message this time.