Skip to content

Commit

Permalink
Change katex limits (#27823) (#27868)
Browse files Browse the repository at this point in the history
Backport #27823 by @KN4CK3R

Fixes #27812

Use higher defaults again but limit the input size.


![grafik](https://github.com/go-gitea/gitea/assets/1666336/23cdf572-de30-4799-b9cf-ef386b1623b9)

Co-authored-by: KN4CK3R <admin@oldschoolhack.me>
  • Loading branch information
GiteaBot and KN4CK3R committed Nov 1, 2023
1 parent 352906b commit 3959611
Showing 1 changed file with 12 additions and 2 deletions.
14 changes: 12 additions & 2 deletions web_src/js/markup/math.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,18 +15,28 @@ export async function renderMath() {
import(/* webpackChunkName: "katex" */'katex/dist/katex.css'),
]);

const MAX_CHARS = 1000;
const MAX_SIZE = 25;
const MAX_EXPAND = 1000;

for (const el of els) {
const target = targetElement(el);
if (target.hasAttribute('data-render-done')) continue;
const source = el.textContent;

if (source.length > MAX_CHARS) {
displayError(target, new Error(`Math source of ${source.length} characters exceeds the maximum allowed length of ${MAX_CHARS}.`));
continue;
}

const displayMode = el.classList.contains('display');
const nodeName = displayMode ? 'p' : 'span';

try {
const tempEl = document.createElement(nodeName);
katex.render(source, tempEl, {
maxSize: 25,
maxExpand: 50,
maxSize: MAX_SIZE,
maxExpand: MAX_EXPAND,
displayMode,
});
target.replaceWith(tempEl);
Expand Down

0 comments on commit 3959611

Please sign in to comment.