Compare commits
1 Commits
palette-sk
...
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.
|
||||||
|
|||||||
@@ -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,7 +15,6 @@ export function App() {
|
|||||||
return (
|
return (
|
||||||
<LanguageProvider>
|
<LanguageProvider>
|
||||||
<BrowserRouter>
|
<BrowserRouter>
|
||||||
<SkipLink />
|
|
||||||
<ScrollToTop />
|
<ScrollToTop />
|
||||||
<FancyCursor />
|
<FancyCursor />
|
||||||
<Navbar />
|
<Navbar />
|
||||||
|
|||||||
@@ -1,26 +0,0 @@
|
|||||||
.skipLink {
|
|
||||||
position: fixed;
|
|
||||||
top: 0;
|
|
||||||
left: 50%;
|
|
||||||
z-index: 1000;
|
|
||||||
transform: translateY(-100%) translateX(-50%);
|
|
||||||
padding: var(--space-sm) var(--space-md);
|
|
||||||
background-color: var(--md-sys-color-primary-container);
|
|
||||||
color: var(--md-sys-color-on-primary-container);
|
|
||||||
font-family: var(--md-sys-typescale-body-font);
|
|
||||||
font-weight: 600;
|
|
||||||
text-decoration: none;
|
|
||||||
border-radius: 0 0 var(--radius-md) var(--radius-md);
|
|
||||||
box-shadow: 0 4px 6px rgba(0, 0, 0, 0.2);
|
|
||||||
transition: transform var(--transition-fast);
|
|
||||||
opacity: 0;
|
|
||||||
pointer-events: none;
|
|
||||||
}
|
|
||||||
|
|
||||||
.skipLink:focus {
|
|
||||||
transform: translateY(0) translateX(-50%);
|
|
||||||
opacity: 1;
|
|
||||||
pointer-events: auto;
|
|
||||||
outline: 2px solid var(--md-sys-color-primary);
|
|
||||||
outline-offset: 2px;
|
|
||||||
}
|
|
||||||
@@ -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.skipToContent}
|
|
||||||
</a>
|
|
||||||
);
|
|
||||||
}
|
|
||||||
@@ -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: {
|
||||||
skipToContent: 'Zum Hauptinhalt 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: {
|
||||||
skipToContent: 'Skip to main content',
|
|
||||||
home: 'Home',
|
home: 'Home',
|
||||||
about: 'About',
|
about: 'About',
|
||||||
contact: 'Contact',
|
contact: 'Contact',
|
||||||
|
|||||||
@@ -46,7 +46,7 @@ export function About() {
|
|||||||
const location = useLocation();
|
const location = useLocation();
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<main id="main-content" tabIndex={-1} className={styles.about} key={location.key}>
|
<main className={styles.about} key={location.key}>
|
||||||
{/* Hero Section */}
|
{/* Hero Section */}
|
||||||
<section className={styles.hero}>
|
<section className={styles.hero}>
|
||||||
<div className="container">
|
<div className="container">
|
||||||
|
|||||||
@@ -133,7 +133,7 @@ export function Contact() {
|
|||||||
};
|
};
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<main id="main-content" tabIndex={-1} className={styles.contact}>
|
<main className={styles.contact}>
|
||||||
{/* Hero */}
|
{/* Hero */}
|
||||||
<section className={styles.hero}>
|
<section className={styles.hero}>
|
||||||
<div className="container">
|
<div className="container">
|
||||||
|
|||||||
@@ -2,7 +2,7 @@ import { Hero, Services } from '../components/sections';
|
|||||||
|
|
||||||
export function Home() {
|
export function Home() {
|
||||||
return (
|
return (
|
||||||
<main id="main-content" tabIndex={-1}>
|
<main>
|
||||||
<Hero />
|
<Hero />
|
||||||
<Services />
|
<Services />
|
||||||
</main>
|
</main>
|
||||||
|
|||||||
@@ -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