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

Change to use hkps over port 443 instead of 11371 to resolve issue https://github.com/actions/runner-images/discussions/9963 #10007

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

lukasbbuehler
Copy link

@lukasbbuehler lukasbbuehler commented Jun 7, 2024

Update: Changed in the second commit to hkps over port 443 to improve security. Works also like a charm.

Change to use port 80 instead of 11371 to fix this issue: #9963
We tested the commands manually on our machines to verify if the script would work.

Description

New tool, Bug fixing, or Improvement?
Please include a summary of the change and which issue is fixed. Also include relevant motivation and context.
For new tools, please provide total size and installation time.

Related issue:

Check list

  • Related issue / work item is attached
  • Tests are written (if applicable)
  • Documentation is updated (if applicable)
  • Changes are tested and related VM images are successfully generated

Change to use port 80 instead of 11371
Changed to hkps and port 443 for added security
@lukasbbuehler lukasbbuehler changed the title Change to use port 80 instead of 11371 to resolve issue https://github.com/actions/runner-images/discussions/9963 Change to use hkps over port 443 instead of 11371 to resolve issue https://github.com/actions/runner-images/discussions/9963 Jun 10, 2024
@tokugler
Copy link

tokugler commented Jul 5, 2024

Could someone approve this? We're facing the same issue.

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.

3 participants