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

TypeError: request.headers.split is not a function #12025

Closed
Alexandero89 opened this issue Dec 3, 2024 · 3 comments · Fixed by #12061
Closed

TypeError: request.headers.split is not a function #12025

Alexandero89 opened this issue Dec 3, 2024 · 3 comments · Fixed by #12061
Labels
in linear Issue or PR has been created in Linear for internal review Released

Comments

@Alexandero89
Copy link

Bug Description

After enabling the sentry node module using:
NODE_FUNCTION_ALLOW_EXTERNAL=@sentry
I see a lot of exceptions in the log when using the sentry module in a custom code node:
TypeError: request.headers.split is not a function
It also crashes n8ns GUI most of the time. So it restarts a lot!

I found this issue which is exactly the mentioned error getsentry/sentry-javascript#10936

The currently sentry module is version 7.87.0 and this bug was fixed in 7.106.0 ( see getsentry/sentry-javascript#10936 (comment))

Please bump the sentry version at least to version 7.106.0 if not higher.
The newest version right now is 8.42.0

To Reproduce

  1. Install n8n
  2. enable @sentry node module via environmental variable in docker-compose
  3. use sentry module in a code node

Expected behavior

not throw an exception and crash the Editor/GUI

Operating System

docker-compose

n8n Version

1.69.2

Node.js Version

...

Database

SQLite (default)

Execution mode

main (default)

@Joffcom
Copy link
Member

Joffcom commented Dec 3, 2024

Hey @Alexandero89,

We have created an internal ticket to look into this which we will be tracking as "GHC-552"

@Joffcom Joffcom added the in linear Issue or PR has been created in Linear for internal review label Dec 3, 2024
@Joffcom
Copy link
Member

Joffcom commented Dec 5, 2024

Hey @Alexandero89

Can you provide a minimal workflow example that shows this happening so we have some thing to test against to make sure that updating will resolve it.

@janober
Copy link
Member

janober commented Dec 11, 2024

Fix got released with n8n@1.72.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in linear Issue or PR has been created in Linear for internal review Released
Projects
None yet
3 participants