Improve logging when posting new users to courses.mooc.fi

Author: Redande

app/models/user.rb

@@ -238,6 +238,7 @@ def update_password_via_courses_mooc_fi(old_password, new_password)
   end
 
   def post_new_user_to_courses_mooc_fi(password)
+    Rails.logger.info("Posting new user #{self.email} to courses.mooc.fi")
     create_url = SiteSetting.value('courses_mooc_fi_create_user_url')
 
     conn = Faraday.new do |f|
@@ -260,26 +261,26 @@ def post_new_user_to_courses_mooc_fi(password)
       data = response.body
 
       unless data.is_a?(Hash) && data['user'].present?
-        Rails.logger.error("Creating user in courses.mooc.fi returned unexpected response for user #{self.id}: #{data}")
-        raise "Creating user in courses.mooc.fi failed for user #{self.id}"
+        Rails.logger.error("Creating user in courses.mooc.fi returned unexpected response for user #{self.email}: #{data}")
+        raise "Creating user in courses.mooc.fi failed for user #{self.email}"
       end
 
       unless data['password_set']
-        Rails.logger.warn("Password was not set for user #{self.id} in courses.mooc.fi")
+        Rails.logger.warn("Password was not set for user #{self.email} in courses.mooc.fi")
       end
 
-      Rails.logger.info("User #{self.id} successfully created in courses.mooc.fi")
+      Rails.logger.info("User #{self.email} successfully created in courses.mooc.fi")
       true
 
     rescue Faraday::ClientError => e
       Rails.logger.error(
-        "Creating user in courses.mooc.fi failed for user #{self.id}: #{e.response}"
+        "Creating user in courses.mooc.fi failed for user #{self.email}: #{e.response}"
       )
       false
 
     rescue => e
       Rails.logger.error(
-        "Unexpected error creating user in courses.mooc.fi for user #{self.id}: #{e.message}"
+        "Unexpected error creating user in courses.mooc.fi for user #{self.email}: #{e.message}"
       )
       false
     end