solomax commented on a change in pull request #85:
URL: https://github.com/apache/openmeetings/pull/85#discussion_r424901002
##########
File path:
openmeetings-web/src/main/java/org/apache/openmeetings/web/room/raw-video-util.js
##########
@@ -211,7 +211,11 @@ var VideoUtil = (function() {
}
function _isEdge(_b) {
const b = _b || kurentoUtils.WebRtcPeer.browser;
- return b.name === 'Edge';
+ return b.name === 'Edge' && "MSGestureEvent" in window;
+ }
+ function _isEdgeChromium(_b) {
+ const b = _b || kurentoUtils.WebRtcPeer.browser;
+ return b.name === 'Edge' && !("MSGestureEvent" in window);
Review comment:
> Are we want to support old EdgeHTML-based edge at all? Currently it is
broken due to CSP bugs with strict-dynamic and inline-unsafe.
Audio/video most probably will not work in this browser as well
But I believe we have no chances to just ignore it's existence
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
