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

feat: select ssh server address #12

Merged
merged 3 commits into from
Feb 14, 2024
Merged

feat: select ssh server address #12

merged 3 commits into from
Feb 14, 2024

Conversation

JyJyJcr
Copy link
Collaborator

@JyJyJcr JyJyJcr commented Feb 13, 2024

server option --proxy-for or -p specifies the address (and port) of the ssh server.
senarios:

  • the ssh server port is changed from default 22, in order to reduce the risk of attacks.
  • the server is behind the proxy.

@JyJyJcr
Copy link
Collaborator Author

JyJyJcr commented Feb 13, 2024

I can't resolve the CI failure of job "FreeBSD-x86_64" in my fork. It doesn't seem my fault, because the re-run of the job in the master branch also failed. Do you know any resolution or workaround? @oowl

README.md Outdated Show resolved Hide resolved
@oowl
Copy link
Owner

oowl commented Feb 14, 2024

Hi @JyJyJcr, I have rerun my repo action https://github.com/oowl/quicssh-rs/actions/runs/7882873596/job/21549775923, but it seems everything works fine.

@oowl
Copy link
Owner

oowl commented Feb 14, 2024

It's so wired, everything is fine in my repo action, I see the current pull request test passed, but your repo action failed.

@oowl oowl changed the title select server address feat: select ssh server address Feb 14, 2024
@oowl
Copy link
Owner

oowl commented Feb 14, 2024

Thanks!

@oowl oowl merged commit 29c2b5b into oowl:master Feb 14, 2024
7 checks passed
@JyJyJcr
Copy link
Collaborator Author

JyJyJcr commented Feb 20, 2024

Github Action's problem was resolved by deleting all caches. It seems that once build failed it remains something buggy.
This result doesn't associate with the pull request, but I think it is better to comment this, for the people annoying with similar problems.

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