Triage important tasks

[iiegor]

:arrow_up: xat-server@0.1.0

---

• [x] Correct load and handle of the user powers.
  • [x] Calculate power values (d4, d5, d6..)
  • [x] Create user powers attributes from user packet (p0, p1, p2..)
• [x] Correct user authentication (workers/user.coffee).
  • [x] Normal user authentication
  • [x] Guest user authentication
• [ ] Verify user identity when sending commands (#L17).
• [ ] Select only needed data (#L5).
• [ ] Add a task to compile coffee-script code for production.
• [x] Support for multiple queries using the same connection (#L30).
• [x] Ping the database before sending application:started event (src/bootstrap.coffee).
• [x] Limit messages; Fix bug using order desc and limit.
• [x] ⚠ Implement a packet builder instead of hardcoding the packets structure.
• [x] Bind services with the main handler class and avoid passing handler class each time.

[huumanoid]

Let's discuss. What is left to do with Correct user authentication?

NOTE:

class Test
  u: { m: 0 }

it's unsafe way to initialize bar. in this way, all instances of Test works with same copy of u

a = new Test
a.u.m++
b = new Test
console.log(b.u,m)

guess what? ;) this note related to iiegor/xat-server@9a476e6ac8ce741a644b5562e2ccb1a87170c173 It seems that it's not tempfix, it is the safe way to initialize 'user' field.

[iiegor]

@HuuMaNoID It seems that only remains to fix the authentication for guest users and probably complete some missing user packets, regarding that task.

wow, you are completely right about the initialization of the variables :fearful:

Non-related: I added you as a collaborator so this way we can make more easier the development. Only push directly to the repo very important and small changes, for the others create pull requests to discuss the changes.

[huumanoid]

Verify user identity when sending commands This task requires user rank implementation, isn't it?

Select only needed data How do you see it? Maybe we may pass required fields to getById function?

[iiegor]

@HuuMaNoID Yes, the user identity verification requires of the user rank. Regarding the second question, I think your idea it's perfect ;)