You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@hertzbeat.apache.org by "MasamiYui (via GitHub)" <gi...@apache.org> on 2025/02/01 10:07:18 UTC

[PR] [feature] Add Sftp config for monitor [hertzbeat]

MasamiYui opened a new pull request, #3038:
URL: https://github.com/apache/hertzbeat/pull/3038

   ## What's changed?
   Add Sftp config for ftp monitor,issus is [#3012](https://github.com/apache/hertzbeat/issues/3012)
   <img width="1439" alt="image" src="https://github.com/user-attachments/assets/2c484643-cec0-471c-b6c5-88389ac6cb90" />
   
   
   ## Checklist
   
   - [ ]  I have read the [Contributing Guide](https://hertzbeat.apache.org/docs/community/code_style_and_quality_guide)
   - [ ]  I have written the necessary doc or comment.
   - [ ]  I have added the necessary unit tests and all cases have passed.
   
   ## Add or update API
   
   - [ ] I have added the necessary [e2e tests](https://github.com/apache/hertzbeat/tree/master/e2e) and all cases have passed.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@hertzbeat.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@hertzbeat.apache.org
For additional commands, e-mail: notifications-help@hertzbeat.apache.org


Re: [PR] [feature] Add Sftp config for monitor [hertzbeat]

Posted by "MasamiYui (via GitHub)" <gi...@apache.org>.
MasamiYui commented on PR #3038:
URL: https://github.com/apache/hertzbeat/pull/3038#issuecomment-2637354293

    use apache sshd-sftp && add LICENSE


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@hertzbeat.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@hertzbeat.apache.org
For additional commands, e-mail: notifications-help@hertzbeat.apache.org


Re: [PR] [feature] Add Sftp config for monitor [hertzbeat]

Posted by "tomsun28 (via GitHub)" <gi...@apache.org>.
tomsun28 merged PR #3038:
URL: https://github.com/apache/hertzbeat/pull/3038


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@hertzbeat.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@hertzbeat.apache.org
For additional commands, e-mail: notifications-help@hertzbeat.apache.org


Re: [PR] [feature] Add Sftp config for monitor [hertzbeat]

Posted by "MasamiYui (via GitHub)" <gi...@apache.org>.
MasamiYui commented on PR #3038:
URL: https://github.com/apache/hertzbeat/pull/3038#issuecomment-2635504616

   > hi, thanks for contribution. I found we use the `JSch` in this pr, but we have introduced a package with similar functions, `apache sshd`. Can we use `apache sshd` instead of `jsch`?
   
   hi, sure, we could use apache sshd implement it; but starting from apache sshd version 2.0, need import other package:sshd-sftp
   ref:https://github.com/apache/mina-sshd/blob/master/docs/sftp.md


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@hertzbeat.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@hertzbeat.apache.org
For additional commands, e-mail: notifications-help@hertzbeat.apache.org


Re: [PR] [feature] Add Sftp config for monitor [hertzbeat]

Posted by "tomsun28 (via GitHub)" <gi...@apache.org>.
tomsun28 commented on PR #3038:
URL: https://github.com/apache/hertzbeat/pull/3038#issuecomment-2629190754

   hi, thanks for contribution. I found we use the `JSch` in this pr, but we have introduced a package with similar functions, `apache sshd`. Can we use `apache sshd` instead of `jsch`?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@hertzbeat.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@hertzbeat.apache.org
For additional commands, e-mail: notifications-help@hertzbeat.apache.org


Re: [PR] [feature] Add Sftp config for monitor [hertzbeat]

Posted by "tomsun28 (via GitHub)" <gi...@apache.org>.
tomsun28 commented on PR #3038:
URL: https://github.com/apache/hertzbeat/pull/3038#issuecomment-2635626636

   > hi, sure, we could use apache sshd implement it; but starting from apache sshd version 2.0, need import other package:sshd-sftp ref:https://github.com/apache/mina-sshd/blob/master/docs/sftp.md
   
   hi, ok. If import new package, we need update the license document here. 
   https://github.com/apache/hertzbeat/tree/master/material/licenses 
   https://github.com/apache/hertzbeat/blob/master/material/licenses/LICENSE
   https://github.com/apache/hertzbeat/blob/master/material/licenses/backend/LICENSE
   https://github.com/apache/hertzbeat/blob/master/material/licenses/collector/LICENSE


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@hertzbeat.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@hertzbeat.apache.org
For additional commands, e-mail: notifications-help@hertzbeat.apache.org