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

🐛 : Custom Image - Repository URL with Port Number #729

Open
candidson opened this issue Sep 19, 2022 · 4 comments
Open

🐛 : Custom Image - Repository URL with Port Number #729

candidson opened this issue Sep 19, 2022 · 4 comments
Labels
🐛 bug Something isn't working

Comments

@candidson
Copy link

candidson commented Sep 19, 2022

Describe the bug
In the current implementation, the frontend automatically splits the custom image repository URL by ":". However this wouldn't allow the usage of port number in the URL; for example "my-custom-repository:5000/myterraform/myimage:latest" where "latest" is actually the tag, wouldn't work.

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'Module edition', and select "Custom" for Terraform Image
  2. Add "my-custom-repository:5000/myterraform/myimage:latest" as custom image

Expected behavior
Repository: my-custom-repository:5000/myterraform/myimage
Tag: latest

Screenshots
image

Desktop (please complete the following information):

  • OS: OSX
  • Browser Edge
  • Version 105.0.1343.42

Additional context

@candidson candidson added the 🐛 bug Something isn't working label Sep 19, 2022
@candidson
Copy link
Author

I guess the following lines in "terraform-image-input.vue" should be removed? Or any other ideas?

image

@juwit
Copy link
Member

juwit commented Sep 19, 2022

I guess the following lines in "terraform-image-input.vue" should be removed? Or any other ideas?

image

Hi @candidson
I think you can replace the line 115 with a regex matcher instead of a split.

Something like that:

[_,this.image.repository, this.image.tag] = /(\S*):(\S*)/.exec(value);

@candidson
Copy link
Author

candidson commented Sep 25, 2022

Hi @juwit
THanks a lot. I have worked on that and have it ready. I have also added some changes in the terraform-mustache file, to allow using custom-terraform images with non-root permissions: indeed the script in there was always installing "curl" - now "curl" is installed only if not existing. Please expect my PR in the next few days.

Regards

@candidson
Copy link
Author

Hi @juwit
I have finally submitted the PR: #730

Regards

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants