search

SysFera / vishnu

Modular and high-level middleware for tasks, files and information management in heterogeneous and distributed HPC environments
http://sysfera.github.com/vishnu.html
Other
4 stars 12 forks source link

Fixes for ticket #382, #337 and misc #405

Closed rchakode closed 10 years ago

rchakode commented 10 years ago

I updated the PR with some refactoring according to Haikel's comments

hguemar commented 10 years ago

Looks good :+1:

keoo commented 10 years ago

Pensez à compiler avec les tests :

Building CXX object TMS/test/src/unit/CMakeFiles/vishnu-tms-server-mock.dir///__/src/server/JobServer.cpp.o /home/keo/Bureau/depot_git_edf/vishnu/TMS/src/server/JobServer.cpp: In member function ‘void JobServer::recordJob2db()’: /home/keo/Bureau/depot_git_edf/vishnu/TMS/src/server/JobServer.cpp:598:43: erreur: ‘class Database’ has no member named ‘escapeData’

hguemar commented 10 years ago

Bien vu

keoo commented 10 years ago

Aucun service ne fonctionne si on fournit des charactères spéciaux dans des paramètres des services est-ce que la PR a été testé ? (par exemple la clé de session n'est jamais échappée). Juste échapper quelques paramètres n'est pas suffisant. Les identifiants ne sont pas échappés non plus (les userId/machineId/transferId)

rchakode commented 10 years ago

c'est possible qu'il y a eu des omissions. Quand tu dis aucun service, tu parles desquels exactement? même ceux à l'origine du bug?

keoo commented 10 years ago

Sur tout les services je peux faire (au moins) une injection SQL, je vais faire une PR pour corriger cela