Skip to content
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

docs: added the python backend guide #981

Merged
merged 3 commits into from
Sep 18, 2023

Conversation

Chigala
Copy link
Contributor

@Chigala Chigala commented Aug 20, 2023

Description

confirmed the backend guide is working, and updated the python backend guide.

/closes #980
/claim #980

@Chigala
Copy link
Contributor Author

Chigala commented Aug 20, 2023

I was wrong, the code works.

@algora-pbc algora-pbc bot mentioned this pull request Aug 20, 2023
@FlxMgdnz
Copy link
Member

Hey @Chigala could you please provide the example app as well so we can test it?

@Chigala
Copy link
Contributor Author

Chigala commented Aug 27, 2023

@FlxMgdnz I'm very sorry for the delay, I just saw your comment like some minutes ago. Here is the example app. https://github.com/Chigala/hanko-fastapi-guide, you can add the necessary token and then run it. while testing I used the example Next project in the repo to get the cookie token, then the HANKO_API_URL was from my hanko dashboard. Everything works fine but for you to run it in development, you will have to add this:

 # Disable SSL certificate verification
    ssl_context = ssl.create_default_context()
    ssl_context.check_hostname = False
    ssl_context.verify_mode = ssl.CERT_NONE

        jwks_client = jwt.PyJWKClient(
            HANKO_API_URL + "/.well-known/jwks.json", ssl_context=ssl_context
        )

Maybe I should have included this in the docs too, what do you think?

@Ashutosh-Bhadauriya
Copy link
Contributor

Ashutosh-Bhadauriya commented Sep 14, 2023

@Chigala sorry we had a bit of a delay from our side. Everything looks good, we'll be merging soon.

Maybe I should have included this in the docs too, what do you think?

Yes, please. Thanks!

@Chigala
Copy link
Contributor Author

Chigala commented Sep 17, 2023

@Ashutosh-Bhadauriya @FlxMgdnz I just updated the docs. I think this can be merged now.

@Ashutosh-Bhadauriya
Copy link
Contributor

Thanks @Chigala, we'll merge it soon

@Ashutosh-Bhadauriya Ashutosh-Bhadauriya merged commit 342b724 into teamhanko:main Sep 18, 2023
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

FastAPI backend guide
3 participants