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.

Sign up for GitHub

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

Jump to bottom

#4 Fix (rmate, net-tools) #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

#4 Fix (rmate, net-tools) #5

wants to merge 1 commit into from

Conversation

ThomasKraaibeek
Copy link
Contributor

Hi!
I just downloaded the content of https://raw.githubusercontent.com/aurora/rmate/master/rmate and saved the file to the container for atom support, since the curl action didn't seem to work. Hope this is fine. Everything should be working properly now.

  • Added rmate file
  • Removed curl for rmate file
  • Added yum install net-tools (needed in bootstrap.sh line 4 to get container ip)

Fixes #4

@pozgo
Copy link
Member

pozgo commented Feb 5, 2021

Firstly. Thanks for the PR.

I have posted a comment on the issue at hand

I think we could add -k to curl command so it would look like:

curl -k -o /usr/local/bin/rmate https://raw.githubusercontent.com/aurora/rmate/master/rmate

Test defualt

[root@a97a38a19a23 ~]# curl -o /usr/local/bin/rmate https://raw.githubusercontent.com/aurora/rmate/master/rmate
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 10553  100 10553    0     0  62885      0 --:--:-- --:--:-- --:--:-- 63191

Test with -k

[root@a97a38a19a23 ~]# curl -k -o /usr/local/bin/rmate https://raw.githubusercontent.com/aurora/rmate/
master/rmate
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 10553  100 10553    0     0  59342      0 --:--:-- --:--:-- --:--:-- 5962

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Curl problem
2 participants
@ThomasKraaibeek @pozgo