Skip to content

Typscript version and upgrade#226

Open
muka wants to merge 13 commits intolocaltunnel:masterfrom
muka:feat/upgrade_ts
Open

Typscript version and upgrade#226
muka wants to merge 13 commits intolocaltunnel:masterfrom
muka:feat/upgrade_ts

Conversation

@muka
Copy link

@muka muka commented Apr 14, 2025

Hi,
I invested some time to upgrade dependencies and also moved to typescript (I know, it is an opinonated choice).
I also filtered out some PR and added them manually.

client counterpart localtunnel/localtunnel#709

@TheBoroer TheBoroer self-assigned this May 5, 2025
ENTRYPOINT ["node", "-r", "esm", "./bin/server"]
RUN npm run build

FROM node:22-alpine

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice

Copy link

@raul338 raul338 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work!!

Just leaving my 2 cents here.

I'm testing this branch with the official client version 2.0.2 installed using npm, with these changes it works mostly fine.

secure: opts.secure,
domain: opts.domain,
landing: opts.landing,
secret: opts.secret,
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

opts.range should be here

.option('--domain, -d <string>', 'Specify the base domain name. This is optional if hosting localtunnel from a regular example.com domain. This is required if hosting a localtunnel server from a subdomain (i.e. lt.example.dom where clients will be client-app.lt.example.com)')
.option('--landing, -l <string>', 'The landing page for redirect from root domain', 'https://localtunnel.github.io/www/')

.option('--max-sockets', 'maximum number of tcp sockets each client is allowed to establish at one time (the tunnels)', intParser, 10)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be '--max-sockets <number>' as first parameter

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants