Skip to content

Conversation

@utsav0209
Copy link

@utsav0209 utsav0209 commented Oct 12, 2021

Introduces support Paseto authentication mentioned here

Work Done

  • Setup basic configuration for JPaseto

WIP

  • Use JPaseto library for authentication

@utsav0209 utsav0209 changed the title Feature/j paseto 504 Add suppoer for Paseto Authentication Oct 12, 2021
@utsav0209 utsav0209 changed the title Add suppoer for Paseto Authentication Add support for Paseto Authentication Oct 12, 2021
@sdelamo
Copy link
Contributor

sdelamo commented Oct 12, 2021

@utsav0209 We are going to need at least implementation of:

  • TokenGenerator
  • TokenValidator

@utsav0209 utsav0209 force-pushed the feature/JPaseto-504 branch from ddb3238 to 7fa3bfb Compare October 12, 2021 16:33
@utsav0209
Copy link
Author

utsav0209 commented Oct 13, 2021

@sdelamo I have added implementation for TokenGenerator and for TokenValidator.

Please, let me know if I am going in the correct direction, as well as what should be the next?

I am yet to add any kind of support for footer of paseto tokens, will be focusing on that next as of now.

@sdelamo
Copy link
Contributor

sdelamo commented Oct 15, 2021

I am fleshing the implementation based on your progress. I will update you next week.

@utsav0209
Copy link
Author

utsav0209 commented Oct 15, 2021

@sdelamo I will wait for your feedback. Meanwhile, I have been trying to figure out how to support Public/Private Key Pairs for PUBLIC perpose of Paseto. Have we supported such feature somewhere else?

@sdelamo
Copy link
Contributor

sdelamo commented Oct 16, 2021

@sdelamo I will wait for your feedback. Meanwhile, I have been trying to figure out how to support Public/Private Key Pairs for PUBLIC perpose of Paseto. Have we supported such feature somewhere else?

I think I have an example for Public/Private Key. I will share on Monday.

@sdelamo
Copy link
Contributor

sdelamo commented Oct 18, 2021

I've moved the progress here: #813

As you can see in that PR I have moved some logic from jwt to security. There is still more to do and I am still not sure If we can do this without entering a breaking change.

I will comment more about the changes in that PR later this week.

If you want to work on this please create PRs with paseto branch as base.

@sdelamo sdelamo closed this Oct 18, 2021
@utsav0209
Copy link
Author

Hi @sdelamo, sure, I will start putting PRs to paseto branch. Should I wait for you to make more changes or continue working on something on it?

@sdelamo
Copy link
Contributor

sdelamo commented Oct 18, 2021

This week, I don't have more time to work on #813 Thus, if you have time check it out and create an PR you find appropriate. If you have any questions or do you want to discuss anything please, ask them in #813

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants