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

Commit

Permalink
Remove dead function in WidgetUtils relating to screenshots/capabilities
Browse files Browse the repository at this point in the history
We always approve the capability these days
  • Loading branch information
turt2live committed May 6, 2022
1 parent 485f42e commit e52c7fe
Showing 1 changed file with 1 addition and 16 deletions.
17 changes: 1 addition & 16 deletions src/utils/WidgetUtils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ limitations under the License.

import * as url from "url";
import { base32 } from "rfc4648";
import { Capability, IWidget, IWidgetData, MatrixCapabilities } from "matrix-widget-api";
import { IWidget, IWidgetData } from "matrix-widget-api";
import { Room } from "matrix-js-sdk/src/models/room";
import { MatrixEvent } from "matrix-js-sdk/src/models/event";
import { logger } from "matrix-js-sdk/src/logger";
Expand Down Expand Up @@ -496,21 +496,6 @@ export default class WidgetUtils {
return app as IApp;
}

static getCapWhitelistForAppTypeInRoomId(appType: string, roomId: string): Capability[] {
const enableScreenshots = SettingsStore.getValue("enableWidgetScreenshots", roomId);

const capWhitelist = enableScreenshots ? [MatrixCapabilities.Screenshots] : [];

// Obviously anyone that can add a widget can claim it's a jitsi widget,
// so this doesn't really offer much over the set of domains we load
// widgets from at all, but it probably makes sense for sanity.
if (WidgetType.JITSI.matches(appType)) {
capWhitelist.push(MatrixCapabilities.AlwaysOnScreen);
}

return capWhitelist;
}

static getLocalJitsiWrapperUrl(opts: {forLocalRender?: boolean, auth?: string} = {}) {
// NB. we can't just encodeURIComponent all of these because the $ signs need to be there
const queryStringParts = [
Expand Down

0 comments on commit e52c7fe

Please sign in to comment.