The default `jwt_get_username_from_payload_handler()` might fail with other username field

[jgadelange]

When I updated my payload by editing the jwt_payload_handler() I all of a sudden got Payload invalid errors. This was due to the fact that I removed the username from the payload altogether, since in my app email is the username field.

I therefore believe the default jwt_get_username_from_payload_handler() should also use the get_username_field() function.

[blueyed]

Is the suggestion still valid?
Then a PR would be nice to have for it.

[jgadelange]

See PR #285, should I update it to be sure it will still work with the current version?

[blueyed]

Great, missed it.
Yes, please ensure it works - the best is to have a test for it.
Thanks in advance!

[ebotean]

I tried implementing this with no success. @jgadelange care to show an example?

[jgadelange]

@blueyed Will try to make some time to add the test

@ebotelho7 It is basically the change in the PR. If you add that function somewhere in your code and change the JWT_PAYLOAD_GET_USERNAME_HANDLER setting to point to that function you should be alright.