There are many calls to header inside the module that needs to be changed to drupal_set_header(), see http://api.drupal.org/api/function/drupal_set_header/6 how this works.

Comments

hass’s picture

hass commented

Is there really a need to write this exit()'s after all the 401 status codes?

jkitching’s picture

jkitching commented
Status: Active » Closed (fixed)

header() calls changed to drupal_set_header().

I do think the exit() is necessary, otherwise all the Drupal site would be output below. But I have refactored to only exit() once and set the header for the 401 code once.

hass’s picture

hass commented
Status: Closed (fixed) » Fixed

So it is fixed

Status: Fixed » Closed (fixed)

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