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.

Sign up for GitHub

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

Jump to bottom

FIXED: Amazon links did not contain og tags with the default user agent #155

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

FIXED: Amazon links did not contain og tags with the default user agent #155

wants to merge 2 commits into from

Commits on Nov 13, 2021

  1. FIXED: Amazon links did not contain og tags with the default user agent 

    - Added 'userAgent' with a default value to preserve current behavior
- Included the 'googleBotUserAgent' static which can be used to override the default
- Not only does it include OG tags but the pages are way smaller due to Amazon optimizing for google search indexing (huge performance boost to SLP)
- Added unit tests for Amazon links and tested across several other major ecommerce sites
    @BrightChad
    BrightChad committed Nov 13, 2021
    Configuration menu
    Copy the full SHA
    546b86e View commit details
    Browse the repository at this point in the history

  2. renamed unit test after purpose of the test had changed

    @BrightChad
    BrightChad committed Nov 13, 2021
    Configuration menu
    Copy the full SHA
    66438b0 View commit details
    Browse the repository at this point in the history