<feed xmlns='http://www.w3.org/2005/Atom'>
<title>delta/gitlab/gitlab-shell.git, branch sh-fix-wrong-user-deploy-key-check</title>
<subtitle>gitlab.com: gitlab-org/gitlab-shell.git
</subtitle>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-shell.git/'/>
<entry>
<title>Fix incorrect actor used to check permissions for SSH receive-pack</title>
<updated>2020-10-19T20:26:46+00:00</updated>
<author>
<name>Stan Hu</name>
<email>stanhu@gmail.com</email>
</author>
<published>2020-10-19T20:06:59+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-shell.git/commit/?id=354f5bf20c3d1b48481bd4e6f4d4219f830b986b'/>
<id>354f5bf20c3d1b48481bd4e6f4d4219f830b986b</id>
<content type='text'>
During a SSH receive-pack request (e.g. `git push`), gitlab-shell was
incorrectly using the user returned by the `/internal/allowed` API
endpoint to make an SSHReceivePack RPC call. This caused a number of
problems with deploy keys with write access:

1. Keys that were generated by a blocked user would be denied the
ability to write.

2. Keys that were generated by user that did not have write access to
the project would also be denied.

GitLab 12.4 removed the Ruby implementation of gitlab-shell in favor of
the Golang implementation, and these implementations worked slightly
differently. In
https://gitlab.com/gitlab-org/gitlab-shell/blob/v10.1.0/lib/gitlab_shell.rb,
the Ruby implementation would always use `@who` (e.g. `key-123`), but in
gitlab-shell v10.2.0 the Go implementation would always use the user
from the API response.

Reads did not have this issue because the user/deploy key is never
passed to Gitaly for additional permission checks. Writes need this
information for the pre-receive to check access to protected branches,
push rules, etc.

Relates to https://gitlab.com/gitlab-org/gitlab-shell/-/issues/479
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
During a SSH receive-pack request (e.g. `git push`), gitlab-shell was
incorrectly using the user returned by the `/internal/allowed` API
endpoint to make an SSHReceivePack RPC call. This caused a number of
problems with deploy keys with write access:

1. Keys that were generated by a blocked user would be denied the
ability to write.

2. Keys that were generated by user that did not have write access to
the project would also be denied.

GitLab 12.4 removed the Ruby implementation of gitlab-shell in favor of
the Golang implementation, and these implementations worked slightly
differently. In
https://gitlab.com/gitlab-org/gitlab-shell/blob/v10.1.0/lib/gitlab_shell.rb,
the Ruby implementation would always use `@who` (e.g. `key-123`), but in
gitlab-shell v10.2.0 the Go implementation would always use the user
from the API response.

Reads did not have this issue because the user/deploy key is never
passed to Gitaly for additional permission checks. Writes need this
information for the pre-receive to check access to protected branches,
push rules, etc.

Relates to https://gitlab.com/gitlab-org/gitlab-shell/-/issues/479
</pre>
</div>
</content>
</entry>
<entry>
<title>Merge branch 'ashmckenzie/set-ssl-cert-dir-env-var' into 'master'</title>
<updated>2020-10-19T09:16:47+00:00</updated>
<author>
<name>Patrick Bajao</name>
<email>ebajao@gitlab.com</email>
</author>
<published>2020-10-19T09:16:47+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-shell.git/commit/?id=c09bdad64dc5ae1a57c61a435edb62b5f9a7c3a8'/>
<id>c09bdad64dc5ae1a57c61a435edb62b5f9a7c3a8</id>
<content type='text'>
Set SSL_CERT_DIR env var when building command

See merge request gitlab-org/gitlab-shell!423</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Set SSL_CERT_DIR env var when building command

