Skip to content
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

Custom video bgs not working #1167

Closed
banholzer opened this issue Apr 3, 2024 · 26 comments · Fixed by #1226
Closed

Custom video bgs not working #1167

banholzer opened this issue Apr 3, 2024 · 26 comments · Fixed by #1226
Labels
angular-2-react blocked This issue depends on external projects

Comments

@banholzer
Copy link
Contributor

banholzer commented Apr 3, 2024

Describe the bug
Custom video backgrounds are not showing - not even the thumbs are showing up.

To Reproduce

  1. starting from cli
❯ teams-for-linux --optInTeamsV2
[WARN] No config file found, using default values
[ERROR] Failed to save remote configuration at '/<homedir>/.config/teams-for-linux/custom_bg_remote.json'

while <homedir> is the actual directory where my home-directory is.
2. Images not showing up in background effects view.

Expected behavior
Custom images showing up - worked in previous versions. Apache config & content was not changed.

Screenshots

Desktop (please complete the following information):
Environment: Ubuntu 22.04 with KDE Plasma Desktop

❯ dpkg -l teams-for-linux | grep teams-for-linux
ii  teams-for-linux 1.4.17       amd64        Teams for Linux
  • Version 1.4.17 pre-release

Additional context
was not yet able to rollback and test with older version. Already cleared local storage - no change. Also deleted complete .config/teams-for-linux/ - no change.

@IsmaelMartinez
Copy link
Owner

I suspect this is related to teams-v2 move as they will probably have changed stuff.

However, the log error looks more of a permissions issue that the later. Anybody help??

@DerXteMensch
Copy link

Custom backgrounds were working for me for many months. But the moment I switch to teams v2, they don't work anymore. See also #1171

@DerXteMensch
Copy link

With an older version (I think v1.4.14, but I'm not all sure) it didn't work either with teams v2, only with teams v1. Switched today from (probably) 1.4.14 to 1.4.17

@IsmaelMartinez
Copy link
Owner

IsmaelMartinez commented Apr 4, 2024

Is probably the URL then! Worth checking if the urls still march the ones in https://github.com/IsmaelMartinez/teams-for-linux/blob/develop/app/mainAppWindow/index.js#L247

@ajoergensen
Copy link

ajoergensen commented Apr 4, 2024

It looks like the background images are fetched from https://statics.teams.cdn.office.net/evergreen-assets/backgroundimages/

@IsmaelMartinez
Copy link
Owner

Ok, I think this is related to the move from angular to react that is probably breaking this file:

https://github.com/IsmaelMartinez/teams-for-linux/blob/develop/app/browser/tools/customBackgrounds.js

but it looks like Microsoft is not supporting custom backgrounds yet in the new teamsv2 so this issue is blocked. Sorry!

@IsmaelMartinez IsmaelMartinez added the blocked This issue depends on external projects label Apr 4, 2024
@banholzer
Copy link
Contributor Author

banholzer commented Apr 4, 2024

today switched back to 1.4.13 which I thought worked before - still seeing same behavior

reverted to teamsv1

and deleted .config/teams-for-linux/

❯ teams-for-linux
[ERROR] Failed to save remote configuration at '/.../.config/teams-for-linux/custom_bg_remote.json'
❯ teams-for-linux --version
1.4.13

same with 1.4.12 - probably the issue is somewhere on how Microsoft code searches for the images

@DerXteMensch
Copy link

So I'll stay as long as it is possible with teams v1

@jkrysakowski
Copy link

jkrysakowski commented Apr 5, 2024

I just managed to run a custom background. Hint: the domain that serves the config and bg files must support https://

I see a bug: the background image is flipped horizontally

@calebdw
Copy link

calebdw commented Apr 8, 2024

I just managed to run a custom background. Hint: the domain that serves the config and bg files must support https://

This is still only on v1 and not v2, I am unable to get the custom background images working even with https support

@IsmaelMartinez
Copy link
Owner

IsmaelMartinez commented Apr 8, 2024 via email

@derekatkins
Copy link

My org just forced me to upgrade to Teams v2 and now I'm hitting this bug. My --customBGServiceBaseUrl=https://my.custom/url which worked a weekk ago isn't working now that teams got updated.

@derekatkins
Copy link

It's not even hitting my link, so it's not even attempting to download the config.json file.

@jijojosephk
Copy link
Collaborator

We'll add more logs to help debugging.

@Cybso
Copy link

Cybso commented Apr 17, 2024

I assume it's related to this error message in the web debug console:

Failed to override MS Method
error @ v2/:3
(anonymous) @ /opt/teams-for-linux/resources/app.asar/app/browser/tools/customBackgrounds.js:21

Edit: I managed to get the error message, it is 'Failed to get app objects after 5 tries'

@jijojosephk
Copy link
Collaborator

Existing implementation is dependent on angular based teams app v1. With the new version, it will not work.

@banholzer
Copy link
Contributor Author

should we then close this issue for that reason?

@jijojosephk
Copy link
Collaborator

We can keep it until we find a solution with latest version of teams.

@jonatasrs
Copy link
Contributor

Any work in progress on this?

@IsmaelMartinez
Copy link
Owner

I am afraid no, at the moment this is just not working and unfortunately I don't have time to look at it, whoever I encourage anyone to have a look.

In the meantime, you can use OBS as described in #1215 (comment) . Not ideal but a workaround.

@jonatasrs
Copy link
Contributor

Thanks for the answer, I added this pull request #1226

@IsmaelMartinez
Copy link
Owner

legend! Left a wee comment. I will probably try to release it with #1228 and https://github.com/IsmaelMartinez/teams-for-linux/pull/1224/files as a pre-release early next week. (No point of having 3 small releases when the code touches different parts). Thanks again and good finding!

@limes007
Copy link

@jonatasrs Great, thank you.

@derekatkins
Copy link

Indeed, thank you.
By my reading of the patch, v2 uses a different JSON format than v1 -- correct?

@DerXteMensch
Copy link

DerXteMensch commented May 2, 2024

Unfortunately I was forced to switch to Teams v2 today. Is there a pre-release yet?

Edit: After a restart of Teams, it started with v1 again... Nevertheless I'm looking forward to a pre-release

@IsmaelMartinez
Copy link
Owner

can one of you try the MR in #1226 ? Got a small open question but it will be good if one of you can validate that is the fix. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
angular-2-react blocked This issue depends on external projects
Projects
None yet
Development

Successfully merging a pull request may close this issue.