-
Notifications
You must be signed in to change notification settings - Fork 166
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #65 from hosannahighertech/master
Add JWT Support
- Loading branch information
Showing
2 changed files
with
171 additions
and
118 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
6d4b6be
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@filsh @mtangoo
This is throwing exception on line:
Trying to get $name = 'user_credentials' from the container.
Exception:
"message": "Class user_credentials does not exist",
6d4b6be
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How do you install? Please post full error.
6d4b6be
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@manoelt
6d4b6be
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mtangoo, what exactly do you want to know?
It is installed via composer and was working fine until today after update.
6d4b6be
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you post:
6d4b6be
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Stack:
Module is the last version.
Config:
6d4b6be
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mtangoo
Could you explain why are you getting 'user_credentials' from container? Where this was set on the container?
6d4b6be
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's my reason to ask tgat you post Module which unfortunately you didn't
6d4b6be
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also I can see you define User credentials grant bur nowhere you define its respective Storage (the user class)
Please see the readme again
6d4b6be
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mtangoo
As I said, the Module is the same as https://github.com/Filsh/yii2-oauth2-server/blob/master/Module.php
What are you saying about "respective Storage"?
6d4b6be
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok. I see about config. Unfortunately I can't help without posting YOUR Module file. Though you say its tge same as that one I doubt it's the case
6d4b6be
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LOL. So I am posting my Module, whitch is the same, as I said. So now please don't say that I am lying. :)
6d4b6be
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@filsh
Is this version working fine for you?
I tried 2.0.1 but I need a change that I suggested (table_prefix) and is not on that version. :(
6d4b6be
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well can you check what was added to Server. php and bootstrap files prior to two PR that I created? I must have added something there
6d4b6be
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mtangoo Why did you remove the method
public function createStorages()
?6d4b6be
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No I didn't
6d4b6be
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
db390f0
Here it is. You removed the method.