Block access by IP

[jelen07]

Základní implementace byla přidaná, mám však pár bodů:

• Třída RemoteAccessManager má dva callbacky, onAllow() a onDeny(). Otázkou je zda-li je to takto dobře nebo metody handlerů volat přímo?
• S obecnám názvem Handler nejsem moc spokojený, je to jen obyčejná třída, s dvěmi metodami deny() a allow(), moc na "handlení" tady toho není. Možná by bylo lepší je rozdělit na Allow a Deny handler s jednou public metodou.
• Pokud v deny() vyhodím BadRequestException, je pro ní ještě brzo, neodchytí se a skončí to na 500.

Todo

• [ ] Povolit zápis pro rozsah IP
• [ ] Panel pro tracy?
• [x] Nezávazně na IP, přes "pass" v cookie, kvůli např. heroku ap.
• [x] Refaktor handleru na službu

[tg666]

1) Za mne napřímo. Dále bych z třídy implementující interface IAccessHandler udělal službu. Default se nastaví v CompilerExtension a v případě vlastní/jiné implementace se nahradí v metody beforeCompile() - podobně je udělaná registrace služby implementující IEnvironmentDetector. Mohlo by se totiž stát, že vlastní IAccessHandler bude přijímat v konstruktoru nějaký parametr, či jinou službu 🙂