Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Plaintext credential not async#375

Closed
mbleigh wants to merge 1 commit intonodegit:masternodegit/nodegit:masterfrom
mbleigh:patch-1mbleigh/nodegit:patch-1Copy head branch name to clipboard
Closed

Plaintext credential not async#375
mbleigh wants to merge 1 commit intonodegit:masternodegit/nodegit:masterfrom
mbleigh:patch-1mbleigh/nodegit:patch-1Copy head branch name to clipboard

Conversation

@mbleigh
Copy link

@mbleigh mbleigh commented Jan 30, 2015

I have no idea what I'm doing, but @johnhaley81 told me this would fix a segfault in Gitter 😉

I have no idea what I'm doing, but @johnhaley81 told me this would fix a segfault 😉
@johnhaley81
Copy link
Collaborator

@maxkorp updated the tests as well, can you rebase this branch on top of https://github.com/nodegit/nodegit/tree/cred-fixes please?

@johnhaley81
Copy link
Collaborator

oh, hah apparently he implemented the change as well.

@mbleigh
Copy link
Author

mbleigh commented Jan 30, 2015

Well, looks like this is being taken care of without me.

@mbleigh mbleigh closed this Jan 30, 2015
@maxkorp
Copy link
Collaborator

maxkorp commented Jan 30, 2015

Whooooops. Sorry, I didn't see that you had already done this.

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.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.