Skip to content

Conversation

gao-sun
Copy link
Member

@gao-sun gao-sun commented Apr 19, 2024

Summary

this pull is a follow-up of #17. the original pull is based on the master branch, so i could not push directly.

the original commit was cherry-picked to this pull. here are the main updates:

  • removed ID token JWT verification since it's a built-in behavior of Logto Python SDK
  • refactor the decorator, inject a user object to the application context (g.user)
  • use environment variables for all configs
  • reorg the documentation for the sample project

huge credit to @ArthoPacini for the original contribution!

Testing

local tested with sign-in/out, visited all four pages. worked as expected.

@gao-sun gao-sun force-pushed the gao-refactor-sample branch from 054b323 to 68b4e13 Compare April 19, 2024 14:18
@gao-sun gao-sun mentioned this pull request Apr 19, 2024
4 tasks
@gao-sun gao-sun marked this pull request as ready for review April 19, 2024 14:21
@gao-sun gao-sun merged commit b890757 into master Apr 19, 2024
@gao-sun gao-sun deleted the gao-refactor-sample branch April 19, 2024 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants