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

credential-osxkeychain is not a git command #772

Closed
cheshire137 opened this issue Dec 24, 2016 · 14 comments
Closed

credential-osxkeychain is not a git command #772

cheshire137 opened this issue Dec 24, 2016 · 14 comments
Assignees

Comments

@cheshire137
Copy link
Contributor

In 0.0.17, I added a newly initialized git repo to the app. I had previously pushed via the command line. The remote is using the https URL. When I made a commit via the app and hit the app's push button, I got this error in response:

screen shot 2016-12-23 at 8 12 22 pm

@shiftkey
Copy link
Member

shiftkey commented Jan 9, 2017

@cheshire137 can you check under Preferences to see that you're signed into a GitHub account? I'm trying to reproduce this, and I think the fact I have git-credential-osxkeychain on my PATH might be an issue - is your setup similar?

$ which git-credential-osxkeychain
/usr/local/bin/git-credential-osxkeychain

@cheshire137
Copy link
Contributor Author

I look signed in:

screen shot 2017-01-09 at 11 22 03 am

Ooh I get a different result for the command:

% which git-credential-osxkeychain
git-credential-osxkeychain not found

@joshaber
Copy link
Contributor

joshaber commented Jan 9, 2017

It shouldn't care about osxkeychain unless authentication with the app's credentials fails.

You might try logging out and then back in again?

@shiftkey
Copy link
Member

shiftkey commented Jan 12, 2017

I could trigger this after logging in, and after relaunching the app. Seems to be not retrieving the user which then means envForAuthentication isn't setup correctly - which would explain why it's trying to use git-credential-osxkeychain

@shiftkey shiftkey self-assigned this Jan 12, 2017
@shiftkey
Copy link
Member

shiftkey commented Jan 12, 2017

@cheshire137 can you tell me if this repository is listed under "GitHub" or "Other" when you look at it in the sidebar? For example, I was seeing this issue when I added the Desktop repository - and this was because it didn't consider it a GitHub repository until I'd signed in and had a refresh occur in the background:

I think we still need to setup GIT_ASKPASS to fail when not signed in, rather than have it fall through to whatever's been configured by Git.

@cheshire137
Copy link
Contributor Author

Ooh, good intuition! It is indeed in my 'Other' section which is odd as the repo is definitely hosted on GitHub.

screen shot 2017-01-12 at 1 26 58 pm

@shiftkey
Copy link
Member

shiftkey commented Jan 12, 2017

Interesting, so I can see the public repo but for some reason it's not being refreshed and populated with the GitHub API information.

One last thing to help reproduce it: could you list the remotes for the repository?

git remote -v

@cheshire137
Copy link
Contributor Author

heroku	https://git.heroku.com/overwatch-counter-picker.git (fetch)
heroku	https://git.heroku.com/overwatch-counter-picker.git (push)
origin	https://github.com/cheshire137/overwatch-counter-picker.git (fetch)
origin	https://github.com/cheshire137/overwatch-counter-picker.git (push)

Think the heroku remote is interfering?

@shiftkey
Copy link
Member

@cheshire137 yeah, I think that's related to the issue at hand. Thanks for all the info 💖

@shiftkey
Copy link
Member

@cheshire137 actually one more question - is master setup to track heroku or origin?

git branch -vv

@cheshire137
Copy link
Contributor Author

origin!

% git branch -vv
  database-stuff      a711082 [origin/database-stuff: gone] Add Heroku database steps to readme
  detect-cards-screen 911d49e [origin/detect-cards-screen: gone] Adjust template scale and Ana template
* master              b017f11 [origin/master] Use full scale sample-screenshot on index

@shiftkey
Copy link
Member

@cheshire137 thanks! i've encountered an issue where I'm signed in but I don't have a token, which I think might be the root cause of this...

@cheshire137
Copy link
Contributor Author

In my ~/.gitconfig, I do have this chunk:

[credential]
	helper = osxkeychain

@cheshire137
Copy link
Contributor Author

I got a variation on this one, this time after hitting 'Push origin' and it spun for a bit without pushing my changes:

screen shot 2017-01-27 at 11 09 39 am

@lock lock bot locked as resolved and limited conversation to collaborators Jul 23, 2018
@desktop desktop unlocked this conversation Sep 27, 2018
yasuking0304 pushed a commit to yasuking0304/desktop that referenced this issue Jan 22, 2023
Bumps [electron-builder](https://github.com/electron-userland/electron-builder/tree/HEAD/packages/electron-builder) from 23.3.3 to 23.6.0.
- [Release notes](https://github.com/electron-userland/electron-builder/releases)
- [Changelog](https://github.com/electron-userland/electron-builder/blob/master/packages/electron-builder/CHANGELOG.md)
- [Commits](https://github.com/electron-userland/electron-builder/commits/v23.6.0/packages/electron-builder)

---
updated-dependencies:
- dependency-name: electron-builder
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
yasuking0304 pushed a commit to yasuking0304/desktop that referenced this issue Jan 22, 2023
Bumps [electron-builder](https://github.com/electron-userland/electron-builder/tree/HEAD/packages/electron-builder) from 23.3.3 to 23.6.0.
- [Release notes](https://github.com/electron-userland/electron-builder/releases)
- [Changelog](https://github.com/electron-userland/electron-builder/blob/master/packages/electron-builder/CHANGELOG.md)
- [Commits](https://github.com/electron-userland/electron-builder/commits/v23.6.0/packages/electron-builder)

---
updated-dependencies:
- dependency-name: electron-builder
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
yasuking0304 pushed a commit to yasuking0304/desktop that referenced this issue Mar 9, 2023
Bumps [electron-builder](https://github.com/electron-userland/electron-builder/tree/HEAD/packages/electron-builder) from 23.3.3 to 23.6.0.
- [Release notes](https://github.com/electron-userland/electron-builder/releases)
- [Changelog](https://github.com/electron-userland/electron-builder/blob/master/packages/electron-builder/CHANGELOG.md)
- [Commits](https://github.com/electron-userland/electron-builder/commits/v23.6.0/packages/electron-builder)

---
updated-dependencies:
- dependency-name: electron-builder
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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

No branches or pull requests

3 participants