Page MenuHomeDevCentral

Adopt local tunnel
Needs RevisionPublic

Authored by MoustaphaAs on Wed, Nov 27, 17:27.

Details

Reviewers
dereckson
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 5745
Build 6027: 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.Wed, Nov 27, 18:03
components/gulpfile.js
18

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

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

Why move the location of the server() function?

That breaks the Git history.

179

No need to define it if it's false

183

(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.Wed, Nov 27, 21:03