-
Notifications
You must be signed in to change notification settings - Fork 6
Open
Labels
difficulty: 🙂🙂Below average taskBelow average taskscope: 🛠️it's job for developersit's job for developerstype: refactorno change in functionality, but in mechanizm under the hood, improving performance or securityno change in functionality, but in mechanizm under the hood, improving performance or security
Description
In WebSocketSplendorServer class there is the reactions field (See the codeblock below). It maps type of request (string) to the appropriate reaction class (request handler). It's safer to use enum as a key, there is created ReactionType enum for this purpose.
Changing this involves tinkering with ReactionManager class and it uses reflection a lot. It's an relatively easy task, but you have to know reflection a bit.
Lines 24 to 27 in da9b9de
| public class WebSocketSplendorServer extends WebSocketServer { | |
| private Map<String, Class<? extends Reaction>> reactions; | |
Metadata
Metadata
Assignees
Labels
difficulty: 🙂🙂Below average taskBelow average taskscope: 🛠️it's job for developersit's job for developerstype: refactorno change in functionality, but in mechanizm under the hood, improving performance or securityno change in functionality, but in mechanizm under the hood, improving performance or security