feat: Add oauth flow for querybook github integration #1497
Conversation
zhangvi7
force-pushed
the
github/oauth
branch
2 times, most recently
from
500a138
to
f0b1c90
Compare
zhangvi7
force-pushed
the
github/oauth
branch
from
f0b1c90
to
5a6db84
Compare
There was a problem hiding this comment.
to make it simple, can remove integration from the names
e.g. folder name can be just /lib/github/
GitHubIntegrationManager -> GitHubManager
| return github_manager.github_integration_callback() | ||
|
|
||
|
|
||
| # Test GitHub OAuth Flow |
There was a problem hiding this comment.
let's not have the test code here. move to a separate test file
| def save_github_token(self, token: str) -> None: | ||
| flask_session["github_access_token"] = token | ||
| LOG.debug("Saved GitHub token to session") |
There was a problem hiding this comment.
will save to the user properties as well?
There was a problem hiding this comment.
Will handle refactoring to encrypt in db in later PR
| @flask_app.route(GITHUB_OAUTH_CALLBACK) | ||
| def github_callback() -> str: | ||
| github_manager = get_github_manager() | ||
| return github_manager.github_integration_callback() |
There was a problem hiding this comment.
would be better to just put this inside the datasources/github.py together with other github routes?
| def get_github_manager() -> GitHubIntegrationManager: | ||
| return GitHubIntegrationManager(additional_scopes=["repo"]) |
There was a problem hiding this comment.
seems we can just just create a module level manager instance
github_manager = GitHubIntegrationManager(additional_scopes=["repo"])
| window.receiveChildMessage = receiveMessage; | ||
|
|
||
| // If the user closes the authentication window manually, clean up | ||
| const timer = setInterval(() => { |
There was a problem hiding this comment.
why it needs a timer here?
| return config | ||
|
|
||
| def save_github_token(self, token: str) -> None: | ||
| flask_session["github_access_token"] = token |
There was a problem hiding this comment.
let's define a const for the key github_access_token as it will be used by other module
OAuth flow for querybook github integration feature
Test Plan
Datadoc rightside bar:
Modal:
GitHub Oauth:
Post OAuth Flow (To be changed):