Skip to content

Conversation

deining
Copy link
Contributor

@deining deining commented Jun 5, 2025

Currently, an invalid/wrong SRI hash is listed for katex.css.

<link rel="stylesheet" href="/css/katex.css" integrity="/css/katex.css" crossorigin="anonymous" />

This PR fixes that issue.
It also corrects an outdated link I spotted.

Copy link

netlify bot commented Jun 5, 2025

Deploy Preview for hugo-hextra ready!

Name Link
🔨 Latest commit 85f4f64
🔍 Latest deploy log https://app.netlify.com/projects/hugo-hextra/deploys/684191388467500008ce689e
😎 Deploy Preview https://deploy-preview-702--hugo-hextra.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@deining deining force-pushed the fix-sri-katex.css branch from 6536ba3 to 85d274b Compare June 5, 2025 12:43
@deining deining force-pushed the fix-sri-katex.css branch from 85d274b to 85f4f64 Compare June 5, 2025 12:44
@deining deining changed the title Fix sri katex.css katex.css: fix wring SRI hash Jun 5, 2025
@deining deining changed the title katex.css: fix wring SRI hash katex.css: fix wrong SRI hash Jun 5, 2025
@imfing imfing requested a review from Copilot June 10, 2025 22:35
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes the invalid SRI hash used for katex.css and updates an outdated documentation link.

  • Revised the SRI hash calculation in the head partial by computing a sha512 fingerprint and referencing its integrity data.
  • Updated the outdated cache busters URL in dev.toml to match the current Hugo documentation.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
layouts/_partials/head.html Updated SRI hash logic for katex.css using an explicit sha512 algorithm.
dev.toml Corrected the outdated cache busters URL comment.

Copy link
Owner

@imfing imfing left a comment

Choose a reason for hiding this comment

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

@deining thank you for the fix

@imfing imfing changed the title katex.css: fix wrong SRI hash fix: wrong SRI hash in katex.css Jun 10, 2025
@imfing imfing changed the title fix: wrong SRI hash in katex.css fix: wrong SRI hash for katex.css Jun 10, 2025
@imfing imfing merged commit b43870a into imfing:main Jun 10, 2025
4 checks passed
imfing pushed a commit that referenced this pull request Jul 21, 2025
* Correct URL given in 'dev.toml'

* stylesheet 'katex.css': fix SRI hash
imfing added a commit that referenced this pull request Jul 21, 2025
* Sync tabs across groups

* feat(tabs): add optional synchronization

* Move tabs sync setting under page params

* fix: spacing between title and site title (#704)

* docs: document configure opengraph image (#706)

* [Docs] document using og:image

* Make example title page match others

* clarify wording

* chore: update tailwind css to latest version 4.1.8 (#703)

* fix: wrong SRI hash for katex.css (#702)

* Correct URL given in 'dev.toml'

* stylesheet 'katex.css': fix SRI hash

* fix(build): run npm update to fix postcss complaint

* feat(tags): improve usability of tags (#698)

* feat(tags): improve usability of tags

* Tags can be shown also at docs
* Documented tag-related config flags
* Added example tags to the site
* Made rendered tags active

* Move tags listing to ToC

* Hide tags section on no tags

* feat(math): add optional MathJax support (#707)

* feat: add MathJax option

* docs: move math engine note

* refactor: update LaTeX documentation and improve MathJax integration

- Adjusted LaTeX documentation for clarity and formatting.
- Enhanced MathJax configuration in the templates to support both KaTeX and MathJax rendering.
- Removed deprecated comments and streamlined the script loading process for MathJax.
- Updated the passthrough extension settings in the Hugo configuration for better compatibility with LaTeX math expressions.

* docs: simplify LaTeX documentation and clarify configuration steps

- Updated LaTeX documentation to reflect that KaTeX is enabled by default, removing the need for manual activation.
- Added examples for using LaTeX math expressions and clarified the configuration for the passthrough extension in Hugo.
- Enhanced MathJax section to emphasize its use as an alternative rendering engine.

* fix(tabs): add null check for panels container and update example items

* fix(tabs): improve tab group key handling and add validation for items parameter

* refactor(tabs): comment out sync option in configuration and adjust tab formatting in documentation

---------

Co-authored-by: hobobandy <[email protected]>
Co-authored-by: Matt Dodson <[email protected]>
Co-authored-by: Andreas Deininger <[email protected]>
Co-authored-by: yuri <[email protected]>
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 this pull request may close these issues.

2 participants