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

Algolia search, fix: point js + css file to specific version #1678

Merged
merged 1 commit into from
Nov 8, 2023

Conversation

deining
Copy link
Collaborator

@deining deining commented Sep 12, 2023

Currently js + css file for algolia docsearch are pulled in via these URLs:

When pulling in these files, they are checked via a SRI hash. In both cases, SRI hashes are given for version 3.5.1. However, meanwhile @docsearch npm is at version 3.5.2. This means, that given SRI hashes are invalid now, algolia docsearch does not work and more.

This PR fixes this by pulling in specific version 3.5.2.

@deining deining added the search Site search / search engines label Sep 12, 2023
@deining deining added this to the 23Q3 milestone Sep 12, 2023
@deining deining requested a review from chalin September 12, 2023 15:13
Copy link
Collaborator

@emckean emckean left a comment

Choose a reason for hiding this comment

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

LGTM, thanks so much for fixing this!

@froboy
Copy link
Contributor

froboy commented Oct 2, 2023

Confirming I'm seeing similar behavior. Docsearch seems to be broken in HEAD without this.

froboy added a commit to YCloudYUSA/yusaopeny_docs that referenced this pull request Oct 3, 2023
@chalin chalin modified the milestones: 23Q3, 23Q4 Nov 3, 2023
@deining deining modified the milestones: 23Q4, 23Q3 Nov 4, 2023
@deining deining mentioned this pull request Nov 4, 2023
22 tasks
@chalin chalin modified the milestones: 23Q3, 23Q4 Nov 7, 2023
Copy link
Collaborator

@chalin chalin left a comment

Choose a reason for hiding this comment

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

Thanks!

@chalin chalin merged commit 2ea2fa4 into google:main Nov 8, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
search Site search / search engines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants