Skip to content
This repository has been archived by the owner on Sep 10, 2020. It is now read-only.

[authentication] Add RadiusToken #232

Closed
nemesifier opened this issue Apr 19, 2019 · 2 comments · Fixed by #237
Closed

[authentication] Add RadiusToken #232

nemesifier opened this issue Apr 19, 2019 · 2 comments · Fixed by #237

Comments

@nemesifier
Copy link
Member

The idea of creating a temporary token to use as a password in captive portals is very handy, but now we are using the rest framework Token model to perform the authorization in freeradius, but in order to completely separate the authentication of a user on the frontend apps that consume the openwisp API and the authorization of users in the wifi service, we should introduce a new RadiusToken model that will be dedicated to this purpose, tokens will be disposable: they are generated during authentication of the user and once they are used (in the freeradius authorize view) they are thrown away.

@nemesifier nemesifier added this to To do (general) in OpenWISP Contributor's Board via automation Apr 19, 2019
@nemesifier nemesifier added this to To do in OpenWISP Priorities for next releases via automation Apr 26, 2019
@nemesifier nemesifier added the Important Higher priority or release blocker label May 1, 2019
@Vivekrajput20
Copy link
Contributor

Hi Fed
I have doubts regarding what we want to achieve here.
Currently we are using rest framework token model to authorize in freeradius and then the same token model is used in openwisp radius for the authentication. Now what we want to do is to keep using the rest framework token model for authorization in free radius and create a Radius token model that will inherit rest framework token model and use this model in the openwisp for authentication purpose.
Is that what we want?
Thanks
Vivek

@nemesifier
Copy link
Member Author

If there's an abstract model in DRF we can inherit from then yes, otherwise we can make a new one.

Vivekrajput20 added a commit to Vivekrajput20/django-freeradius that referenced this issue May 24, 2019
Vivekrajput20 added a commit to Vivekrajput20/django-freeradius that referenced this issue May 26, 2019
Vivekrajput20 added a commit to Vivekrajput20/django-freeradius that referenced this issue May 26, 2019
Vivekrajput20 added a commit to Vivekrajput20/django-freeradius that referenced this issue May 26, 2019
Vivekrajput20 added a commit to Vivekrajput20/django-freeradius that referenced this issue May 26, 2019
@atb00ker atb00ker moved this from To do (general) to To do (python & django) in OpenWISP Contributor's Board May 28, 2019
Vivekrajput20 added a commit to Vivekrajput20/django-freeradius that referenced this issue Jun 7, 2019
Vivekrajput20 added a commit to Vivekrajput20/django-freeradius that referenced this issue Jun 7, 2019
Vivekrajput20 added a commit to Vivekrajput20/django-freeradius that referenced this issue Jun 7, 2019
Vivekrajput20 added a commit to Vivekrajput20/django-freeradius that referenced this issue Jun 7, 2019
Vivekrajput20 added a commit to Vivekrajput20/django-freeradius that referenced this issue Jun 14, 2019
Vivekrajput20 added a commit to Vivekrajput20/django-freeradius that referenced this issue Jun 14, 2019
Vivekrajput20 added a commit to Vivekrajput20/django-freeradius that referenced this issue Jun 14, 2019
Vivekrajput20 added a commit to Vivekrajput20/django-freeradius that referenced this issue Jun 24, 2019
Vivekrajput20 added a commit to Vivekrajput20/django-freeradius that referenced this issue Jun 29, 2019
Vivekrajput20 added a commit to Vivekrajput20/django-freeradius that referenced this issue Jun 29, 2019
OpenWISP Contributor's Board automation moved this from To do (python & django) to Done Jul 5, 2019
OpenWISP Priorities for next releases automation moved this from To do to Done Jul 5, 2019
nemesifier added a commit that referenced this issue Jul 5, 2019
[authentication] Add RadiusToken #232
2stacks pushed a commit to 2stacks/django-freeradius that referenced this issue Nov 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement feature-request Important Higher priority or release blocker
Development

Successfully merging a pull request may close this issue.

2 participants