diff --git a/lib/diaspora/importer.rb b/lib/diaspora/importer.rb
index c0a7aa2557fe9ac4d7626c823de79407cdb66acb..dbe15825ae6b99cf2514b6c5e083243a458754a1 100644
--- a/lib/diaspora/importer.rb
+++ b/lib/diaspora/importer.rb
@@ -9,21 +9,21 @@ module Diaspora
       self.class.send(:include, strategy)
     end
     
-    def commit(user, person, aspects, people, posts)
+    def commit(user, person, aspects, people, posts, opts = {})
       filter = verify_and_clean(user, person, people, aspects, posts)
       #assume data is good
       
       # to go 
-      user.email = "tits@tits.tits"
-      user.password= "megatits@tits.tits"
-      user.password_confirmation = "megatits@tits.tits"
+      user.email = opts[:email]
+      user.password= opts[:password]
+      user.password_confirmation = opts[:pasword_confirmation]
 
      
      
       user.person = person
 
 
-      user.person.diaspora_handle = "obby@foo.com"
+      user.person.diaspora_handle = opts[:diaspora_handle] 
       
       user.visible_post_ids = filter[:whitelist].keys
 
@@ -46,7 +46,7 @@ module Diaspora
 
 
       people.each do |p|
-        p.save! #if filter[:people].include? person.id
+        p.save! if filter[:people].include? p.id.to_s
       end
     end
 
@@ -121,7 +121,7 @@ module Diaspora
 
   module Parsers
     module XML
-      def execute(xml)
+      def execute(xml, opts = {})
         doc = Nokogiri::XML.parse(xml)
 
         user, person = parse_user_and_person(doc)
@@ -130,7 +130,7 @@ module Diaspora
         posts = parse_posts(doc)
 
         user
-        commit(user, person, aspects, people, posts)
+        commit(user, person, aspects, people, posts, opts)
       end
 
       def parse_user_and_person(doc)
diff --git a/spec/lib/importer_spec.rb b/spec/lib/importer_spec.rb
index 7385c35b25f01c7be92df1d445ce052ad9afaf03..d243ae345d1f607310c2f21cd8d0521e2a4a8418 100644
--- a/spec/lib/importer_spec.rb
+++ b/spec/lib/importer_spec.rb
@@ -109,7 +109,11 @@ describe Diaspora::Importer do
 
     it 'should import a user' do
       pending
-      user = @importer.execute(@xml)
+      user = @importer.execute(@xml,
+                              :email => "bob@bob.com",
+                              :password => "bobbybob",
+                              :password => "bobbybob",
+                              :diaspora_handle => "bob@diaspora.com")
       user.class.should == User
     end
 
@@ -200,7 +204,11 @@ describe Diaspora::Importer do
         User.count.should == 0
         Person.count.should == 0
         
-        @importer.execute(@xml)
+        @importer.execute(@xml,
+                          :email => "bob@bob.com",
+                          :password => "bobbybob",
+                          :password => "bobbybob",
+                          :diaspora_handle => "bob@diaspora.com")
         
         User.count.should == 1
         n = User.first