I'd like to fix up HandlerProvider to be a bit more sane. Currently, it requires a Map<String, Class<? extends DataRequestHandler>>. ApiServlet then uses an Injector instance to create the classes.
This is an odd way to use Guice - changing HandlerProvider to vend a Map<String, Provider<? extends DataRequestHandler>> avoids any funky need for Injector instances. Any objections? I'm working on a patch to address this now. -- Adam
