Skip to content

Conversation

marcy-terui
Copy link
Contributor

@marcy-terui marcy-terui commented Nov 6, 2017

ref: #43

Sorry for writing in Japanese 🙇

とりあえず、最低限の動作確認までできたのでご相談したいこともあったので送らせていただきます。
下記についてご意見いただきたく 🙇

現状の悩みとしては、以下のようなものです。

  • cffiからnullableな値を受け取る方法が分からなかったので未指定項目は空文字列という仕様
    • 私的な基準ではありますが、ここはまだ許容できる
  • できればポートも指定できるようにしたい
    • ポートの未指定時が例えば 0-1 というように定義したとして、内部で 22 に置き換わるというのはイマイチ
    • かといって、SSHのポートを変えたいケースは多くはないので 22 を渡させるというインターフェースは避けたい

現状では以下を案として考えています

  1. cffiからnullableな値を受け取る方法を引き続き模索する
  2. ポートを変えたい場合はhostnameに :2222 のように含ませる仕様とする
  3. ポートについては現状は必須ではないので一旦無しで入れる

@marcy-terui
Copy link
Contributor Author

↑書いていて 2 のhostnameに含ませるのが無難かと思ったので、一旦そのように実装しました

@marcy-terui
Copy link
Contributor Author

一旦形になったので整理して出し直しました #62

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.

1 participant