search

hapijs / good

hapi process monitoring
Other
525 stars 161 forks source link

request.id is now request.info.id since Hapi 17 #580

Closed simlevesque closed 6 years ago

simlevesque commented 6 years ago

Since Hapi 17, request.id was moved to request.info.id.

Source: https://github.com/hapijs/hapi/issues/3658

Moved request.id to request.info.id.

simlevesque commented 6 years ago

I guess we should add a test for this regression. I can do that too if you need.

EdouardDem commented 6 years ago

I've tried the same fix and I had the request id back in my logs.

arb commented 6 years ago

Yeah let's add a test for this since it caused a regression.

simlevesque commented 6 years ago

I added some assertions so it does not happen again :)

lock[bot] commented 4 years ago

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.