-
Notifications
You must be signed in to change notification settings - Fork 187
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
add secureview flag when listing apps via http #9277
Conversation
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
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.
Hm, don't we need a default value here for Collabora, I don't want force people to configure it manually. Things should work out of the box.
Co-authored-by: Alex <[email protected]>
Yep, collabora is the favorite and default |
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
fixed typo and set default to collabora online
Can you wait for cs3 and bump in this pr? |
sure |
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
Quality Gate passedIssues Measures |
307 changed files? |
go back to your holiday! ;-) the reva bump pulled in the deps for the posix driver as well. |
add secureview flag when listing apps via http
Default value is wrong, should be Collabora not Collabora Online. |
To allow clients to see which application supports secure view we add a flag to the http response when the app name matches a configured secure view app. The app can be configured by setting
FRONTEND_APP_HANDLER_SECURE_VIEW_APP
to the name of the app registered as a CS3 app provider.Part of #9272
Only effective when cs3org/reva#4703 got merged and reva bumped