Compare commits
1 Commits
palette-ux
...
sentinel-s
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
5d9f78d64f |
@@ -27,3 +27,8 @@
|
|||||||
**Vulnerability:** Allowing users to register or submit forms with disposable email addresses (e.g., mailinator.com) can lead to spam, abuse, and polluted data.
|
**Vulnerability:** Allowing users to register or submit forms with disposable email addresses (e.g., mailinator.com) can lead to spam, abuse, and polluted data.
|
||||||
**Learning:** While true email verification requires a backend or API, a simple client-side blocklist of common disposable domains is a highly effective, low-cost first line of defense.
|
**Learning:** While true email verification requires a backend or API, a simple client-side blocklist of common disposable domains is a highly effective, low-cost first line of defense.
|
||||||
**Prevention:** Maintain a list of known disposable domains (e.g., `BLOCKED_DOMAINS`) and check the domain part of the email address during validation.
|
**Prevention:** Maintain a list of known disposable domains (e.g., `BLOCKED_DOMAINS`) and check the domain part of the email address during validation.
|
||||||
|
|
||||||
|
## 2026-02-14 - Backtick Escaping in Sanitization
|
||||||
|
**Vulnerability:** Standard HTML entity encoding often overlooks backticks (`` ` ``), which are dangerous in JavaScript template literals.
|
||||||
|
**Learning:** While `&`, `<`, `>`, `"`, `'` are standard, backticks are unique to modern JS. If a sanitized string is interpolated into a JS template literal, an unescaped backtick can break out of the string context and allow arbitrary code execution.
|
||||||
|
**Prevention:** Always include `.replace(/`/g, "`")` in custom HTML sanitization functions to prevent injection in JS contexts.
|
||||||
|
|||||||
21
src/App.tsx
21
src/App.tsx
@@ -3,7 +3,7 @@ import { BrowserRouter, Routes, Route } from 'react-router-dom';
|
|||||||
import { LanguageProvider } from './i18n';
|
import { LanguageProvider } from './i18n';
|
||||||
import { Navbar, Footer, FancyCursor, ScrollToTop } from './components/layout';
|
import { Navbar, Footer, FancyCursor, ScrollToTop } from './components/layout';
|
||||||
import { Home } from './pages/Home';
|
import { Home } from './pages/Home';
|
||||||
import { PageLoader, SkipLink } from './components/ui';
|
import { PageLoader } from './components/ui';
|
||||||
import './styles/global.css';
|
import './styles/global.css';
|
||||||
|
|
||||||
// Lazy load pages to reduce initial bundle size.
|
// Lazy load pages to reduce initial bundle size.
|
||||||
@@ -15,20 +15,17 @@ export function App() {
|
|||||||
return (
|
return (
|
||||||
<LanguageProvider>
|
<LanguageProvider>
|
||||||
<BrowserRouter>
|
<BrowserRouter>
|
||||||
<SkipLink />
|
|
||||||
<ScrollToTop />
|
<ScrollToTop />
|
||||||
<FancyCursor />
|
<FancyCursor />
|
||||||
<Navbar />
|
<Navbar />
|
||||||
<main id="main-content" tabIndex={-1} style={{ flex: 1, outline: 'none' }}>
|
{/* Suspense handles the loading state for lazy-loaded routes */}
|
||||||
{/* Suspense handles the loading state for lazy-loaded routes */}
|
<Suspense fallback={<PageLoader />}>
|
||||||
<Suspense fallback={<PageLoader />}>
|
<Routes>
|
||||||
<Routes>
|
<Route path="/" element={<Home />} />
|
||||||
<Route path="/" element={<Home />} />
|
<Route path="/about" element={<About />} />
|
||||||
<Route path="/about" element={<About />} />
|
<Route path="/contact" element={<Contact />} />
|
||||||
<Route path="/contact" element={<Contact />} />
|
</Routes>
|
||||||
</Routes>
|
</Suspense>
|
||||||
</Suspense>
|
|
||||||
</main>
|
|
||||||
<Footer />
|
<Footer />
|
||||||
</BrowserRouter>
|
</BrowserRouter>
|
||||||
</LanguageProvider>
|
</LanguageProvider>
|
||||||
|
|||||||
@@ -1,20 +0,0 @@
|
|||||||
.skipLink {
|
|
||||||
position: absolute;
|
|
||||||
top: 1rem;
|
|
||||||
left: 1rem;
|
|
||||||
z-index: 100;
|
|
||||||
padding: 0.75rem 1.25rem;
|
|
||||||
background-color: var(--md-sys-color-primary);
|
|
||||||
color: var(--md-sys-color-on-primary);
|
|
||||||
font-weight: 600;
|
|
||||||
border-radius: var(--radius-md);
|
|
||||||
text-decoration: none;
|
|
||||||
transform: translateY(-200%);
|
|
||||||
transition: transform var(--transition-normal);
|
|
||||||
outline: none;
|
|
||||||
box-shadow: var(--md-sys-color-shadow) 0px 4px 8px 3px;
|
|
||||||
}
|
|
||||||
|
|
||||||
.skipLink:focus {
|
|
||||||
transform: translateY(0);
|
|
||||||
}
|
|
||||||
@@ -1,12 +0,0 @@
|
|||||||
import { useTranslation } from '../../i18n';
|
|
||||||
import styles from './SkipLink.module.css';
|
|
||||||
|
|
||||||
export function SkipLink() {
|
|
||||||
const { t } = useTranslation();
|
|
||||||
|
|
||||||
return (
|
|
||||||
<a href="#main-content" className={styles.skipLink}>
|
|
||||||
{t.nav.skipLink}
|
|
||||||
</a>
|
|
||||||
);
|
|
||||||
}
|
|
||||||
@@ -1,33 +0,0 @@
|
|||||||
// @vitest-environment jsdom
|
|
||||||
import { render, screen, cleanup } from '@testing-library/react';
|
|
||||||
import { describe, it, expect, afterEach } from 'vitest';
|
|
||||||
import { SkipLink } from '../SkipLink';
|
|
||||||
import { LanguageProvider } from '../../../i18n';
|
|
||||||
|
|
||||||
describe('SkipLink', () => {
|
|
||||||
afterEach(() => {
|
|
||||||
cleanup();
|
|
||||||
});
|
|
||||||
|
|
||||||
const renderWithProviders = (component: React.ReactNode) => {
|
|
||||||
return render(
|
|
||||||
<LanguageProvider>
|
|
||||||
{component}
|
|
||||||
</LanguageProvider>
|
|
||||||
);
|
|
||||||
};
|
|
||||||
|
|
||||||
it('renders correctly with correct href', () => {
|
|
||||||
renderWithProviders(<SkipLink />);
|
|
||||||
const link = screen.getByRole('link');
|
|
||||||
expect(link).toBeTruthy();
|
|
||||||
expect(link.getAttribute('href')).toBe('#main-content');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('renders translated text', () => {
|
|
||||||
renderWithProviders(<SkipLink />);
|
|
||||||
const link = screen.getByRole('link');
|
|
||||||
const text = link.textContent;
|
|
||||||
expect(text === 'Zum Inhalt springen' || text === 'Skip to content').toBe(true);
|
|
||||||
});
|
|
||||||
});
|
|
||||||
@@ -2,4 +2,3 @@ export { Button } from './Button';
|
|||||||
export { Card } from './Card';
|
export { Card } from './Card';
|
||||||
export { Input, Textarea } from './Input';
|
export { Input, Textarea } from './Input';
|
||||||
export { PageLoader } from './PageLoader';
|
export { PageLoader } from './PageLoader';
|
||||||
export { SkipLink } from './SkipLink';
|
|
||||||
|
|||||||
@@ -1,7 +1,6 @@
|
|||||||
export const de = {
|
export const de = {
|
||||||
// Navigation
|
// Navigation
|
||||||
nav: {
|
nav: {
|
||||||
skipLink: 'Zum Inhalt springen',
|
|
||||||
home: 'Startseite',
|
home: 'Startseite',
|
||||||
about: 'Über uns',
|
about: 'Über uns',
|
||||||
contact: 'Kontakt',
|
contact: 'Kontakt',
|
||||||
|
|||||||
@@ -3,7 +3,6 @@ import type { Translations } from './de';
|
|||||||
export const en: Translations = {
|
export const en: Translations = {
|
||||||
// Navigation
|
// Navigation
|
||||||
nav: {
|
nav: {
|
||||||
skipLink: 'Skip to content',
|
|
||||||
home: 'Home',
|
home: 'Home',
|
||||||
about: 'About',
|
about: 'About',
|
||||||
contact: 'Contact',
|
contact: 'Contact',
|
||||||
|
|||||||
@@ -25,6 +25,11 @@ describe('Security Utils', () => {
|
|||||||
const expected = '<script>alert("XSS")</script>';
|
const expected = '<script>alert("XSS")</script>';
|
||||||
expect(sanitizeInput(input)).toBe(expected);
|
expect(sanitizeInput(input)).toBe(expected);
|
||||||
});
|
});
|
||||||
|
|
||||||
|
it('escapes backticks', () => {
|
||||||
|
expect(sanitizeInput('`')).toBe('`');
|
||||||
|
expect(sanitizeInput('user`name')).toBe('user`name');
|
||||||
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
describe('isValidEmail', () => {
|
describe('isValidEmail', () => {
|
||||||
|
|||||||
@@ -14,7 +14,8 @@ export function sanitizeInput(input: string): string {
|
|||||||
.replace(/</g, "<")
|
.replace(/</g, "<")
|
||||||
.replace(/>/g, ">")
|
.replace(/>/g, ">")
|
||||||
.replace(/"/g, """)
|
.replace(/"/g, """)
|
||||||
.replace(/'/g, "'");
|
.replace(/'/g, "'")
|
||||||
|
.replace(/`/g, "`");
|
||||||
}
|
}
|
||||||
|
|
||||||
// Common disposable email providers and invalid domains
|
// Common disposable email providers and invalid domains
|
||||||
|
|||||||
Reference in New Issue
Block a user