Skip to content
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.

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

force header case for a single header #134

Open
yovanoc opened this issue Mar 4, 2024 · 3 comments
Open

force header case for a single header #134

yovanoc opened this issue Mar 4, 2024 · 3 comments
Labels
t-tooling Issues with this label are in the ownership of the tooling team.

Comments

@yovanoc
Copy link

yovanoc commented Mar 4, 2024

is there a way to override per-request header casing (e.g choosing a uppercase for a single header even when http2) ?

@B4nan
Copy link
Member

B4nan commented Mar 4, 2024

i think the headers are normalized directly in got (to lower case), sounds like a wontfix to me.

@B4nan B4nan added the t-tooling Issues with this label are in the ownership of the tooling team. label Mar 4, 2024
@yovanoc
Copy link
Author

yovanoc commented Mar 4, 2024

In got? I think this is happening here in got-scraping:

fixedHeaders.set(entry[0].toLowerCase(), entry);

So the only way will be to override got-scraping and add a before request hook to handle specific case like this?

@B4nan
Copy link
Member

B4nan commented Mar 4, 2024

Yes but even if we hadn't done that, it would have happened on the lower level, as got does the same

sindresorhus/got#1335

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t-tooling Issues with this label are in the ownership of the tooling team.
Projects
None yet
Development

No branches or pull requests

2 participants