glassfish
  1. glassfish
  2. GLASSFISH-6390

Validation Check: All CLIs that require classname should verify if the class can be loaded

    Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: V3
    • Fix Version/s: None
    • Component/s: command_line_interface
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: Sun

    • Issuezilla Id:
      6,390

      Description

      I tried this command:
      glassfish@~/WS/gf/v3.trunk.new$ asadmin create-auth-realm --classname=foo bar

      and it succeeded even when there is no class called foo.

        Activity

        Hide
        Sanjeeb Sahoo added a comment -

        I am very surprised that we don't do basic verification. I am reopening this bug
        as an RFE to do more verification at command execution time. Feel free to change
        the subject to reflect the scope of this issue.

        Show
        Sanjeeb Sahoo added a comment - I am very surprised that we don't do basic verification. I am reopening this bug as an RFE to do more verification at command execution time. Feel free to change the subject to reflect the scope of this issue.
        Hide
        ne110415 added a comment -

        Changing the issue per earlier discussions.

        Show
        ne110415 added a comment - Changing the issue per earlier discussions.
        Hide
        ne110415 added a comment -

        forgot to change component

        Show
        ne110415 added a comment - forgot to change component
        Hide
        janey added a comment -

        Reassign to Bill.

        Show
        janey added a comment - Reassign to Bill.
        Hide
        Tom Mueller added a comment -

        Cleared the Fix version field since this issue isn't going to be fixed in V3.

        Show
        Tom Mueller added a comment - Cleared the Fix version field since this issue isn't going to be fixed in V3.

          People

          • Assignee:
            Bill Shannon
            Reporter:
            Sanjeeb Sahoo
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated: