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

ssh: support multiple keys #6519

Merged
merged 1 commit into from
Sep 2, 2021

Conversation

isidentical
Copy link
Contributor

Resolves #6508

@isidentical isidentical requested a review from a team as a code owner September 1, 2021 20:38
@isidentical isidentical merged commit 64bf633 into iterative:master Sep 2, 2021
@efiop efiop added bugfix fixes bug fs: ssh Related to the SSH filesystem labels Sep 2, 2021
@Erotemic
Copy link
Contributor

Erotemic commented Sep 2, 2021

Should there ( / could there) be a DEBUG logging step here?

# In dvc/fs/ssh.py
import rich
import logging
logger = logging.getLogger(__name__)

...
       # <in: SSHFileSystem._prepare_credentials>

        login_info_display = login_info.copy()
        if 'password' in login_info_display:
            login_info_display['password'] = '<redacted>'
        logger.debug(
            "login_info = {}".format(rich.pretty.pretty_repr(login_info_display))
        )

I can make the PR if this change if adding ssh info to the debug output is desirable. (It would make me feel better as a sanity check if I could see it in the future).

@isidentical
Copy link
Contributor Author

isidentical commented Sep 2, 2021

I don't think this should be done, not at least on the SSH level. The function is called _prepare_credentials since it might contain any sort of confidential data (e.g the host IP, the username, or the password as you simply redacted) and generally we ask users to submit their whole debug log to us through discord, which is a public place so people might be even unaware that they are sharing such info, and some might even not to do so. So I'd say that since DVC is a python program, it is much simpler to just insert a breakpoint() and restart.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix fixes bug fs: ssh Related to the SSH filesystem
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DVC Pull Ignores identityfile in the SSH config.
4 participants