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

chore: document non-standard glob client #328

Merged
merged 3 commits into from
Mar 28, 2023
Merged

chore: document non-standard glob client #328

merged 3 commits into from
Mar 28, 2023

Conversation

muhlemmer
Copy link
Collaborator

@muhlemmer muhlemmer commented Mar 14, 2023

Update the docs and example on glob support regarding standard deviation.
Glob pattern matching was introduced as an optional interface in #297 to provide wildcard support.

  1. Fix a typo
  2. Document standard deviation when using globs.
  3. Add example on how to toggle the underlying client implementation based on DevMode.

Open to discussion.

Closes #308

@codecov
Copy link

codecov bot commented Mar 14, 2023

Codecov Report

Merging #328 (3ca19d4) into main (62caf5d) will increase coverage by 0.16%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #328      +/-   ##
==========================================
+ Coverage   50.80%   50.97%   +0.16%     
==========================================
  Files          74       74              
  Lines        5623     5648      +25     
==========================================
+ Hits         2857     2879      +22     
- Misses       2506     2508       +2     
- Partials      260      261       +1     
Impacted Files Coverage Δ
pkg/op/client.go 92.98% <ø> (ø)
pkg/op/auth_request.go 53.70% <100.00%> (ø)

... and 4 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@muhlemmer muhlemmer added this to the v2 milestone Mar 14, 2023
@muir
Copy link
Contributor

muir commented Mar 15, 2023

@muhlemmer LGTM.

Other auth servers also support globs. For example, Keycloak.

@muhlemmer muhlemmer removed this from the v2 milestone Mar 15, 2023
rename checkURIAginstRedirects to checkURIAgainstRedirects
add example on how to toggle the underlying
client implementation based on DevMode.
@muhlemmer muhlemmer changed the base branch from next to main March 17, 2023 06:29
@muhlemmer muhlemmer changed the title feat: glob support for RedirectURIs chore: document non-standard glob client Mar 17, 2023
@muhlemmer muhlemmer marked this pull request as ready for review March 17, 2023 06:53
@muhlemmer muhlemmer requested a review from livio-a March 17, 2023 06:53
pkg/op/client.go Outdated Show resolved Hide resolved
Co-authored-by: David Sharnoff <dsharnoff@singlestore.com>
@muhlemmer muhlemmer merged commit b7d18bf into main Mar 28, 2023
@muhlemmer muhlemmer deleted the glob-support-next branch March 28, 2023 11:58
@github-actions
Copy link

🎉 This PR is included in version 2.2.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@github-actions
Copy link

github-actions bot commented May 2, 2023

🎉 This PR is included in version 3.0.0-next.6 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Follow up on Wildcard support
3 participants