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

Add warning on camera feed disconnect #606

Merged
merged 1 commit into from
Oct 27, 2023
Merged

Conversation

msathieu
Copy link
Member

No description provided.

@zeuswpi-bot
Copy link

Check out the preview on https://606.pr.zeus.gent/

@redfast00
Copy link
Member

lIk zou misschien wat meer duiding geven rond wat de bedoeling is van cammie (zien wie er in de kelder is voorsat je zelf afkomt, en zien hoe druk het is) en waarom dat disconnect. Nu lijkt het gewoon alsof die camera brak is.

@redfast00
Copy link
Member

En dan meer technisch: we gaan op termijn wss afstappen van jQuery, zou je het mss in plain JS willen schrijven?

@msathieu
Copy link
Member Author

@redfast00 zou dit beter zijn:

Camera feed automatically disconnected due to inactivity, reload the page to resume viewing
Cammie is meant to check if someone is present and shouldn't be left open for longer periods of time

@redfast00
Copy link
Member

Das idd al een pak beter :) mss veranderen naar 'meant for checking if the kelder is busy before coming', zodat het niet lijkt alsof cammie er is specifiek om mensen te stalken via het checken of iemand in de kelder is

@msathieu msathieu requested a review from redfast00 October 26, 2023 15:24
Copy link
Member

@redfast00 redfast00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ziet er goed uit!

@xerbalind
Copy link
Member

xerbalind commented Oct 27, 2023

Wanneer ik de camera bekijk op mijn browser app op android (duckduckgo) verschijnd de warning al na 5 seconden (zal die timeout zijn)

@redfast00
Copy link
Member

Hm, die navigator.onLine houdt niet echt steek, denk ik, dat gaat over je volledige browser. Kunnen we van het cammie-mjpeg zelf zien of die nog afspeelt?

@msathieu
Copy link
Member Author

Chromium roept onload helaas enkel op bij de eerste frame van een multipart/x-mixed-replace afbeelding (https://bugs.chromium.org/p/chromium/issues/detail?id=7731) en voor zover ik weet is er geen andere goede manier om te zien wanneer de stream stopt. Misschien moet ik dit gewoon implementeren door een 30s timer te hardcoden.

Hm, die navigator.onLine houdt niet echt steek, denk ik, dat gaat over je volledige browser. Kunnen we van het cammie-mjpeg zelf zien of die nog afspeelt?

Dit leek me de beste methode om het verschil te detecteren tussen een stream die bewust afgesloten werd door de server of door een netwerk error, maar het is inderdaad niet optimaal.

@redfast00
Copy link
Member

Gewoon hardcoded is idd nog een goed idee dan, je mag die timer op 25s zetten

@redfast00
Copy link
Member

Ik zie net dat de controls blijven staan als cammie niet meer werkt, mss die elementen ook verwijderen?

@redfast00 redfast00 merged commit 8071e22 into master Oct 27, 2023
1 check passed
@redfast00 redfast00 deleted the disconnect-warning branch October 27, 2023 16:05
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.

4 participants