Support other authentication backends

[aleffert]

I'd like to use https://github.com/James1345/django-rest-knox instead of the built in TokenAuthentication backend. I'm fine using the existing system for the initial 6-digit codes, this change would only be for the final token generated after a successful callback.

I have an idea about what this PR would look like and I think it's pretty small, but wanted to talk it out before spending time on implementation.

In particular, it looks like https://github.com/aaronn/django-rest-framework-passwordless/blob/master/drfpasswordless/views.py#L131 is the only place a 6-digit code gets turned into a token, so I'd like to abstract that out.

This would entail:

1. Adding a new setting PASSWORDLESS_AUTH_TOKEN_CREATOR which is a string that will get mapped to a string by the settings infrastructure. This function would take a user and return (string, bool) matching the existing call. This would default to a function containing the current code.

2. Update the call linked above to invoke whatever function is bound to PASSWORDLESS_AUTH_TOKEN_CREATOR .

Does this sound good?

Thanks for a great project!

[aaronn]

Yeah, I'm actually interested in using Knox myself– so I would be down for a PR that implements a way to swap that out.

I don't understand your implementation though– in 1. did you mean a string that will get mapped to a function? Which existing call returns (string, bool)? Basically though I'm fine with what you're proposing as long as it still works with DRF tokens default out of the box (which it sounds like it will).

[aleffert]

yeah, I meant the string will get mapped to a function. I'll try to whip up a PR in the next few days

[bwesen]

We're interested in the same, but we need it to add a refresh token in between. The only Django module I could find that does this is the Oauth Toolkit, so I guess I need a way to get drfpasswordless to ask oauth toolkit for a token (instead of the built-in Token directly). So, same issue, other use-case :) Would be cool if we can support a generic mechanism.

[charleshan]

Any updates for this? Looks like @aleffert made a pull request (#21) but never got merged.

[aaronn]

Should be in 1.3.9+

[Terkea]

Could anyone provide an example where using Knox Tokens?

[eluce2]

@aaronn I'm trying to integrate this with Knox, but after implementing my custom create_authentication_token method that generated the knox token, I get an error in the view:

AttributeError: 'str' object has no attribute '__dict__'

Would you mind sharing how you were able to implement this with Knox? What is it that Knox needs to create for this to work?