Feature implementation from commits c6e0d80..43ad9f7

[codeOwlAI]

PR Summary

Streamline Context API and Enhance HTTP Standards Compliance

Overview

This PR refactors custom context handling with a new simplified API (NewWithCustomCtx) while improving HTTP standards compliance across several components. It enhances cookie handling, authentication security, header parsing, and adds expanded test coverage.

Change Types

Type	Description
Enhancement	Simplified custom context API with NewWithCustomCtx
Enhancement	Improved HTTP standards compliance (RFC 9111, RFC 3986)
Enhancement	Better security in BasicAuth middleware
Enhancement	More robust header parsing and content-type handling
Refactor	Consolidated request handlers and simplified context logic
Test	Expanded test coverage for edge cases and security features

---

Affected Modules

Module / File	Change Description
app.go	Added NewWithCustomCtx, simplified handlers, renamed internal functions
ctx.go/ctx_interface.go	Improved standards compliance, refactored context handling
router.go	Consolidated request handlers, removed unused imports
basicauth/	Added charset support and optional password storage
cache/	Improved directive handling with case-insensitivity
helpers.go	Optimized cache checking, removed redundant methods
state.go	Refactored getters with generic implementation
Various test files	Expanded test coverage for edge cases

---

Notes for Reviewers

• The custom context API has been significantly simplified with NewWithCustomCtx function replacing separate New() and NewCtxFunc() calls
• HTTP header parsing (Range, MIME types) is now more robust and standards-compliant
• BasicAuth middleware has enhanced security with optional password storage

[codeowlai]

🐛 Correctness Issue

Breaking Change: Removed Custom Context Handler Support.

The change eliminates support for custom context functions, which can break applications relying on this feature for specialized routing behavior.

Current Code (Diff):

- 	_, err = c.app.next(c)
+ 	if c.app.newCtxFunc != nil {
+ 		_, err = c.app.nextCustom(c)
+ 	} else {
+ 		_, err = c.app.next(c)
+ 	}

📝 Committable suggestion

‼️ IMPORTANT
Trust, but verify! 🕵️ Please review this suggestion with the care of a code archaeologist - check that it perfectly replaces the highlighted code, preserves all lines, maintains proper indentation, and won't break anything in production. Your future self will thank you! 🚀

Suggested change

	_, err = c.app.next(c)
	if c.app.newCtxFunc != nil {
	_, err = c.app.nextCustom(c)
	} else {
	_, err = c.app.next(c)
	}

[codeowlai]

🐛 Correctness Issue

Breaking API change: Return type modification.

Changing the return type of AcquireCtx from Ctx to CustomCtx will break existing code that expects the Ctx interface type.

Current Code (Diff):

+ func (app *App) AcquireCtx(fctx *fasthttp.RequestCtx) CustomCtx {
- func (app *App) AcquireCtx(fctx *fasthttp.RequestCtx) Ctx {

📝 Committable suggestion

‼️ IMPORTANT
Trust, but verify! 🕵️ Please review this suggestion with the care of a code archaeologist - check that it perfectly replaces the highlighted code, preserves all lines, maintains proper indentation, and won't break anything in production. Your future self will thank you! 🚀

Suggested change

	func (app *App) AcquireCtx(fctx *fasthttp.RequestCtx) CustomCtx {
	func (app *App) AcquireCtx(fctx *fasthttp.RequestCtx) Ctx {

🔄 Dependencies Affected

app_test.go

Function: TestApp

Issue: Type assertion to (*DefaultCtx) will fail if CustomCtx is not compatible with DefaultCtx

Suggestion: Update type assertions or ensure CustomCtx is compatible with DefaultCtx

Proposed Code:

c, _ := app.AcquireCtx(fctx).(*DefaultCtx)
c, _ := app.AcquireCtx(&fasthttp.RequestCtx{}).(*DefaultCtx)

---

[codeowlai]

🐛 Correctness Issue

Breaking API change: Parameter type modification.

Changing the parameter type of ReleaseCtx from Ctx to CustomCtx will break existing code that passes Ctx interface instances.

Current Code (Diff):

+ func (app *App) ReleaseCtx(c CustomCtx) {
- func (app *App) ReleaseCtx(c Ctx) {

📝 Committable suggestion

‼️ IMPORTANT
Trust, but verify! 🕵️ Please review this suggestion with the care of a code archaeologist - check that it perfectly replaces the highlighted code, preserves all lines, maintains proper indentation, and won't break anything in production. Your future self will thank you! 🚀

Suggested change

	func (app *App) ReleaseCtx(c CustomCtx) {
	func (app *App) ReleaseCtx(c Ctx) {

🔄 Dependencies Affected

app.go

Function: Unknown

Issue: Existing code calls ReleaseCtx with Ctx interface instances

Suggestion: Update all callers to use CustomCtx or keep the original parameter type

Proposed Code:

defer app.ReleaseCtx(c)

---

app_test.go

Function: Unknown

Issue: Multiple test functions call ReleaseCtx with Ctx interface instances

Suggestion: Update all test callers to use CustomCtx or keep the original parameter type

Proposed Code:

app.ReleaseCtx(c)

---

bind_test.go

Function: Unknown

Issue: Test code calls ReleaseCtx with Ctx interface instances

Suggestion: Update test code to use CustomCtx or keep the original parameter type

Proposed Code:

defer app.ReleaseCtx(c)

---