Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.4.0
    • Fix Version/s: 1.5.0
    • Component/s: studio-ldapbrowser
    • Labels:
      None

      Description

      I have an entry which is a MD5 encoded password field.

      userpassword MD5 hashed password

      When I right click and select edit or "edit with" Password Editor or Hex editor nothing happens, no error message, no popup dialog.
      I tried to edit that field with JXplorer, it works fine.

        Activity

        Hide
        Stefan Seelmann added a comment -

        When you disable the "Show decorated values", can could see the raw "

        {MD5}

        ...." string. Is there something special with it? Could you share the raw string?

        Could you also check ".ApacheDirectoryStudio/.metadata/.log" for an exception please?

        Show
        Stefan Seelmann added a comment - When you disable the "Show decorated values", can could see the raw " {MD5} ...." string. Is there something special with it? Could you share the raw string? Could you also check ".ApacheDirectoryStudio/.metadata/.log" for an exception please?
        Hide
        Jürgen Weber added a comment -

        I don't see "Show decorated values", but copy as LDIF gives

        userpassword:: e01ENX3UmEKcl7cCxY0bg96sbigb

        (for password "studio")

        When I enter the password with Jarek Gawor's Browser, Studio can edit the field afterwards.

        LDIF is now

        userpassword:: e01ENX15VVJqUlZER21QNjkyY2hvMjVDTm5RPT0=

        When I enter the password with JXplorer and edit afterwards with Jarek 's Browser the password seems to start with

        {MD5}

        , but does not verify.

        So, I guess originally the password had a problem and JXplorer created the same problem (which?).

        Still, Studio should catch the Exception and cope with the situation (offer to enter a new password?).

        Stacktrace is

        java.lang.ArrayIndexOutOfBoundsException: -17
        at org.apache.commons.codec.binary.Base64.isBase64(Base64.java:137)
        at org.apache.commons.codec.binary.Base64.discardNonBase64(Base64.java:478)
        at org.apache.commons.codec.binary.Base64.decodeBase64(Base64.java:374)
        at org.apache.directory.studio.ldifparser.LdifUtils.base64decodeToByteArray(LdifUtils.java:154)
        at org.apache.directory.studio.ldapbrowser.core.model.Password.<init>(Password.java:116)
        at org.apache.directory.studio.ldapbrowser.core.model.Password.<init>(Password.java:94)
        at org.apache.directory.studio.valueeditors.password.PasswordDialog.<init>(PasswordDialog.java:148)
        at org.apache.directory.studio.valueeditors.password.PasswordValueEditor.openDialog(PasswordValueEditor.java:55)
        at org.apache.directory.studio.valueeditors.AbstractDialogValueEditor.activate(AbstractDialogValueEditor.java:159)
        at org.eclipse.jface.viewers.CellEditor.activate(CellEditor.java:899)
        at org.eclipse.jface.viewers.ColumnViewerEditor.activateCellEditor(ColumnViewerEditor.java:203)
        at org.eclipse.jface.viewers.ColumnViewerEditor.handleEditorActivationEvent(ColumnViewerEditor.java:443)
        at org.eclipse.jface.viewers.ColumnViewer.triggerEditorActivationEvent(ColumnViewer.java:680)
        at org.eclipse.jface.viewers.ColumnViewer.editElement(ColumnViewer.java:438)
        at org.eclipse.jface.viewers.TreeViewer.editElement(TreeViewer.java:1128)
        at org.apache.directory.studio.ldapbrowser.common.widgets.entryeditor.AbstractOpenEditorAction.activateEditor(AbstractOpenEditorAction.java:130)
        at org.apache.directory.studio.ldapbrowser.common.widgets.entryeditor.AbstractOpenEditorAction.run(AbstractOpenEditorAction.java:101)
        at org.apache.directory.studio.ldapbrowser.common.widgets.entryeditor.OpenBestEditorAction.run(OpenBestEditorAction.java:198)
        at org.apache.directory.studio.ldapbrowser.common.actions.proxy.BrowserActionProxy.run(BrowserActionProxy.java:259)
        at org.apache.directory.studio.ldapbrowser.common.widgets.entryeditor.OpenDefaultEditorAction.run(OpenDefaultEditorAction.java:126)

        Show
        Jürgen Weber added a comment - I don't see "Show decorated values", but copy as LDIF gives userpassword:: e01ENX3UmEKcl7cCxY0bg96sbigb (for password "studio") When I enter the password with Jarek Gawor's Browser, Studio can edit the field afterwards. LDIF is now userpassword:: e01ENX15VVJqUlZER21QNjkyY2hvMjVDTm5RPT0= When I enter the password with JXplorer and edit afterwards with Jarek 's Browser the password seems to start with {MD5} , but does not verify. So, I guess originally the password had a problem and JXplorer created the same problem (which?). Still, Studio should catch the Exception and cope with the situation (offer to enter a new password?). Stacktrace is java.lang.ArrayIndexOutOfBoundsException: -17 at org.apache.commons.codec.binary.Base64.isBase64(Base64.java:137) at org.apache.commons.codec.binary.Base64.discardNonBase64(Base64.java:478) at org.apache.commons.codec.binary.Base64.decodeBase64(Base64.java:374) at org.apache.directory.studio.ldifparser.LdifUtils.base64decodeToByteArray(LdifUtils.java:154) at org.apache.directory.studio.ldapbrowser.core.model.Password.<init>(Password.java:116) at org.apache.directory.studio.ldapbrowser.core.model.Password.<init>(Password.java:94) at org.apache.directory.studio.valueeditors.password.PasswordDialog.<init>(PasswordDialog.java:148) at org.apache.directory.studio.valueeditors.password.PasswordValueEditor.openDialog(PasswordValueEditor.java:55) at org.apache.directory.studio.valueeditors.AbstractDialogValueEditor.activate(AbstractDialogValueEditor.java:159) at org.eclipse.jface.viewers.CellEditor.activate(CellEditor.java:899) at org.eclipse.jface.viewers.ColumnViewerEditor.activateCellEditor(ColumnViewerEditor.java:203) at org.eclipse.jface.viewers.ColumnViewerEditor.handleEditorActivationEvent(ColumnViewerEditor.java:443) at org.eclipse.jface.viewers.ColumnViewer.triggerEditorActivationEvent(ColumnViewer.java:680) at org.eclipse.jface.viewers.ColumnViewer.editElement(ColumnViewer.java:438) at org.eclipse.jface.viewers.TreeViewer.editElement(TreeViewer.java:1128) at org.apache.directory.studio.ldapbrowser.common.widgets.entryeditor.AbstractOpenEditorAction.activateEditor(AbstractOpenEditorAction.java:130) at org.apache.directory.studio.ldapbrowser.common.widgets.entryeditor.AbstractOpenEditorAction.run(AbstractOpenEditorAction.java:101) at org.apache.directory.studio.ldapbrowser.common.widgets.entryeditor.OpenBestEditorAction.run(OpenBestEditorAction.java:198) at org.apache.directory.studio.ldapbrowser.common.actions.proxy.BrowserActionProxy.run(BrowserActionProxy.java:259) at org.apache.directory.studio.ldapbrowser.common.widgets.entryeditor.OpenDefaultEditorAction.run(OpenDefaultEditorAction.java:126)
        Hide
        Stefan Seelmann added a comment -

        Thanks Jürgen for the detailed analysis.

        Show
        Stefan Seelmann added a comment - Thanks Jürgen for the detailed analysis.
        Hide
        Stefan Seelmann added a comment -

        Fixed in trunk:
        http://svn.apache.org/viewvc?rev=823103&view=rev

        An invalid password hash is now catched and an appropriate message ("Invalid password hash") displayed in the password dialog.

        Show
        Stefan Seelmann added a comment - Fixed in trunk: http://svn.apache.org/viewvc?rev=823103&view=rev An invalid password hash is now catched and an appropriate message ("Invalid password hash") displayed in the password dialog.
        Hide
        Pierre-Arnaud Marcelot added a comment -

        Apache Directory studio version 1.5.0 has been released.

        Show
        Pierre-Arnaud Marcelot added a comment - Apache Directory studio version 1.5.0 has been released.

          People

          • Assignee:
            Stefan Seelmann
            Reporter:
            Jürgen Weber
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development