Ditch the Goto/Gosub methods - they usually result in code that's hard to debug.

Here's how to do what you need:
 Code:
If Not @USERID = 'Sysadmin'

  If INGROUP('Dir-entry-check')

    $Cmd = '"c:\Program Files\Internet Explorer\iexplore"'
    $Cmd = $Cmd + ' http://hww-kingsweb/secure/directory/authenticate/autotest.asp
    'run: ' $Cmd ? ; for verification
    run $Cmd

  EndIf

EndIf

Exit 0

Note the layered If statements, and the negative logic of the first If - if the user is NOT the SysAdm user, check if they are a member of a specific group. If they are, do stuff...

The If could be combined with an AND, like this
If Not @USERID='xxx' And InGroup('yyy')
but that can be a bit more complex to understand at first, even though it reduces the logic to a single If.

The advantage of the $Cmd and display is that you can copy the command from the screen and run it by pasting it into a command prompt - does it work? Then the script should, too.

I usually comment-out the Run/Shell statements and read what is going to happen - if it looks good, I uncomment the run command. When it works, I comment-out the display command. This is especially true when the commands can do something destructive, or otherwise modify data. For your example - launching a browser on a specific page - there's little danger to combining the display and run commands.

Glenn
_________________________
Actually I am a Rocket Scientist! \:D