oidc icon indicating copy to clipboard operation
oidc copied to clipboard

PostLogoutRedirectURIGlobs uses path.Match and RedirectURIGlobs uses doublestar.Match

Open rbinder-vg opened this issue 1 year ago • 1 comments

Preflight Checklist

  • [x] I could not find a solution in the existing issues, docs, nor discussions
  • [ ] I have joined the ZITADEL chat

Describe your problem

The op.HasRedirectGlobs.PostLogoutRedirectURIGlobs() uses path.Match and op.HasRedirectGlobs.RedirectURIGlobs() uses doublestar.Match (since #297). This is not very intuitive to use. You can use doublestar for the redirect uri, but not for the post logout redirect uri.

Describe your ideal solution

Use doublestar.Match here

Version

No response

Environment

Self-hosted

Additional Context

No response

rbinder-vg avatar Oct 24 '24 15:10 rbinder-vg

Thank you for sharing your idea. If there is a significant demand from customers/community, we will carefully consider implementing the feature. Currently, the issue will be added to our product backlog to collect feedback.

Meanwhile, if you're interested in implementing it yourself, we also welcome pull requests.

hifabienne avatar Oct 25 '24 12:10 hifabienne