Skip to content

SDK with custom host and optional Verify #22

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 23 commits into from
Jul 27, 2022
Merged

SDK with custom host and optional Verify #22

merged 23 commits into from
Jul 27, 2022

Conversation

meirwah
Copy link
Contributor

@meirwah meirwah commented Jul 26, 2022

also includes dockerized version of the sample app

@github-actions
Copy link

github-actions bot commented Jul 26, 2022 •
edited
Loading

Coverage

Coverage Report
FileStmtsMissCoverMissing
descope
   auth.py207199%55
TOTAL710199% 

Tests Skipped Failures Errors Time
62 0 💤 0 ❌ 0 🔥 1.097s ⏱️

meirwah
meirwah commented Jul 26, 2022
View reviewed changes
@meirwah
Copy link
Contributor Author

meirwah commented Jul 26, 2022

@guyp-descope let's talk about this one
Things here are for integration tests, but make py-sdk a bit ugly, so let's see if any better way

@guyp-descope
Copy link
Contributor

@meirwah yea lets talk tomorrow..
Im wondering what we are doing on the go-sdk..

@meirwah meirwah requested a review from guyp-descope July 27, 2022 07:51
@meirwah meirwah marked this pull request as ready for review July 27, 2022 08:26
@meirwah meirwah merged commit 8f07e3a into main Jul 27, 2022
@meirwah meirwah deleted the test-diff-host branch July 27, 2022 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@guyp-descope guyp-descope guyp-descope approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@meirwah @guyp-descope