Skip to content

Conversation

@daffl
Copy link
Member

@daffl daffl commented Apr 3, 2014

This is related to #44, #48 and #32 adding SocketIO handshake data as the service parameters like:

app.configure(feathers.socketio(function(io) {
  io.set('authorization', function (handshake, callback) {
    handshake.feathers.user = { name: 'David' };
  });
}));

With something like:

app.use('todos', {
  create: function(data, params, callback) {
    // When called via SocketIO:
    params.user // -> { name: 'David' }
  }
});

@daffl daffl added this to the 0.4.0 milestone Apr 3, 2014
@daffl daffl changed the title Adding SocketIO handshake data to service call parameters. Adding SocketIO handshake data to service call parameters Apr 3, 2014
daffl added a commit that referenced this pull request Apr 3, 2014
Adding SocketIO handshake data to service call parameters
@daffl daffl merged commit 04b3931 into master Apr 3, 2014
@daffl daffl deleted the socket-params branch April 3, 2014 23:26
daffl pushed a commit that referenced this pull request Aug 19, 2018
* chore(package): update dependencies

https://greenkeeper.io/

* docs(readme): add Greenkeeper badge

https://greenkeeper.io/

* Update to latest Semistandard
daffl pushed a commit that referenced this pull request Aug 21, 2018
* chore(package): update dependencies

https://greenkeeper.io/

* docs(readme): add Greenkeeper badge

https://greenkeeper.io/

* Update to latest Semistandard
daffl pushed a commit that referenced this pull request Aug 25, 2018
daffl added a commit that referenced this pull request Aug 25, 2018
* Update plugin generator to correspond with latest release

* Add mocha to Semistandard env
daffl added a commit that referenced this pull request Aug 29, 2018
daffl pushed a commit that referenced this pull request Aug 29, 2018
daffl added a commit that referenced this pull request Aug 29, 2018
daffl pushed a commit that referenced this pull request Aug 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant