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: add ssh support for age #1496

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

albinvass
Copy link

This PR rebases and continues #1134

@devstein devstein requested review from hiddeco and a team April 30, 2024 15:07
if err != nil {
return nil, err
}
defer in.Close()

Check warning

Code scanning / CodeQL

Writable file handle closed without error handling

File handle may be writable as a result of data flow from a [call to OpenFile](1) and closing it may result in data loss upon failure, which is not handled explicitly.
if err != nil {
return nil, err
}
defer out.Close()

Check warning

Code scanning / CodeQL

Writable file handle closed without error handling

File handle may be writable as a result of data flow from a [call to OpenFile](1) and closing it may result in data loss upon failure, which is not handled explicitly.
if err != nil {
return nil, err
}
defer tty.Close()

Check warning

Code scanning / CodeQL

Writable file handle closed without error handling

File handle may be writable as a result of data flow from a [call to OpenFile](1) and closing it may result in data loss upon failure, which is not handled explicitly.
Signed-off-by: Marvin Strangfeld <marvin@strangfeld.io>
@overfl0
Copy link

overfl0 commented Oct 30, 2024

@albinvass would you be willing to take a look at the current conflicts and resolve them, as felixfontein said that he could try to take a look at it and maybe do something about it, here: #1134 (review)

Maybe there is a chance to finally have this merged :)

@haoqixu
Copy link

haoqixu commented Dec 1, 2024

Hi. I have submitted another PR #1692 to continue #1134. @overfl0

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.

5 participants