sentry-javascript icon indicating copy to clipboard operation
sentry-javascript copied to clipboard

AWS Lambda integration - Improve error message & allow for port change

Open t-h-man opened this issue 2 years ago • 3 comments

Problem Statement

Currently our lambda integration is running on port 8000. If there is an another service running on that port, the user will see ERROR: bind to interface failed caused by: Address already in use (os error 98)

a) we should improve the error message to make it more informative b) we can consider allowing port switch to avoid the above issue

Solution Brainstorm

Posssible solution: a) start relay on an obscure port. b) start by default on 8000, but make it configurable via env variable (needs changes in sdk and relay, can be complicated to work on all different js frameworks.

So better go with a)

t-h-man avatar Jul 12 '22 13:07 t-h-man

I am having that message but I do not know why. I think I am not running any service on that port

titiloxx avatar Jul 17 '22 10:07 titiloxx

Also running into this issue when using the aws lambda insights layer.

thyming avatar Aug 04 '22 03:08 thyming

We will just pick an obscure port here, exposing this as config doesn't hold much value. Will update issue when fixed/shipped.

sl0thentr0py avatar Aug 04 '22 14:08 sl0thentr0py

@sl0thentr0py are you still taking a look at this? Cleaning up some issues for maintenance stuff so wanted to see if this should be backlogged.

AbhiPrasad avatar Oct 06 '22 13:10 AbhiPrasad

@AbhiPrasad it's already on our backlog, you can remove it from yours.

sl0thentr0py avatar Oct 06 '22 13:10 sl0thentr0py