-
Notifications
You must be signed in to change notification settings - Fork 164
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: token refresh in single binary and wopi deployment example #9167
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
9490017
to
b4f8ec1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
AFAIK the problem is also present in a default deployment (i.e. without wopi and any other external stuff). So I guess we need to update the default config https://github.com/owncloud/ocis/blob/master/services/proxy/pkg/config/csp.yaml as well?
Testing this ... |
Single binary has the same issue.... |
b4f8ec1
to
c1eb5e6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
c1eb5e6
to
0cf3298
Compare
rebased ..... let's see if drone likes this now ..... 🤷 |
@DeepDiver1975 The tests are failing because they expect different CSP headers. |
0cf3298
to
ba2ff61
Compare
Quality Gate passedIssues Measures |
Description
When refreshing the oidc client loads ocis in an iframe - which was disallowed until this fix.
Related Issue
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: