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

WKWebView cancelled request(NSURLErrorCancelled -999) leads to errorUrl page instead of the second request #959

Closed
2 tasks done
arunn opened this issue Jul 28, 2020 · 1 comment · Fixed by #1483
Closed
2 tasks done
Milestone

Comments

@arunn
Copy link

arunn commented Jul 28, 2020

Bug Report

Problem

A request cancelled due to another request ends up showing the error page instead of response of the second request.

What is expected to happen?

The response of the second request should be shown.

What does actually happen?

The errorUrl page is shown.

Information

The error is similar to apache/cordova-plugin-wkwebview-engine#83 . We have two requests being run one after the other. The first request is cancelled once the second request is made. Instead of waiting for the response of the second request, the didFailLoadWithError shows the errorUrl page.

Environment, Platform, Device

cordova-ios, iPhone 8(iOS 13.6)

Version information

cordova(9.0.0)
cordova-ios(5.1.1)
cordova-plugin-wkwebviewengine(1.2.1)

Filing it here since the code has been integrated as of cordova-ios 6.0.0 and the issue doesn't seem to be fixed in the code.

Checklist

  • I searched for existing GitHub issues
  • [] I updated all Cordova tooling to most recent version
  • I included all the necessary information above
@tdfeslo
Copy link

tdfeslo commented Jan 19, 2022

Hi. Is there any news about this problem?
The solution proposed here solves the problem but may be introduce other problems (regression) ?
Thanks

@dpogue dpogue added this to the 8.0.0 milestone Aug 24, 2024
dpogue added a commit to dpogue/cordova-ios that referenced this issue Aug 28, 2024
This was previously implemented back in the UIWebView days as CB-13222
in apache#334 and was then proposed
in apache/cordova-plugin-wkwebview-engine#84 for
the WKWebView plugin, but that didn't get reviewed/merged before the
repo was archived. The fix is still valid, so we'll port it to the
current version of the WKWebView plugin.

Closes apacheGH-959.

Co-Authored-By: Gwyn Judd <gjudd@stayinfront.com>
@dpogue dpogue closed this as completed in 8b60dca Aug 29, 2024
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.

3 participants