Skip to content

Token caching/GitHub authentication dependent on Mac keychain #22

@cvaylen

Description

@cvaylen

The current setup for creating and storing a token, or depending on one existing in the keychain, doesn't work cross-platform. If we skip the keychain checks, github_token doesn't work if a token already exists.

The convenience for Mac users is nice, but is there a way we can find an abstraction for cross-platform keychains, or some other alternative? Not being able to create a token when one exists also makes this difficult. Also, all of the current tests passed despite this critical issue.

Maybe we can scale back to manual token authentication, write more rigorous tests, and make things less platform-dependent?

Personal suggestion for simplifying password input:

def get_secure
  require 'highline'
  cli = HighLine.new
  input = cli.ask("Enter #{username}'s GitHub password (never stored):" { |q| q.echo = false } # alt. q.echo = "*"
  input
end

Cutting out the dependence on stty

Eager to hear your thoughts!

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions