openlab-at-city-tech / webworkqa

WeBWorK integration for WordPress and BuddyPress
GNU General Public License v2.0
4 stars 2 forks source link

iOS/Safari: WeBWorK-Dev doesn't send any identifying information #128

Closed bree-z closed 5 years ago

bree-z commented 6 years ago

Sorry, for not catching this earlier. I tested this in Chrome and thought I'd tested in Safari too. In Chrome (iOS) it's working fine, but in Safari there's no user or question information sent from WW-Dev:

boonebgorges commented 6 years ago

Thanks, Bree. There are two issues here, both of which are tracked in #80.

  1. On openlabdev.org, POST data doesn't come through the Ask for Help chain. This appears to be related to the HTTP authentication requirements on openlabdev.org. This means that course, problem set, etc is not received by WP.

  2. Safari doesn't natively retain the URL fragment (the portion after #) in redirects. I built a workaround that stores the problem data in a cookie, and uses it to load the proper content in OL-WW. However, because of the POST problem in 1, the problem data is never received.

See especially https://github.com/livinglab/webwork-for-wordpress/issues/80#issuecomment-353484806 for more background. A closely related issue is https://discussions.apple.com/thread/8232508?answerId=8232508021#8232508021, though I can't find documentation on our precise issue from Safari/Apple.

As such, I'm fairly sure that this issue is limited to openlabdev.org, and that it's not new to this round of testing. (And it should probably be noted in the testing documentation.) The only way to confirm, as far as I can see, is to wait until after release.

bree-z commented 6 years ago

Thanks, Boone! I'll make sure to test after release, and have added a note to the testing documentation (since I may forget all over again next round!).

boonebgorges commented 6 years ago

Bree, I'm going to move this into the next milestone. Could you have a test on the production site when you have a moment?

bree-z commented 5 years ago

Ack! Sorry for never responding here! This is working fine on the production site on iOS/Safari.