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

Page morphs fail to refresh (blank operations hash) in Safari/Edge on staging #453

Closed
existentialmutt opened this issue Feb 9, 2021 · 2 comments · Fixed by #454
Closed

Comments

@existentialmutt
Copy link
Contributor

existentialmutt commented Feb 9, 2021

Describe the bug

When my app is running on staging and production, and browsing with Safari (13.0.5) or Edge (88), page morphs fail to update the page. Looking at the websocket messages, I see the reflex coming back from the server with a blank cableready operations hash. This doesn't happen in other browsers, with selector morphs in Safari and Edge on staging, or in any browser running the site locally in development mode.

I've tried creating an MVCE with stimulus_reflex_harness and can't reproduce it there, so I guess it's something in my app that's screwing things up. Any ideas on what could cause the operations hash to go blank, or suggestions on how I could approach debugging?

Screenshots or reproduction

Web_Inspector_—staging_cumulusrisk_com—_facility_profile

Versions

StimulusReflex

  • Gem: 3.4.1
  • Node package: 3.4.1

External tools

  • Ruby: 2.7.2
  • Rails: 6.1.1
  • Node: 10.18.0

Browser

Safari 13.0.5
Edge 88

@existentialmutt
Copy link
Contributor Author

As soon as I asked I figured it out. We have HTTP_BASIC authentication on the staging site and those browsers weren't sending credentials. Fixed it by disabling the authenticate_or_request_with_http_basic before filter if @stimulus_reflex is true.

@existentialmutt existentialmutt changed the title Page morphs fail to refresh (blank operations hash) in Safari/Edge on staging/production Page morphs fail to refresh (blank operations hash) in Safari/Edge on staging Feb 9, 2021
@leastbad
Copy link
Contributor

leastbad commented Feb 9, 2021

I'll add this to the troubleshooting page! Sorry that you were having troubles.

@leastbad leastbad mentioned this issue Feb 10, 2021
3 tasks
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 a pull request may close this issue.

2 participants