Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

API for Mootiro Maps #91

Open
aleborba opened this issue Mar 11, 2013 · 4 comments
Open

API for Mootiro Maps #91

aleborba opened this issue Mar 11, 2013 · 4 comments

Comments

@aleborba
Copy link

Primeiros rascunhos de código para a API
https://github.com/CodingForChange/mootiro-maps-api

@acasimiro
Copy link
Contributor

@aleborba vi que começou a implementação. Bacana! =D

Viu, vc tá pensando em usar algum ORM ou vai escrever o SQL na unha mesmo?

@aleborba
Copy link
Author

Nessa primeira versão, onde só estou fazendo algumas experimentações, vou deixar na unha. Mas acho bacana abstrair isso de alguma forma depois :)

@acasimiro
Copy link
Contributor

Saquei, beleza!

Só um comentário que pensei aqui:
Eu não duvido de suas habilidades no SQL roots, mas essa abordagem estaria mais suscetível a ataques de injection. Não conheço muito as consequências desses ataques, mas penso que poderia ser desastroso para uma API pública!

Valeu, @aleborba! =)

@aleborba
Copy link
Author

Sim sim, concordo! o/

O ideal é trabalhar com isso bem abstrato e tratar essa segurança no ORM, retirar esse trabalho do core da API :)

Já vou até deixar uma issue aberta pra isso :)

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

No branches or pull requests

2 participants