Skip to content
This repository has been archived by the owner on Oct 11, 2022. It is now read-only.

replace the invocation with the real one #118

Open
wants to merge 1 commit into
base: devel
Choose a base branch
from

Conversation

cxdongjack
Copy link

  • to support the Meteor.userId() in the method

- to support the Meteor.userId() in the method
@dotansimha
Copy link

👍


var handlerArgs = options.getArgsFromRequest(req);
var handlerReturn = handler.apply(methodInvocation, handlerArgs);
connection: {},
Copy link

@dotansimha dotansimha May 10, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can use the req as the connection here, so the Meteor methods will have access for the HTTP request object.

@dpatte
Copy link

dpatte commented May 22, 2017

this package resolved all my Simple:Rest issues

@rijk
Copy link

rijk commented Sep 25, 2018

To get this to work, I also had to add 'ddp-common' in the package.js file:

  api.use([
    'check',
    'ddp',
    'ddp-common', // <--
    'ejson',
    'meteor',
    'mongo',
    'simple:[email protected]',
    'underscore',
    'webapp',
  ], 'server');

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants