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

      Studio does not include a Version Specifier ("version: 1") in the generated LDIF export
      (see http://www.novell.com/coolsolutions/feature/5997.html -> LDIF Content Records)

      E.g. the Novell LDAP classes for Java will not accept an LDIF that does not contain the Version Specifier.

      Perhaps Studio could make this optional, e.g. by a setting on the Text formats preferences page:
      [x] add Version Specifier

        Activity

        Hide
        Stefan Seelmann added a comment -

        Thanks Jürgen for the report.

        I would suggest to add this setting to the export wizard's 2nd page. I think this setting is only valid for the LDIF export. When copying an entry as LDIF or when printing the modification logs this setting makes no sense. Or do I miss something?

        Show
        Stefan Seelmann added a comment - Thanks Jürgen for the report. I would suggest to add this setting to the export wizard's 2nd page. I think this setting is only valid for the LDIF export. When copying an entry as LDIF or when printing the modification logs this setting makes no sense. Or do I miss something?
        Hide
        Jürgen Weber added a comment -

        Hi Stefan,

        RFC 2849 requires a version-spec (http://www.ietf.org/rfc/rfc2849.txt), so I think it should always be included.
        And it might puzzle users if print output is different to export output.

        Contrary to what I wrote earlier, I am not sure if there should a checkbox at all, as the RFC requires the version spec.

        Or maybe there should be a checkbox for not having the version spec
        [ ] Omit Version Specification

        Show
        Jürgen Weber added a comment - Hi Stefan, RFC 2849 requires a version-spec ( http://www.ietf.org/rfc/rfc2849.txt ), so I think it should always be included. And it might puzzle users if print output is different to export output. Contrary to what I wrote earlier, I am not sure if there should a checkbox at all, as the RFC requires the version spec. Or maybe there should be a checkbox for not having the version spec [ ] Omit Version Specification
        Hide
        Stefan Seelmann added a comment -

        Hi Jürgen,

        you are totally right, the version spec is required at the beginning of an LDIF file, so it should always be added.

        What I meant with "no sense" was the following:

        • If you right-click on an entry and select Advanced->Copy Entry as LDIF only one LDIF record is copied. I use this quite often to extract some sample data from a directory to an LDIF file. I copy one entry and paste it into an LDIF file, then the second and the third and so on. In that case I don't want the version spec on top of each LDIF record.
        • The modification logs is a rolling log file. On each modification another LDIF fragment is added to the log. So I think in that case it makes no sense to add the version spec in front of each LDIF fragment.

        So IMHO the version spec should be added when producing complete LDIF files, but not for LDIF fragments.

        Show
        Stefan Seelmann added a comment - Hi Jürgen, you are totally right, the version spec is required at the beginning of an LDIF file, so it should always be added. What I meant with "no sense" was the following: If you right-click on an entry and select Advanced->Copy Entry as LDIF only one LDIF record is copied. I use this quite often to extract some sample data from a directory to an LDIF file. I copy one entry and paste it into an LDIF file, then the second and the third and so on. In that case I don't want the version spec on top of each LDIF record. The modification logs is a rolling log file. On each modification another LDIF fragment is added to the log. So I think in that case it makes no sense to add the version spec in front of each LDIF fragment. So IMHO the version spec should be added when producing complete LDIF files, but not for LDIF fragments.
        Hide
        Stefan Seelmann added a comment -

        I just implemented a simple fix, the version spec is always added in LDIF export.

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

        Show
        Stefan Seelmann added a comment - I just implemented a simple fix, the version spec is always added in LDIF export. Fixed in trunk: http://svn.apache.org/viewvc?rev=772403&view=rev
        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