Superpatterns Pat Patterson on the Cloud, Identity and Single Malt Scotch

18Nov/093

OpenSSO User Group Meetings in Northern Europe – Nov/Dec 2009

SupportRockAlthough I'm no longer as active in the OpenSSO community as I once was, some things still catch my eye - for example, news of a series of user group meetings across Northern Europe in late November and early December. OpenSSO experts Allan Foster, Jonathan Scudder, Steve Ferris and Victor Ake (not a blogger amongst them!?!?) will be presenting on OpenSSO-related topics ranging from monitoring to the Fedlet, via entitlements and OAuth, in Helsinki, Stockholm, Copenhagen, Oslo, London and Brussels. Seems like SupportRock might be a name to watch in the world of OpenSSO...

Comments (3) Trackbacks (0)
  1. Hi Pat,

    I was looking through your module and might’ve found a bug:
    http://pastie.org/823011

    // Need to parse name/value pairs, to get value for Drupal username
    attribute

    $lines = explode(“\n”, $response->data);
    reset($lines);

    foreach ($lines as $line) {
    if ($line == (‘userdetails.attribute.name=’ .
    OPENSSO_DRUPAL_USERNAME_ATTRIBUTE)) {

    // ‘current’ line holds attribute value
    // 28 points to character after ‘userdetails.attribute.value=’
    $name = substr(current($lines), 28);
    break;
    }
    next($lines); // <——— This was missing
    }

    return $name;
    } // function _opensso_get_name

    Thanks for writing this module, please upload it to drupal.org :) A lot
    of people can contribute and benefit from bug fixes while it's on
    drupal.org.

    Cheers,

    Yonas

  2. Hi Yonas,

    Thanks for the fix! I still have the developer role over at OpenSSO, so I should be able to commit it.

    However that code belongs to Oracle and I no longer work there, so, unfortunately, I don’t think I can push it to drupal.org :-(

    Cheers,

    Pat

  3. Actually, I just checked – PHP foreach doesn’t need next() – see http://php.net/manual/en/control-structures.foreach.php


Leave a comment

No trackbacks yet.