<div dir="ltr">Thanks. Let me know if it makes sense for me to push those changes upstream. I was not aware of the git credentials helper... I know git-review is an infra project and thus not touched too much since everyone uses it... <div><br></div><div><div>Best Regards,</div></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 25, 2015 at 5:06 PM, Clark Boylan <span dir="ltr"><<a href="mailto:cboylan@sapwetik.org" target="_blank">cboylan@sapwetik.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I see what you mean about defaulting the port now. I think having the<br>
port default to 443 when scheme is HTTPS would be a good improvement to<br>
git review. For password handling its probably best for users to use the<br>
git credential helper [0] instead of having git review try to be smart<br>
with that info. The nice thing about this method is that talking to the<br>
server with other git commands should just work as well.<br>
<br>
[0] <a href="http://git-scm.com/docs/gitcredentials" target="_blank">http://git-scm.com/docs/gitcredentials</a><br>
<div class="HOEnZb"><div class="h5"><br>
On Wed, Feb 25, 2015, at 04:56 PM, Diego Garcia del Rio wrote:<br>
> This would be the page <a href="https://wiki.openstack.org/wiki/Gerrit_Workflow" target="_blank">https://wiki.openstack.org/wiki/Gerrit_Workflow</a> ,<br>
> in<br>
> particular the section called  "Accessing Gerrit over HTTPS"<br>
><br>
> I think i still need to catch up on the automated infrastructure testing<br>
> as<br>
> well.<br>
><br>
> Thanks!<br>
><br>
><br>
><br>
><br>
><br>
><br>
> On Wed, Feb 25, 2015 at 4:39 PM, Clark Boylan <<a href="mailto:cboylan@sapwetik.org">cboylan@sapwetik.org</a>><br>
> wrote:<br>
><br>
> > On Wed, Feb 25, 2015, at 04:23 PM, Diego Garcia del Rio wrote:<br>
> > > Hi,<br>
> > ><br>
> > > I'm pretty new to contributing code to openstack, but I had ran into some<br>
> > > issues trying to push changes to gerrit using the HTTPS option. (In<br>
> > > particular, added the ability to save your HTTP password into the git<br>
> > > config and  to automatically use port 443 if the scheme is HTTPS instead<br>
> > > of<br>
> > > the default SSH. I had a bit of head-banging to get the HTTPS working<br>
> > > (especially since the SSH port was still in place and not overridden). I<br>
> > > documented my steps on the wiki but thought that adding a bit more<br>
> > > intelligence to git-review would be useful for other new users or people<br>
> > > dealing with constrained connectivity options.<br>
> > ><br>
> > > should I submit my changes up to the openstack-infra/git-review?<br>
> > ><br>
> > Can you link to the wiki documentation? I think I need a bit more<br>
> > context before I can offer advice on pushing changes to git-review.<br>
> ><br>
> > Clark<br>
> ><br>
> > _______________________________________________<br>
> > OpenStack-Infra mailing list<br>
> > <a href="mailto:OpenStack-Infra@lists.openstack.org">OpenStack-Infra@lists.openstack.org</a><br>
> > <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra</a><br>
> ><br>
</div></div></blockquote></div><br></div>