Compare commits
1 Commits
sentinel-s
...
palette-ux
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
0808fcc2d5 |
@@ -27,8 +27,3 @@
|
||||
**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.
|
||||
**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 Injection in Template Strings
|
||||
**Vulnerability:** Standard HTML sanitization often ignores backticks (` `), which can be dangerous if the sanitized string is injected into a JavaScript template literal context.
|
||||
**Learning:** While HTML entities (`<`, `"`) protect HTML contexts, modern JS uses backticks for strings. Failing to escape them allows attackers to break out of the string boundary if the data is used in a JS context.
|
||||
**Prevention:** Explicitly replace backticks with ``` in sanitization routines intended for general-purpose use.
|
||||
|
||||
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 { Navbar, Footer, FancyCursor, ScrollToTop } from './components/layout';
|
||||
import { Home } from './pages/Home';
|
||||
import { PageLoader } from './components/ui';
|
||||
import { PageLoader, SkipLink } from './components/ui';
|
||||
import './styles/global.css';
|
||||
|
||||
// Lazy load pages to reduce initial bundle size.
|
||||
@@ -15,17 +15,20 @@ export function App() {
|
||||
return (
|
||||
<LanguageProvider>
|
||||
<BrowserRouter>
|
||||
<SkipLink />
|
||||
<ScrollToTop />
|
||||
<FancyCursor />
|
||||
<Navbar />
|
||||
{/* Suspense handles the loading state for lazy-loaded routes */}
|
||||
<Suspense fallback={<PageLoader />}>
|
||||
<Routes>
|
||||
<Route path="/" element={<Home />} />
|
||||
<Route path="/about" element={<About />} />
|
||||
<Route path="/contact" element={<Contact />} />
|
||||
</Routes>
|
||||
</Suspense>
|
||||
<main id="main-content" tabIndex={-1} style={{ flex: 1, outline: 'none' }}>
|
||||
{/* Suspense handles the loading state for lazy-loaded routes */}
|
||||
<Suspense fallback={<PageLoader />}>
|
||||
<Routes>
|
||||
<Route path="/" element={<Home />} />
|
||||
<Route path="/about" element={<About />} />
|
||||
<Route path="/contact" element={<Contact />} />
|
||||
</Routes>
|
||||
</Suspense>
|
||||
</main>
|
||||
<Footer />
|
||||
</BrowserRouter>
|
||||
</LanguageProvider>
|
||||
|
||||
20
src/components/ui/SkipLink.module.css
Normal file
20
src/components/ui/SkipLink.module.css
Normal file
@@ -0,0 +1,20 @@
|
||||
.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);
|
||||
}
|
||||
12
src/components/ui/SkipLink.tsx
Normal file
12
src/components/ui/SkipLink.tsx
Normal file
@@ -0,0 +1,12 @@
|
||||
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>
|
||||
);
|
||||
}
|
||||
33
src/components/ui/__tests__/SkipLink.test.tsx
Normal file
33
src/components/ui/__tests__/SkipLink.test.tsx
Normal file
@@ -0,0 +1,33 @@
|
||||
// @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,3 +2,4 @@ export { Button } from './Button';
|
||||
export { Card } from './Card';
|
||||
export { Input, Textarea } from './Input';
|
||||
export { PageLoader } from './PageLoader';
|
||||
export { SkipLink } from './SkipLink';
|
||||
|
||||
@@ -1,6 +1,7 @@
|
||||
export const de = {
|
||||
// Navigation
|
||||
nav: {
|
||||
skipLink: 'Zum Inhalt springen',
|
||||
home: 'Startseite',
|
||||
about: 'Über uns',
|
||||
contact: 'Kontakt',
|
||||
|
||||
@@ -3,6 +3,7 @@ import type { Translations } from './de';
|
||||
export const en: Translations = {
|
||||
// Navigation
|
||||
nav: {
|
||||
skipLink: 'Skip to content',
|
||||
home: 'Home',
|
||||
about: 'About',
|
||||
contact: 'Contact',
|
||||
|
||||
@@ -10,7 +10,6 @@ describe('Security Utils', () => {
|
||||
expect(sanitizeInput('foo & bar')).toBe('foo & bar');
|
||||
expect(sanitizeInput('"quotes"')).toBe('"quotes"');
|
||||
expect(sanitizeInput("'single quotes'")).toBe(''single quotes'');
|
||||
expect(sanitizeInput('`backticks`')).toBe('`backticks`');
|
||||
expect(sanitizeInput('>')).toBe('>');
|
||||
});
|
||||
|
||||
@@ -75,8 +74,6 @@ describe('Security Utils', () => {
|
||||
expect(isValidEmail('spam@mailinator.com')).toBe(false);
|
||||
expect(isValidEmail('bot@yopmail.com')).toBe(false);
|
||||
expect(isValidEmail('temp@temp-mail.org')).toBe(false);
|
||||
expect(isValidEmail('spam@sharklasers.com')).toBe(false);
|
||||
expect(isValidEmail('bot@maildrop.cc')).toBe(false);
|
||||
});
|
||||
|
||||
it('rejects blocked domains regardless of case', () => {
|
||||
|
||||
@@ -14,8 +14,7 @@ export function sanitizeInput(input: string): string {
|
||||
.replace(/</g, "<")
|
||||
.replace(/>/g, ">")
|
||||
.replace(/"/g, """)
|
||||
.replace(/'/g, "'")
|
||||
.replace(/`/g, "`");
|
||||
.replace(/'/g, "'");
|
||||
}
|
||||
|
||||
// Common disposable email providers and invalid domains
|
||||
@@ -26,13 +25,8 @@ const BLOCKED_DOMAINS = new Set([
|
||||
"yopmail.com",
|
||||
"temp-mail.org",
|
||||
"guerrillamail.com",
|
||||
"guerrillamail.net",
|
||||
"10minutemail.com",
|
||||
"trashmail.com",
|
||||
"sharklasers.com",
|
||||
"dispostable.com",
|
||||
"maildrop.cc",
|
||||
"getairmail.com",
|
||||
]);
|
||||
|
||||
/**
|
||||
|
||||
Reference in New Issue
Block a user