Skip to content

handle teams v2 meeting invites differently than v1 invites. making the default logger electron-log, as the config options seeem to be broken #1378

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

Merged

Conversation

IsmaelMartinez
Copy link
Owner

…he default logger electron-log, as the config options seeem to be broken
@IsmaelMartinez IsmaelMartinez merged commit 53971e3 into develop Aug 23, 2024
7 of 8 checks passed
Copy link

@@ -199,19 +199,25 @@
}

function processArgs(args) {
const regMS = /^msteams:\/.*(?:meetup-join|channel)/g;
const v1msTeams = /^msteams:\/l\/(?:meetup-join|channel)/g;
const v2msTeams = /^msteams:\/\/teams.microsoft.com\/l\/(?:meetup-join|channel)/g;

Check failure

Code scanning / CodeQL

Incomplete regular expression for hostnames High

This regular expression has an unescaped '.' before 'microsoft.com', so it might match more hosts than expected.
@IsmaelMartinez IsmaelMartinez deleted the feature-1373-handle-teams-v2-msteams-invites branch October 27, 2024 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Teams Link not opening
1 participant