-
-
Notifications
You must be signed in to change notification settings - Fork 68
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WIP: Refresh a supabase example #60
base: master
Are you sure you want to change the base?
Conversation
chore: moved migrations to the db folder
refactor: parametrize compose versions with .env
|
refactor (ory): add types
chore (lint): run lintfix on original client code
Hey, thanks for looking into it! |
fix (db): bug in return values for shortify chore (db): upd readme
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.
This is looking pretty good!
Should I already review?
Could you please share the steps that you use to run the example now?
I would also update our existing tutorial after merging and we should improve the README - the current one is also lacking concise steps to run locally.
Thanks for your contribution!
Guten tag @vinckr WIP in PR stands for Work-in-Progress 👀 . I've not yet finished. Now updating a client. I'll remove the WIP and ping you when I'm ready Vielen dank 😎 |
@vinckr has smth changed recently? I can't instantiate an Ory Agent with:
Switched to ory/client v1.0.1 |
@drev74 yes the syntax has changed a bit, const ory = {
- v0alpha2: V0alpha2Api
+ identity: new IdentityApi(config)
+ frontend: new FrontendApi(config),
+ oauth2: new OAuth2Api(config),
} see the upgrade guide: its pretty new, so any feedback appreciated 🙏 we are also working on more docs |
feat (make): add shorts api test
@vinckr Could you pls help me a bit 👀 . I'm trying to implement a login-logout for a I added an allowed return URL for kratos here {
"code": 403,
"reason": "Unable to log out because the logout token in the URL query does not match the session cookie.",
"status": "Forbidden",
"message": "The requested action was forbidden"
} Any ideas 🤔 PS: I see a major bump to ORY v1. I'll try to bump all my stuff and try again 🙄 |
fix (kratos): add localhost to valid return URLs
Related Issue or Design Document
I'm trying to reproduce
kratos-keto-oathkeeper-supabase
without bumpingory
deps. I've simplified thecompose
by removing redundantpostgres
versions.However, I can't make it work having an issue with
keto-perms
:Original impl doesn't work for me as well.
Could anyone pls help me to reproduce the example?
Checklist
If this pull request addresses a security vulnerability,
I confirm that I got approval (please contact [email protected]) from the maintainers to push the changes.
Further comments