The hook node load in the module http auth is getting problems using the second parameter. Sometimes isn't passing on this array the value for site and we lose the config. I changed this function. We skip this condition and I add the validation in the foreach. I think at the end is the same and isn't bad for the performance.

CommentFileSizeAuthor
bug_node_load_using_content_types.patch935 bytesfastangel

Comments

fastangel created an issue. See original summary.

shreddedbacon’s picture

shreddedbacon commented

This patch fixes the issue I was experiencing

jon pugh’s picture

jon pugh
he/him
Primary language English
Location Newburgh, NY
commented
Priority: Normal » Critical
Status: Needs review » Reviewed & tested by the community

Marking critical because this morning l module is effectively broken without this patch!

Will commit and release when I get to my office in ~3 hours, unless Helmo gets to it before I do ;)

jon pugh’s picture

jon pugh
he/him
Primary language English
Location Newburgh, NY
commented

I've added a Behat test to verify the problem. Confirmed failure is here: https://travis-ci.org/opendevshop/devmaster/jobs/399770975#L2901

About to push a commit with the patch applied.

  • Jon Pugh committed a33b8d6 on 7.x-3.x authored by fastangel 
    Issue #2982704 by fastangel: Bug loading info for http auth
jon pugh’s picture

jon pugh
he/him
Primary language English
Location Newburgh, NY
commented
Status: Reviewed & tested by the community » Fixed

Committed. Thanks Jacinto!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.