Skip to content

Adds detection for Flow browser engine and for Flow browser #6467

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

Merged
merged 14 commits into from
Feb 1, 2021
Merged

Adds detection for Flow browser engine and for Flow browser #6467

merged 14 commits into from
Feb 1, 2021

Conversation

liviuconcioiu
Copy link
Collaborator

No description provided.

@liviuconcioiu
Copy link
Collaborator Author

The engine version is not seen correctly. I tried multiple regexes and all failed. I think there is a bug in Engine.php.

sanchezzzhak
sanchezzzhak previously approved these changes Dec 2, 2020
Copy link
Collaborator

@sanchezzzhak sanchezzzhak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

According to this news they use their own engine
https://www.ekioh.com/news/ekioh-launches-the-worlds-first-multithreaded-embedded-browser/

rv gecko (RenderVersion) is equal to the Firefox version, starting from the browser version 3.6+

The presence of FF + Gecko is a formality so that the browser is not blocked by sites.

sanchezzzhak
sanchezzzhak previously approved these changes Jan 8, 2021
@sgiehl
Copy link
Member

sgiehl commented Feb 1, 2021

@sanchezzzhak I guess this one would be good to merge, right?

@sanchezzzhak
Copy link
Collaborator

sanchezzzhak commented Feb 1, 2021

Yes

@sgiehl sgiehl merged commit 49e94f6 into matomo-org:master Feb 1, 2021
@liviuconcioiu liviuconcioiu deleted the flow branch February 1, 2021 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants