poorejc commented on issue #50: URL: https://github.com/apache/incubator-flagon-useralejs/issues/50#issuecomment-806903564
@UncleGedd you've confirmed some of my suspicions that the HTTP request isn't going to work triggered against events at the tale end of the page lifecycle. Regarding your thoughts: 1. Sounds like an interesting idea. Will wait for more thoughts from @confusingstraw 2. We actually did have some user interest in a Websocket option for sendLogs. It's not a bad option for sendLogs, but I don't think it should be the 'only' option, which is what we're saying if we tie it to expected behavior for sendOnClose. The thing I don't like about that idea for canonical use is that it ties the library too closely to a specific backend. Our loose integration with back-end datastores was a design choice. I know some of our users are now using UserALE with different back-ends (not ELK). > Ok, so I've tried writing a cypress/journey test for `sendBeacon` and it's pretty flaky, but manual testing works in the browser (Chrome) consistently though +1. Same here. > RE 3. Safari doesn’t support visibility change events as uniformly as Moz and Chrome. I think after some more testing, we might end up settling on a conditional that sniffs for visibilitychange events and/or whatever most reliable equivalent is for Safari. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
