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

PostLogoutRedirectURIGlobs uses path.Match and RedirectURIGlobs uses doublestar.Match #668

Open
1 of 2 tasks
rbinder-vg opened this issue Oct 24, 2024 · 1 comment
Open
1 of 2 tasks

Comments

@rbinder-vg
Copy link

Preflight Checklist

  • 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

@hifabienne
Copy link
Member

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 hifabienne moved this to 📨 Product Backlog in Product Management Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 📨 Product Backlog
Development

No branches or pull requests

2 participants