Skip to content
This repository has been archived by the owner on May 29, 2024. It is now read-only.

Sentry is not working #1225

Closed
ia3andy opened this issue Apr 9, 2019 · 2 comments
Closed

Sentry is not working #1225

ia3andy opened this issue Apr 9, 2019 · 2 comments
Assignees

Comments

@ia3andy
Copy link
Contributor

ia3andy commented Apr 9, 2019

[Error] Cross-origin redirection to https://errortracking.prod-preview.openshift.io/api/29/?environment=production/store/?sentry_key=12813ed59a0546eaaf068d90127044c9&sentry_version=7 denied by Cross-Origin Resource Sharing policy: Origin https://developers.redhat.com is not allowed by Access-Control-Allow-Origin.
[Error] Fetch API cannot load https://errortracking.prod-preview.openshift.io/api/29?environment=production/store/?sentry_key=12813ed59a0546eaaf068d90127044c9&sentry_version=7 due to access control checks.
[Error] Failed to load resource: Cross-origin redirection to https://errortracking.prod-preview.openshift.io/api/29/?environment=production/store/?sentry_key=12813ed59a0546eaaf068d90127044c9&sentry_version=7 denied by Cross-Origin Resource Sharing policy: Origin https://developers.redhat.com is not allowed by Access-Control-Allow-Origin. (29, line 0)

This error put the frontend in an inconsistant state. We should fix the bug and make sure this can't happen even if sentry fails..

@ia3andy
Copy link
Contributor Author

ia3andy commented Apr 9, 2019

This bug is due to an obsolete sentry dsn config containing ?... at the end

@gastaldi
Copy link
Contributor

gastaldi commented Apr 9, 2019

Asked SD team to remove the parameter from the URL in https://gitlab.cee.redhat.com/dtsd/housekeeping/issues/2595

@ia3andy ia3andy closed this as completed Apr 11, 2019
ia3andy added a commit that referenced this issue Apr 23, 2019
- Fix public url request redirection (Closes #1225)
- Remove unecessary layer with  keycloak-react package (move code to launcher-app)
- Fix bug where request query  was removed in openshift redirect_uri
- Fix bug where requert query was removed in cleanUrl (end of init)
- Uniformise and refactor Authorization (prepare for #1254)
- Remove hack and standardize AuthenticationError
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants