Page MenuHomeDevCentral

Adopt local tunnel
ClosedPublic

Authored by MoustaphaAs on Nov 27 2024, 17:27.
Tags
None
Referenced Files
F3981908: D3600.diff
Wed, Jan 8, 02:39
Unknown Object (File)
Sun, Jan 5, 14:37
Unknown Object (File)
Fri, Jan 3, 08:43
Unknown Object (File)
Mon, Dec 30, 22:13
Unknown Object (File)
Sun, Dec 29, 19:44
Unknown Object (File)
Sun, Dec 29, 17:09
Unknown Object (File)
Sat, Dec 21, 09:21
Unknown Object (File)
Sat, Dec 21, 08:41
Subscribers
None

Details

Summary

Create a tunnel from the local development server
(running on config.dev.port) to a publicly accessible URL.

Diff Detail

Repository
rUPSECTION Upsection
Lint
Lint Passed
Unit
No Test Coverage
Branch
localtunnel
Build Status
Buildable 5744
Build 6026: arc lint + arc unit

Event Timeline

MoustaphaAs created this revision.
MoustaphaAs retitled this revision from Adopt local tunnel and reorder imports to Adopt local tunnel.Nov 27 2024, 18:03
components/gulpfile.js
18–19

If you require it at line 23, you probably don't need import here.

dereckson requested changes to this revision.Nov 27 2024, 21:03
dereckson added inline comments.
components/gulpfile.js
174

Why move the location of the server() function?

That breaks the Git history.

180

No need to define it if it's false

184

(1) We can put the new code in a setupTunnel() function.

See https://jeroendedauw.github.io/slides/craftmanship/functions/#/1 to understand the logic to use short clean functions.

(2) In the localtunnel configuration, we should also take the value of tunnel setting to allow the subdomain.

(3) We should ensure the code also works when tunnel IS NOT defined in config.yml (no tunnel in that case)

(4) The done method should be called AFTER everything is called, including the tunnel, so instead of passing it to browser.init, we can call it afterwards

We would then have something like:

browser.init(...);

if (config.dev.tunnel) {
    setupTunnel()
}

done()
This revision now requires changes to proceed.Nov 27 2024, 21:03
MoustaphaAs marked an inline comment as done.

setupTunnel() function defined, allowing the use of a subdomain and code working when the tunnel is not defined

MoustaphaAs marked 2 inline comments as done.

Verify the existence of the subdomain; if not, generate a random one.

Verify the existence of tunnel attributs

MoustaphaAs marked an inline comment as done.

Revision

This revision is now accepted and ready to land.Nov 28 2024, 10:49
This revision was automatically updated to reflect the committed changes.