sparcs-kaist / sparcssso

Single Sign On System for SPARCS
https://sparcssso.kaist.ac.kr/
MIT License
18 stars 8 forks source link

Unified rate limit module #139

Closed leeopop closed 7 years ago

leeopop commented 8 years ago

시간날때 한번 만들어보는건? https://django-ratelimit.readthedocs.io/en/latest/ 이거 참고해서 해도 좋은데 얼핏 보니까 저기 per-user limit 가 되는지 모르겠네

leeopop commented 8 years ago

이미 요청이 있긴 했었네 https://github.com/jsocol/django-ratelimit/issues/52

여기 기여해보고 반영되면 써보는걸로?

samjo-nyang commented 8 years ago

오 좋은 패키지네요 근데 rate limit을 어디다가 써야 하나요...?

leeopop commented 8 years ago

그냥 모든 api에 적용시켜놓고 나중에 돈내면 풀어주는.... 아 별론가 ㅋㅋ

samjo-nyang commented 8 years ago

네....? SSO API는 SPARCS 서비스 서버들이 사용하는거라서요.... OTL팀에서 돈 뺏어오기 읍읍

leeopop commented 8 years ago

http://django-ratelimit.readthedocs.io/en/latest/rates.html#callables

이미 구현되어있네

samjo-nyang commented 8 years ago

근데 이거 진짜 필요한가요...? @leeopop