Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error occurred while sending sms TypeError: Cannot read property 'profile' of undefined #3356

Closed
aaronjudd opened this issue Nov 28, 2017 · 3 comments · Fixed by #3605
Closed
Labels
bug For issues that describe a defect or regression in the released software

Comments

@aaronjudd
Copy link
Contributor

Orders can complete, but are delayed while this error occurs.

23:46:32.639Z  WARN Reaction:
  Error: error occured while sending sms TypeError: Cannot read property 'profile' of undefined
      at DDPCommon.MethodInvocation.smsSend (imports/plugins/included/sms/server/methods/sms.js:52:30)
      at packages/check.js:128:16
      at Meteor.EnvironmentVariable.EVp.withValue (packages/meteor.js:1184:15)
      at Object._failIfArgumentsAreNotAllChecked (packages/check.js:127:41)
      at maybeAuditArgumentChecks (packages/ddp-server/livedata_server.js:1765:18)
      at DDP._CurrentMethodInvocation.withValue (packages/ddp-server/livedata_server.js:1686:15)
      at Meteor.EnvironmentVariable.EVp.withValue (packages/meteor.js:1184:15)
      at resolve (packages/ddp-server/livedata_server.js:1684:36)
      at new Promise (<anonymous>)
      at Server.applyAsync (packages/ddp-server/livedata_server.js:1683:12)
      at Server.apply (packages/ddp-server/livedata_server.js:1622:26)
      at Server.call (packages/ddp-server/livedata_server.js:1604:17)
      at DDPCommon.MethodInvocation.notificationSend (imports/plugins/included/notifications/server/methods/notifications.js:57:16)
      at packages/check.js:128:16
      at Meteor.EnvironmentVariable.EVp.withValue (packages/meteor.js:1184:15)
      at Object._failIfArgumentsAreNotAllChecked (packages/check.js:127:41)
      at maybeAuditArgumentChecks (packages/ddp-server/livedata_server.js:1765:18)
      at DDP._CurrentMethodInvocation.withValue (packages/ddp-server/livedata_server.js:719:19)
      at Meteor.EnvironmentVariable.EVp.withValue (packages/meteor.js:1184:15)
      at DDPServer._CurrentWriteFence.withValue (packages/ddp-server/livedata_server.js:717:46)
      at Meteor.EnvironmentVariable.EVp.withValue (packages/meteor.js:1184:15)
      at Promise (packages/ddp-server/livedata_server.js:715:46)

Steps to reproduce the behavior

Placed order with Shippo + PayPal Express + Stripe.

Configured with rc-config, nothing done specifically to setup SMS.

Place an order as a user.
Login as admin to fullfill order, witness server console.

Versions

Node: 6.11.1
NPM: 5.5.1
Meteor Node: 8.8.1
Meteor NPM: 5.4.2
Reaction CLI: 0.23.2
Reaction: 1.6.0
Reaction branch: meteor-1.6
Docker: 17.09.0-ce
@aaronjudd aaronjudd added the bug For issues that describe a defect or regression in the released software label Nov 28, 2017
@aaronjudd aaronjudd mentioned this issue Nov 29, 2017
@iakhator iakhator self-assigned this Nov 30, 2017
@iakhator
Copy link

iakhator commented Dec 4, 2017

@aaronjudd , I can't seem to reproduce this error following the guidelines above. Maybe I am missing something.

@iakhator iakhator assigned iakhator and unassigned iakhator Dec 6, 2017
@codejockie
Copy link
Contributor

codejockie commented Dec 19, 2017

@iakhator I think this issue is reproducible only when a user adds billing/shipping info without including a phone number. The error is trying to send an sms to an undefined phone book value.

@iakhator
Copy link

iakhator commented Dec 19, 2017

@codejockie , I have tried this still can't get the logged error. I think I will try to find another way to reproduce this.
so, what I basically did was to delete the entire user from the database before capturing payment.
Though the error doesn't shoe after capturing, it only shows when the all items packed button is clicked

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug For issues that describe a defect or regression in the released software
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants