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

detect and warn of misconfigured server #1707

Closed
indirectlylit opened this issue Jun 21, 2017 · 3 comments
Closed

detect and warn of misconfigured server #1707

indirectlylit opened this issue Jun 21, 2017 · 3 comments
Labels
TAG: ux update Improved user-facing feature

Comments

@indirectlylit
Copy link
Contributor

Admins occasionally run into issues where people can't interact with streaming content and spend time trying to debug the cause. This can occur because either the user is running the devserver, or because of misconfigured nginx front-ends.

For example:

If possible, it would be great to detect a misconfigured server and show a warning to the user.

@rtibbles
Copy link
Member

Yeah, we could attempt a byte range request and see if it works, if it doesn't, we can display a warning to the user, and also disable the seeking functionality.

@indirectlylit indirectlylit added the TAG: ux update Improved user-facing feature label Oct 27, 2017
@indirectlylit indirectlylit added this to the 0.7.0 milestone Oct 27, 2017
@indirectlylit indirectlylit added P1 - important Priority: High impact on UX TODO: needs gherkin update Add to our manual integration tests DEV: frontend labels Oct 27, 2017
@indirectlylit
Copy link
Contributor Author

this could be a good place to display more technical info resulting from this warning: #1910

@indirectlylit indirectlylit removed the P1 - important Priority: High impact on UX label Nov 21, 2017
@indirectlylit indirectlylit removed this from the 0.7.0 milestone Nov 21, 2017
@indirectlylit indirectlylit added this to the 0.11.0 milestone Aug 27, 2018
@indirectlylit indirectlylit removed this from the 0.11.0 milestone Sep 4, 2018
@indirectlylit
Copy link
Contributor Author

this has been mostly addressed. Let's open issues for specific things if we need more warnings

@radinamatic radinamatic removed the TODO: needs gherkin update Add to our manual integration tests label Dec 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TAG: ux update Improved user-facing feature
Projects
None yet
Development

No branches or pull requests

3 participants