ptejada / ApePubSub

A PubSub JavaScript Framework for the APE Server
MIT License
10 stars 6 forks source link

Channel property update #20

Closed lcharette closed 10 years ago

lcharette commented 10 years ago

I know it's possible to do server side, but is it possible to edit a channel property client side, in the same way we can do it with user properties using user.update({ ... }); ?

ptejada commented 10 years ago

No. Did not added support for security reasons. It would not be save to allow any connected user to update a channel's properties. On May 4, 2014 4:38 PM, "Louis Charette" notifications@github.com wrote:

I know it's possible to do server side, but is it possible to edit a channel property client side, in the same way we can do it with user properties using user.update({ ... }); ?

— Reply to this email directly or view it on GitHubhttps://github.com/ptejada/ApePubSub/issues/20 .

lcharette commented 10 years ago

Yes, you're right, forget about that.

At first I thought it wouldn't matter since if I write a custom ssjs, the result (and security issue) would be the same. But the truth is that custom script would only update to the properties I want to be updated. Indeed, I don't want a client being able to update a channel name for instance.

Anyway, after further brainstorming with myself, I realized whatever I do, I need I custom ssjs for what I'm working on. So it doesn't mater in the end.

Case closed.