Skip to content

Username and password support for git_pull addon#217

Merged
pvizeli merged 7 commits into
home-assistant:masterfrom
raiford:git_pull_password
Jan 19, 2018
Merged

Username and password support for git_pull addon#217
pvizeli merged 7 commits into
home-assistant:masterfrom
raiford:git_pull_password

Conversation

@raiford
Copy link
Copy Markdown
Contributor

@raiford raiford commented Dec 29, 2017

This adds basic support for username and password authentication in the git_pull addon. This is necessary to use private repos that don't support ssh keys. An example of this is Google Cloud source repositories that use complex generated passwords for authentication over https.

https://cloud.google.com/source-repositories/

Doc update: home-assistant/home-assistant.io#4300

@homeassistant
Copy link
Copy Markdown

Hi @raiford,

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

raiford added a commit to raiford/home-assistant.github.io that referenced this pull request Dec 29, 2017
This is a documentation update that corresponds to home-assistant/addons#217
@pvizeli
Copy link
Copy Markdown
Member

pvizeli commented Jan 2, 2018

Look nice, Can we merge that with #218 @lazytesting ?

@raiford
Copy link
Copy Markdown
Contributor Author

raiford commented Jan 2, 2018

Sure, I can re-format my changes to match the new function layout that @lazytesting implemented. Should I wait until his changes are submitted to merge or is there a better way?

@pvizeli
Copy link
Copy Markdown
Member

pvizeli commented Jan 12, 2018

Please rebase this PR and applay last changes 👍

@raiford
Copy link
Copy Markdown
Contributor Author

raiford commented Jan 17, 2018

Ok, I rebased and updated it to follow the new format of the script. Please let me know what you think.

Copy link
Copy Markdown
Member

@pvizeli pvizeli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please update the CHANGELOG too

@pvizeli
Copy link
Copy Markdown
Member

pvizeli commented Jan 17, 2018

now we wait for travis :)

@pvizeli pvizeli merged commit a3adcca into home-assistant:master Jan 19, 2018
frenck pushed a commit to home-assistant/home-assistant.io that referenced this pull request Jan 24, 2018
This is a documentation update that corresponds to home-assistant/addons#217
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants