__dirname undefined

[y-nk]

when deploying on edge, __dirname is not defined, also it's said as a best practice to bundle all js in a single file to optimize cold start. hence, the code in here:

fast-geoip/index.ts

Line 12 in a5d3c6b

const DATA_DIR = path.join(path.dirname(__dirname), "data");

breaks execution of a lambda@edge using this package.

i was wondering if you had any suggestion for a suitable workaround?

[Doc999tor]

1. Yes, you're right. For fixing it, you can use a workaround using path.dirname(url.fileURLToPath(import.meta.url)) - from here
2. This repo is not maintained for a long time. If you don't build the MaxMind db in the @edge, you can use the fork, I actively maintain: https://github.com/Doc999tor/fast-geoip
3. You can create a PR with __dirname fix to my repo, I'd be happy to let this great project evolve

[y-nk]

@Doc999tor thanks for the fix provided, i will propose a PR with the fix you have mentioned.

meanwhile it gets accepted and merged, and to give a little context to visitors of this issue, since i'm using esbuild to pack my lambda@edge it can be solved without modifying the code by adding banner.js containing const __dirname = url.fileURLToPath(import.meta.url).

[y-nk]

for people following this. for edge functions in aws, you don't need this package anymore since geoheaders come for free in cloudfront.