Specify all handlers using decorators instead of app config values?

[sloria]

There are currently two different ways to specify handlers:

• App config values: JWT_ENCODE_HANDLER, JWT_DECODE_HANDLER, JWT_PAYLOAD_HANDLER
• Decorators: JWT#user_handler, JWT#error_handler

Why not just have one mechanism for defining these handlers? Personally, the decorator syntax feels most natural:

@jwt.payload_handler
def make_payload(user):
    return {'user_id': user.id }

@jwt.user_handler
def get_user(payload):
    return User.get(payload['user_id']

This makes for a more consistent and cohesive API.

[sloria]

P.S. I would be glad to implement this if the maintainers believe this is a good idea.

[mattupstate]

@sloria I agree. I'm not so sure the configuration options are necessary.