Needless code in AuthorizationManager
The AuthorizationManager has a getInstance() method and private constructor, which makes it a singleton.
Unfortunately, it has only static data and methods so the creation of any instance is completely pointless. It is a utility class (i.e. static), and should not be singleton. The getInstance() method and instance member should be removed.
Removed getInstance(), instance variable, and constructor.
Agreed, and is a quick fix. I found no usages of the extraneous getInstance() method.