See merge request gitlab-org/gitlab-shell!423</pre>
</div>
</content>
</entry>
<entry>
<title>Set SSL_CERT_DIR env var when building command</title>
<updated>2020-10-19T07:53:12+00:00</updated>
<author>
<name>Ash McKenzie</name>
<email>amckenzie@gitlab.com</email>
</author>
<published>2020-10-19T03:29:12+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-shell.git/commit/?id=0478ba97950bd6606f823c8a26eeeecf617df653'/>
<id>0478ba97950bd6606f823c8a26eeeecf617df653</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Remove prefixing with SSL_CERT_DIR</title>
<updated>2020-10-19T07:53:11+00:00</updated>
<author>
<name>Ash McKenzie</name>
<email>amckenzie@gitlab.com</email>
</author>
<published>2020-10-19T03:41:45+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-shell.git/commit/?id=f5f9ffc086fe52e2651fb498a76673bda3392bfd'/>
<id>f5f9ffc086fe52e2651fb498a76673bda3392bfd</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Merge branch 'sh-release-13-10-0' into 'master'</title>
<updated>2020-10-16T08:28:52+00:00</updated>
<author>
<name>Igor Drozdov</name>
<email>idrozdov@gitlab.com</email>
</author>
<published>2020-10-16T08:28:52+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-shell.git/commit/?id=12353c0c1d6ddf57b738b04d86cdd34ce64f2f18'/>
<id>12353c0c1d6ddf57b738b04d86cdd34ce64f2f18</id>
<content type='text'>
Release v13.10.0

See merge request gitlab-org/gitlab-shell!422</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Release v13.10.0

See merge request gitlab-org/gitlab-shell!422</pre>
</div>
</content>
</entry>
<entry>
<title>Release v13.10.0</title>
<updated>2020-10-16T06:58:29+00:00</updated>
<author>
<name>Stan Hu</name>
<email>stanhu@gmail.com</email>
</author>
<published>2020-10-16T06:58:29+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-shell.git/commit/?id=faa5e19abd9f114529f08d80446292a9f4794e8c'/>
<id>faa5e19abd9f114529f08d80446292a9f4794e8c</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Merge branch 'sh-add-version-arg' into 'master'</title>
<updated>2020-10-16T06:27:48+00:00</updated>
<author>
<name>Ash McKenzie</name>
<email>amckenzie@gitlab.com</email>
</author>
<published>2020-10-16T06:27:48+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-shell.git/commit/?id=6da1094b190a76231ec6d4ee4f0a0846de916446'/>
<id>6da1094b190a76231ec6d4ee4f0a0846de916446</id>
<content type='text'>
Add support for -version argument

See merge request gitlab-org/gitlab-shell!421</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Add support for -version argument

See merge request gitlab-org/gitlab-shell!421</pre>
</div>
</content>
</entry>
<entry>
<title>Apply 1 suggestion(s) to 1 file(s)</title>
<updated>2020-10-16T06:06:42+00:00</updated>
<author>
<name>Stan Hu</name>
<email>stanhu@gmail.com</email>
</author>
<published>2020-10-16T06:06:42+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-shell.git/commit/?id=89a23f1923aa122734e92ddb588f20694959afbf'/>
<id>89a23f1923aa122734e92ddb588f20694959afbf</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Add support for -version argument</title>
<updated>2020-10-16T05:41:48+00:00</updated>
<author>
<name>Stan Hu</name>
<email>stanhu@gmail.com</email>
</author>
<published>2020-10-16T03:02:19+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-shell.git/commit/?id=9010cdccde441d37aff8a1d76f2e701134e42801'/>
<id>9010cdccde441d37aff8a1d76f2e701134e42801</id>
<content type='text'>
This will help determine the version of the binary particularly on Cloud
Native GitLab, where VERSION may not be shipped with the binaries.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This will help determine the version of the binary particularly on Cloud
Native GitLab, where VERSION may not be shipped with the binaries.
</pre>
</div>
</content>
</entry>
<entry>
<title>Merge branch 'release-shell-13-9-0' into 'master'</title>
<updated>2020-10-15T16:14:43+00:00</updated>
<author>
<name>Nick Thomas</name>
<email>nick@gitlab.com</email>
</author>
<published>2020-10-15T16:14:43+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-shell.git/commit/?id=890bda90c9a09aa8d5a2441974138b92c8e38c18'/>
<id>890bda90c9a09aa8d5a2441974138b92c8e38c18</id>
<content type='text'>
Release v13.9.0

See merge request gitlab-org/gitlab-shell!420</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Release v13.9.0

See merge request gitlab-org/gitlab-shell!420</pre>
</div>
</content>
</entry>
</feed>
