Skip to content

Commit ff72324

Browse files
committed
docs: catch the last Fiber v2 declarations
Round-2 review found the sweep had still missed the package godoc (internal/web/doc.go, the surface pkg.go.dev renders), docs/INDEX.md and cmd/AGENTS.md; a repo-wide grep now finds zero live Fiber v2 claims (dated plan archives under docs/superpowers/ excepted). Also aligns the AGENTS.md CSRF example with the real config (extractors.FromForm, not FromHeader), plus the touched files' pre-existing markdownlint findings. Assisted-by: claude-code:claude-fable-5 Agent-Session: https://claude.ai/code/session_01L7tF9XuJfAfFk4yuY5KfPK Agent-Host: 0493f0 Signed-off-by: Sebastian Mendel <info@sebastianmendel.de>
1 parent 064f305 commit ff72324

4 files changed

Lines changed: 20 additions & 16 deletions

File tree

cmd/AGENTS.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,12 @@
44

55
## Overview
66

7-
The `cmd/` directory contains the main entry point for the LDAP Manager application. This follows Go's standard project layout for executable binaries.
7+
The `cmd/` directory contains the main entry point for the LDAP Manager application. This follows Go's standard
8+
project layout for executable binaries.
89

910
- **Location**: `cmd/ldap-manager/main.go`
1011
- **Purpose**: CLI initialization, configuration parsing, server startup
11-
- **Framework**: Uses Fiber v2 web framework via `internal/web` package
12+
- **Framework**: Uses Fiber v3 web framework via `internal/web` package
1213

1314
## Setup & Environment
1415

docs/INDEX.md

Lines changed: 15 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,8 @@ Comprehensive technical analysis and session documentation (21 reports):
135135
| [**Frontend Analysis**](../claudedocs/comprehensive-frontend-analysis.md) | Complete frontend architecture review | 2025-09-30 |
136136
| [**CSS Analysis**](../claudedocs/css-analysis.md) | CSS structure and optimization opportunities | 2025-09-30 |
137137

138-
**Purpose:** These reports provide deep technical insights for developers, architects, and operations teams. Generated during development sessions for knowledge preservation and decision documentation.
138+
**Purpose:** These reports provide deep technical insights for developers, architects, and operations teams. Generated
139+
during development sessions for knowledge preservation and decision documentation.
139140

140141
---
141142

@@ -250,7 +251,7 @@ LDAP_POOL_HEALTH_CHECK_INTERVAL=30s
250251

251252
## 🏗️ Project Structure
252253

253-
```
254+
```text
254255
ldap-manager/
255256
├── cmd/ldap-manager/ # Application entry point
256257
├── internal/ # Core application code
@@ -337,27 +338,28 @@ Current project health indicators:
337338

338339
### Official Documentation
339340

340-
- **Go:** https://go.dev/doc/
341-
- **Fiber v2:** https://docs.gofiber.io/
342-
- **Templ:** https://templ.guide/
343-
- **TailwindCSS:** https://tailwindcss.com/docs
344-
- **go-ldap:** https://pkg.go.dev/github.com/go-ldap/ldap/v3
341+
- **Go:** <https://go.dev/doc/>
342+
- **Fiber v3:** <https://docs.gofiber.io/>
343+
- **Templ:** <https://templ.guide/>
344+
- **TailwindCSS:** <https://tailwindcss.com/docs>
345+
- **go-ldap:** <https://pkg.go.dev/github.com/go-ldap/ldap/v3>
345346

346347
### Related Projects
347348

348349
- **simple-ldap-go v1.5.0:** Custom LDAP wrapper with indexed cache
349-
- **Our Contribution:** [PR #45](https://github.com/netresearch/simple-ldap-go/pull/45) - Multi-key indexed cache (287x improvement)
350+
- **Our Contribution:** [PR #45](https://github.com/netresearch/simple-ldap-go/pull/45) - Multi-key indexed cache
351+
(287x improvement)
350352
- **Status:** Merged and released in v1.5.0
351-
- **GitHub:** https://github.com/netresearch/simple-ldap-go
353+
- **GitHub:** <https://github.com/netresearch/simple-ldap-go>
352354
- **BBolt:** Embedded key-value database for sessions
353355
- **Use Case:** Persistent session storage for development and production
354-
- **GitHub:** https://github.com/etcd-io/bbolt
356+
- **GitHub:** <https://github.com/etcd-io/bbolt>
355357

356358
---
357359

358360
## 🆘 Getting Help
359361

360-
### Documentation Not Found?
362+
### Documentation Not Found
361363

362364
1. **Check AGENTS.md:** AI guidelines may have what you need
363365
2. **Search codebase:** `grep -r "pattern" internal/`
@@ -450,6 +452,7 @@ Potential future enhancements (not committed):
450452

451453
---
452454

453-
_This index is maintained automatically and manually. Last comprehensive update: 2025-09-30 (Enhanced with PR #267 details, claudedocs integration, and API module reference)_
455+
This index is maintained automatically and manually. Last comprehensive update: 2025-09-30 (Enhanced with PR #267
456+
details, claudedocs integration, and API module reference)
454457

455458
**📌 Bookmark this page** - it's your hub for all LDAP Manager documentation.

internal/web/AGENTS.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -221,7 +221,7 @@ if err := validate.Struct(req); err != nil {
221221
```go
222222
// CSRF middleware enabled for state-changing operations
223223
app.Use(csrf.New(csrf.Config{
224-
Extractor: extractors.FromHeader("X-CSRF-Token"),
224+
Extractor: extractors.FromForm("csrf_token"), // matches createCSRFConfig in server.go
225225
CookieName: "csrf_",
226226
CookieSameSite: "Strict",
227227
IdleTimeout: 1 * time.Hour,

internal/web/doc.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
// Package web provides the HTTP server and web interface for LDAP Manager.
22
//
3-
// This package implements a complete web application using the Fiber v2 framework,
3+
// This package implements a complete web application using the Fiber v3 framework,
44
// providing HTTP handlers, middleware, session management, template rendering,
55
// and static asset serving for LDAP directory management operations.
66
//

0 commit comments

Comments
 (0)