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

Fix: Traefik path style #1088

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

honghainguyen777
Copy link

@honghainguyen777 honghainguyen777 commented Dec 4, 2023

Hello everyone!

I've encountered an issue with the regexPathStyle in Traefik. Currently, the pattern api/{id:[1-9][0-9]}/{path:.} is effectively handling Error Messaging, but it's not functioning correctly for XHR requests, such as logging out, issue merging/deleting, and commenting.

After reviewing the Traefik documentation, I suspect the path style might not be optimally configured. To address this, I've created a Pull Request with an updated path style. This change has successfully resolved the issue in my tests, ensuring both exception handling and XHR requests work seamlessly.

Related issue: #1086

I'm looking forward to your feedback and suggestions on this adjustment.

Best regards,
Hai

@patsevanton
Copy link
Contributor

@honghainguyen777
The regexPathStyle support for traefik seems to be implemented in this pull request https://github.com/sentry-kubernetes/charts/pull/1466/files.
Please check if the problem is reproduced after this pull request?

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

Successfully merging this pull request may close these issues.

2 participants