TurnWheel/jReject

Defective mobile recognition (Android)

m1n0 opened this issue · 4 comments

m1n0 commented

I noticed that some android browsers contain string "Version/X", e.g. "Mozilla/5.0 (Linux; Android 4.4.3; Galaxy Nexus Build/KTU84M) AppleWebKit/537.36 (KHTML, like Gecko) Version/4.0 Chrome/33.0.0.0 Mobile Safari/537.36"

This string gets mistaken as a chrome version and is rejected. (as Chrome 4 in my example).

The parsing mechanism should be aware of that and deal with it somehow (perhaps ignore it? I dont know what is that part used for.)

👍

👍

Is there a solution available to this? We're using the Drupal module that uses this library and have to disable all rejection of old Chrome browsers in order for Androids to work. Not 100% sure that solution covers all Androids either as some don'e have the Chrome part of the string (though those that don't seem to be accepted and don't get the popup).

I've changed underlaying library which detect browser to more up-to-date one within this pull request. It detects mobile browsers, so it can be solution for your problem.
Try this version (I don't suppose jReject's authors will merge them in fast, judging by their recent activity) and say if it worked for you, please.