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

Fix regression introduced by feature for #17 #19

Merged
merged 1 commit into from
Apr 3, 2020

Conversation

cynthia
Copy link
Contributor

@cynthia cynthia commented Mar 30, 2020

#17 구현시 args에 jump_host를 넣는 스코프를 최소화하기 위해서 별도 루프를 돌렸으나, 적어도 한군데에서 경유 호출이 (go -> ssh) 일어나는것이 확인되어 쓸모는 없더라도 일단은 추가를 다 하는 형태로 바꿨습니다.

이 패치가 없으면 경유해서 ssh/scp 로 가는 명령어 수행시 (e.g. go) 에러가 납니다.

@cynthia
Copy link
Contributor Author

cynthia commented Mar 30, 2020

@kanghyojun 상기 버그 픽스가 없으면 정상 동작 안하는 명령어들이 있습니다. (CI가 계속 실패하지 않았다면 잡혔을텐데, 어쨋거나 테스트를 안돌려본 제 불찰입니다.)

@pbzweihander
Copy link

마스터로 리베이스해주실 수 있나요?

@kanghyojun
Copy link
Contributor

@pbzweihander 님이 리베이스만 하면 된다고 리뷰해주셨으니 리베이스 머지하겠습니다. 감사합니다.

@kanghyojun kanghyojun merged commit dca6090 into geofront-auth:master Apr 3, 2020
@cynthia
Copy link
Contributor Author

cynthia commented Apr 3, 2020

머지 감사합니다.

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.

None yet

4 participants