custom-learning-office-365
custom-learning-office-365 copied to clipboard
pnp
[BUG] Content from Microsoft which has embedded videos stopped loading today
Describe the bug
Starting this morning, videos that were embedded within content provided by default with the solution stopped loading. This looks like a problem with the underlying page from Microsoft.
As this is an open source project and not covered by any support from Microsoft, I'm not quite sure where to submit this. It seems to be that the pages used by the webpart are just URL changes substituting "article" for "client" in the URL and that Microsoft itself would support the pages on their site, but that only this webpart would be making use of them. Hopefully the project maintainers have contacts they can use within MS to sort this out.
To Reproduce
-
Visit browsable page from Microsoft: https://support.office.com/en-us/article/join-a-meeting-in-teams-1613bb53-f3fa-431e-85a9-d6a91e3468c9#ID0EAABAAA=Mobile - this works and has an embedded video.
-
Visit page used by Learning Pathways (substitute "article" for "client" in URL): https://support.office.com/en-us/client/join-a-meeting-in-teams-1613bb53-f3fa-431e-85a9-d6a91e3468c9#ID0EAABAAA=Mobile - the video does not load. Web server returns a 302 response to an error page.
Expected behavior
Video should load as expected.
Screenshots
Not needed. Look at steps to reproduce.
Learning Pathways version number
N/A. Issue is with Microsoft web server/responses.
Additional context
Add any other context about the problem here.
Same error most likely. I'm getting a "www.microsoft.com refused to connect" regardless of browser when attempting to view video content in our Learning pathways.
Workaround for those in a bind:
- Find the CustomConfig list on the site where you're storing configuration of the solution
- Edit the CustomConfig item
- Copy all of the JSONData value into a text editor and do a find/replace all finding /client/ and replacing with /article/
- Find the LastUpdated property and change the date to a future date. This will prevent visitors to the site using the WebPart from overriding your custom config with new data from Microsoft (thanks @juliemturner)
- Change the title of the item to "CustomConfig-Original"
- Create a new item with title "CustomConfig", your custom JSON Data, and CDN set to Default or whatever your other item was
- Refresh pages that use the solution. Should be working, except it will also show the Microsoft header/nav within the webpart. But the content is usable.
- Avoid visiting the Admin page, as that can update the configuration and override your workaround (thanks again @juliemturner).
- When this issue is resolved by MS, you can delete your custom item and rename the other one back to CustomConfig.
NOTE This worked in my tenant and it may not work in yours. You are responsible for your own environment and take whatever risks on your own.
Yeah not only is going to the CustomLearningAdmin.aspx page going to break this fix but every 24 hours the web part will refresh the cache as well so this isn't going to be a long term fix. Glad it gets' you moving now, hopefully, it will get fixed quickly.
Thanks for the heads up Julie. We'll keep an eye on it. It should only update if someone who has rights to edit the backing list visits the site, so I'll tell those folks in my tenant to stay away from the site for now.
Looked into it and apparently the perms on the list from the solution install is to have even visitors have contribute rights... I've edited those in my tenant such that no one but Site Collection Admins can edit the file and those folks have been informed.
@juliemturner do you foresee any issue when the WebPart attempts to edit the config file and the person viewing it has no edit rights on the backing config file?
I do that on purpose so that when the cache needs to be refreshed the user can update it. You will end up seeing slower performance because of this change, as the cache will have expired and every time the web part loads it's going to do all the work to update it and then will fail, but if you're ok with that then I guess it's fine. I wouldn't keep this change in place after it's fixed and I can't guarantee you that I'll be informed that it's fixed and update this issue that it has been as this is outside of my visibility. Probably instead of changing the permissions, I would have set the "LastUpdated" date at the end of the JSON blob out into the future, although that won't protect you if you visit the admin page.
@juliemturner Thanks. Any way we can make it a configuration setting in the future to allow visitors to the site to even perform this functionality? Would rather not have a list open to modification by visitors, but understand that some people would be okay with it.
Get the code put in GitHub and I'll submit a PR :)
Are you asking me if you can set visitors to read-only on the list but then allow them to update the item in the list? No there's no way to do that.
Oh, I misunderstood... you want a config option... potentially possible I would submit an enhancement request.
Same error most likely. I'm getting a "www.microsoft.com refused to connect" regardless of browser when attempting to view video content in our Learning pathways.
The Learning Pathways team is aware of the "www.microsoft.com refused to connect" issue and is actively investigating.
@gagarine Thanks for letting us know. @jhwang-ms can you please let the content team know that this is happening again?
