流星：无法存储来自外部api的数据

Question

I need to store IP data of registered user. I'm using telize api for retrieving IP and collection hooks for updating after inserting new registered user.

server/methods.js

Meteor.methods({
  // The method expects a valid IPv4 address
  'geoIp': function () {
    // Construct the API URL
    this.unblock();
    var apiUrl = 'http://www.telize.com/geoip/';
    // query the API
    var response = HTTP.get(apiUrl);
    Meteor.users.update(
          {_id: this.userId}, 
          {$set: {ipdata: response}}
      )
  }
});

lib/schemas.js

  ipdata: {
    type: Object,
    optional: true
  }

server/hooks.js

Meteor.users.after.insert(function (userId, doc) {
  Meteor.call('geoIp');
});

These code resulting no error, successfully registering new user, but fail to store ipdata

Anyone have figured out what's wrong with my code?

thank You very much..

Answer 1

I'm pretty sure this.userId is not set in this case. You're calling Meteor.call() server-side and not from another method's body, so the called method has no context.

Instead, you'd probably want to just have a regular function for this. Eg

addGeoIp = function(userId, ip){
    var response = HTTP.get(apiUrl);
    Meteor.users.update(userId, {$set: {ipdata: response}});
}

Meteor.users.after.insert(function (userId, doc) {
    addGeoIp(userId, ip);
});

Also, as Mark said, http://www.telize.com/geoip/ on its own would just return the server's IP, not the client's IP. So you need to pass the client's IP into the API to get the location.