google-code-export / ruby-activeldap

Automatically exported from code.google.com/p/ruby-activeldap
Other
1 stars 1 forks source link

error when loading ldif files #3

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?

I'm trying to upload ldif to server. Code looks like this:

require 'active_ldap'
basedn="dc=test,dc=local"
ActiveLdap::Base.establish_connection(
  :host=>"127.0.0.1",
  :port=>9009,
  :base=>basedn,
  :binddn=>"cn=root,#{basedn}",
  :password_block=>Proc.new { 't4qra349' },
  :allow_anonymous=>false,
  :try_sasl=>false
)
ldiftxt=<<EOF
version: 1

dn: ou=users,#{basedn}
objectClass: top
objectClass: organizationUnit
ou: users
EOF
ldif=ActiveLdap::Ldif.parse(ldiftxt)

What is the expected output? What do you see instead?

It should parse object in given ldif. Intead I get:
/usr/lib/ruby/gems/1.8/gems/activeldap-0.9.0/lib/active_ldap/ldif.rb:27:in
`parse': version: 1 (ActiveLdap::LdifInvalid)

dn: ou=users,dc=test,dc=local
objectClass: top
objectClass: organizationUnit
ou: users
 is invalid LDIF: unsupported version: 1        from
/usr/lib/ruby/gems/1.8/gems/activeldap-0.9.0/lib/active_ldap/ldif.rb:46:in
`parse'
        from bin/test_ldap.rb:30

What version of the product are you using? On what operating system?
activeldap-0.9.0 on debian etch (ruby 1.8)

Please provide any additional information below.

Problem is in line 27 of ldif.rb - you compare integer against string. I
attached patch i diff format (it's my first patch ever :)

Original issue reported on code.google.com by marsza...@gmail.com on 12 Jan 2008 at 6:52

Attachments:

GoogleCodeExporter commented 9 years ago
LDIF parser on 0.9.0 isn't completed.
In trunk, LDIF parser will work well. :)

Original comment by kou...@gmail.com on 13 Jan 2008 at 1:20