Don't use $_REQUEST['q'], use Drupal's built in request_path() instead (or you may get redirect loops)

This has been fixed already

Cheers

@dgtlmoon

I'm sorry if I ofended you with the way my comment was posted, but I feel telling someone that one is wrong with no further explanation of why one is wrong is a bit rude.

As per the "wrong", I insist, it has been fixed.
I think you are looking at the master branch and nod the 7.x-1.x branch.
Last update to master was 20 months ago, last update to 7.x-1.x was 5 weeks ago.

Indeed, master branch is still using $_REQUEST['q'] in many places, but that branch is not used for any releases.

Cheers

I got an infinite loop because my $_REQUEST['q'] was empty.
The globalredirect_request_path function always return an empty path :

function globalredirect_request_path() {
  if (request_uri()) {
    if (isset($_REQUEST['q'])) {
      $path = $_REQUEST['q']; // Return empty
    }
...

And this cause an infinite loop because of this code in the globalredirect_init function :

if ($request_path != $prefix . $alias) { // Request_path is empty due to the function above so it's never equal to $prefix . $alias
    if (str_replace($prefix . $alias, '', $request_path) != '/' || $settings['deslash']) {
      globalredirect_goto($alias, $options); // And redirecting to the same page over and over again
    }
  }

So idon't know about your branch but the actual dev and stable module still have the $_REQUEST['q'].

FYI I stopped this loop by changing the testing condition of $_REQUEST['q'] (until a better fix) :

function globalredirect_request_path() {
  if (request_uri()) {
    if (isset($_REQUEST['q']) && !empty($_REQUEST['q'])) {
      $path = $_REQUEST['q'];
    }
...

Tested, it is already fixed in 7.x-1.5 release. So closing this issue.