✨ `[headers]` Added utilities for header sanitisation #734

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

acabarbaye merged 3 commits into master from headers-management

Oct 27, 2025

Contributor

acabarbaye commented Oct 24, 2025

Description

fix some bugs
improve the sanitisation

Test Coverage

This change is covered by existing or additional automated tests.
Manual testing has been performed (and evidence provided) as automated testing was not feasible.
Additional tests are not required for this change (e.g. documentation update).

acabarbaye added 2 commits

October 24, 2025 17:04


          ✨ [headers] Added utilities for header sanitisation

c3cad55


          💚 linting

17950bf

acabarbaye force-pushed the headers-management branch from 7fdc7ae to 17950bf Compare

October 24, 2025 18:41

joshjennings98 reviewed

View reviewed changes

utils/http/headers/headers.go Outdated Show resolved Hide resolved

joshjennings98 reviewed

View reviewed changes

utils/http/headers/headers.go Outdated Show resolved Hide resolved

joshjennings98 reviewed

View reviewed changes

utils/http/headers/headers.go Outdated Show resolved Hide resolved

joshjennings98 reviewed

View reviewed changes

utils/http/headers/headers.go Outdated Show resolved Hide resolved

joshjennings98 reviewed

View reviewed changes

utils/http/headers/headers.go Outdated Show resolved Hide resolved

joshjennings98 previously approved these changes

View reviewed changes

Contributor

joshjennings98 left a comment

Just a few minor comments but this looks good


          💚 Address review comments

9226b9e

acabarbaye dismissed joshjennings98’s stale review via

9226b9e

October 27, 2025 09:48

acabarbaye force-pushed the headers-management branch from 318eebe to 9226b9e Compare

October 27, 2025 09:48

acabarbaye changed the title ~~✨ [headers] Added utilities for header sanitisation~~ ✨ [headers] Added utilities for header sanitisation

acabarbaye merged commit f0a8c8e into master

6 checks passed

acabarbaye deleted the headers-management branch

October 27, 2025 09:53

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet