Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow a custom name, not just auto-generated #130

Merged
merged 3 commits into from
Feb 9, 2024
Merged

Conversation

alexellis
Copy link
Member

Description

Allow a custom name, not just auto-generated

This affected @richardcase when a team-mate deleted his VM from a common company cloud account not knowing what it was for, or who it belonged to.

How are existing users impacted? What migration steps/scripts do we need?

No change, if you leave the name blank, it works as before.

Checklist:

I've updated the help text for the command.

This affected @richardcase when a team-mate deleted his VM
from a common company cloud account not knowing what it was
for, or who it belonged to.

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <[email protected]>
This commit makes HTTPS tunnels the default, and a TCP tunnel
can still be created with --tcp.

Tested against digitalocean

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <[email protected]>
@alexellis
Copy link
Member Author

Here's a custom tunnel name, with a HTTPS domain using LE's staging issuer to avoid using up the available rate limit.

Screenshot 2024-02-09 at 09 32 06

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <[email protected]>
@alexellis alexellis merged commit 80a0678 into master Feb 9, 2024
0 of 2 checks passed
@alexellis alexellis deleted the custom_name branch February 9, 2024 09:35
@richardcase
Copy link

Thanks @alexellis . I just tried this and it works.

@alexellis
Copy link
Member Author

Thank you. The release will be available via arkade get very shortly.

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.

2 participants