Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(550)

Issue 1363003: Add mock https server cert/key to testing builds (Closed)

Created:
10 years, 9 months ago by Chris Masone
Modified:
9 years, 7 months ago
Reviewers:
petkov
CC:
chromium-os-reviews_chromium.org
Visibility:
Public.

Description

Add mock https server cert/key to testing builds

Patch Set 1 #

Total comments: 1

Patch Set 2 : making sure I fixed a botched merge #

Unified diffs Side-by-side diffs Delta from patch set Stats (+19 lines, -0 lines) Patch
A src/scripts/mod_for_test_scripts/710enableAuthTesting View 1 chunk +19 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Chris Masone
10 years, 9 months ago (2010-03-26 00:17:52 UTC) #1
petkov
LGTM http://codereview.chromium.org/1363003/diff/1/2 File src/scripts/mod_for_test_scripts/710enableAuthTesting (right): http://codereview.chromium.org/1363003/diff/1/2#newcode13 src/scripts/mod_for_test_scripts/710enableAuthTesting:13: cat "${CERT_DIR}/${CERT_NAME}.key" > "${ROOT_FS_DIR}/etc/${CERT_NAME}.key" cat vs cp ?
10 years, 9 months ago (2010-03-26 04:09:19 UTC) #2
Chris Masone
10 years, 9 months ago (2010-03-26 15:54:33 UTC) #3
On 2010/03/26 04:09:19, petkov wrote:
> LGTM
> 
> http://codereview.chromium.org/1363003/diff/1/2
> File src/scripts/mod_for_test_scripts/710enableAuthTesting (right):
> 
> http://codereview.chromium.org/1363003/diff/1/2#newcode13
> src/scripts/mod_for_test_scripts/710enableAuthTesting:13: cat
> "${CERT_DIR}/${CERT_NAME}.key" > "${ROOT_FS_DIR}/etc/${CERT_NAME}.key"
> cat vs cp ?

Yeah, I didn't want changes in the ownership/perms to propagate into the image.

Powered by Google App Engine
This is Rietveld 408576698