diff options
author | Azul <azul@riseup.net> | 2016-08-29 10:26:54 +0200 |
---|---|---|
committer | Azul <azul@riseup.net> | 2016-08-29 10:34:17 +0200 |
commit | becd26b0bdf44b3625caaa7643914d0379a4fea5 (patch) | |
tree | 8f6e5ddf548ce9262a7cabac76ed2f4e3873be22 /lib/nickserver/request_handlers/fingerprint_handler.rb | |
parent | 58d687f927aabc8aa2fdfc46132cb71706bdde46 (diff) |
refactor: let handlers check if they are applicable
Instead of testing the preconditions for each handler in the dispatcher
the dispatcher hands a request to one handler after the other until one of
them responds.
This is similar to the Chain of Responsibility patter but we iterate over the
'handler_chain' array instead of a linked list.
To change the order of handlers or add other handlers change the array in the
handler_chain function.
Diffstat (limited to 'lib/nickserver/request_handlers/fingerprint_handler.rb')
-rw-r--r-- | lib/nickserver/request_handlers/fingerprint_handler.rb | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/lib/nickserver/request_handlers/fingerprint_handler.rb b/lib/nickserver/request_handlers/fingerprint_handler.rb index 3202c41..3c04fcd 100644 --- a/lib/nickserver/request_handlers/fingerprint_handler.rb +++ b/lib/nickserver/request_handlers/fingerprint_handler.rb @@ -6,6 +6,13 @@ module Nickserver class FingerprintHandler def call(request) + return unless request.fingerprint + handle_request(request) + end + + protected + + def handle_request(request) fingerprint = request.fingerprint if fingerprint.length == 40 && !fingerprint[/\H/] source = Nickserver::Hkp::Source.new |