Before, there was a cycle along the lines of: (guix store) -> (guix nar) -> (guix ui) -> (guix store) This caused problems, as discussed at: http://lists.gnu.org/archive/html/guix-devel/2014-10/msg00109.html This patch removes cycles in the (guix ...) modules. * guix/nar.scm (&nar-error, &nar-read-error, dump, write-contents, read-contents, %archive-version-1, write-file, restore-file): Move to... * guix/serialization.scm: ... here. * guix/store.scm: Remove dependency on (guix nar). * guix/scripts/hash.scm, guix/scripts/offload.scm, guix/scripts/substitute-binary.scm, tests/nar.scm, tests/store.scm, tests/substitute-binary.scm: Adjust accordingly.
		
			
				
	
	
		
			284 lines
		
	
	
	
		
			11 KiB
		
	
	
	
		
			Scheme
		
	
	
	
	
	
			
		
		
	
	
			284 lines
		
	
	
	
		
			11 KiB
		
	
	
	
		
			Scheme
		
	
	
	
	
	
| ;;; GNU Guix --- Functional package management for GNU
 | ||
| ;;; Copyright © 2014 Nikita Karetnikov <nikita@karetnikov.org>
 | ||
| ;;; Copyright © 2014 Ludovic Courtès <ludo@gnu.org>
 | ||
| ;;;
 | ||
| ;;; This file is part of GNU Guix.
 | ||
| ;;;
 | ||
| ;;; GNU Guix is free software; you can redistribute it and/or modify it
 | ||
| ;;; under the terms of the GNU General Public License as published by
 | ||
| ;;; the Free Software Foundation; either version 3 of the License, or (at
 | ||
| ;;; your option) any later version.
 | ||
| ;;;
 | ||
| ;;; GNU Guix is distributed in the hope that it will be useful, but
 | ||
| ;;; WITHOUT ANY WARRANTY; without even the implied warranty of
 | ||
| ;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 | ||
| ;;; GNU General Public License for more details.
 | ||
| ;;;
 | ||
| ;;; You should have received a copy of the GNU General Public License
 | ||
| ;;; along with GNU Guix.  If not, see <http://www.gnu.org/licenses/>.
 | ||
| 
 | ||
| (define-module (test-substitute-binary)
 | ||
|   #:use-module (guix scripts substitute-binary)
 | ||
|   #:use-module (guix base64)
 | ||
|   #:use-module (guix hash)
 | ||
|   #:use-module (guix serialization)
 | ||
|   #:use-module (guix pk-crypto)
 | ||
|   #:use-module (guix pki)
 | ||
|   #:use-module (guix config)
 | ||
|   #:use-module (guix base32)
 | ||
|   #:use-module ((guix store) #:select (%store-prefix))
 | ||
|   #:use-module ((guix ui) #:select (guix-warning-port))
 | ||
|   #:use-module ((guix build utils) #:select (delete-file-recursively))
 | ||
|   #:use-module (rnrs bytevectors)
 | ||
|   #:use-module (rnrs io ports)
 | ||
|   #:use-module (web uri)
 | ||
|   #:use-module (ice-9 regex)
 | ||
|   #:use-module (srfi srfi-26)
 | ||
|   #:use-module (srfi srfi-34)
 | ||
|   #:use-module (srfi srfi-35)
 | ||
|   #:use-module ((srfi srfi-64) #:hide (test-error)))
 | ||
| 
 | ||
| (define-syntax-rule (test-quit name error-rx exp)
 | ||
|   "Emit a test that passes when EXP throws to 'quit' with value 1, and when
 | ||
| it writes to GUIX-WARNING-PORT a messages that matches ERROR-RX."
 | ||
|   (test-equal name
 | ||
|     '(1 #t)
 | ||
|     (let ((error-output (open-output-string)))
 | ||
|       (parameterize ((guix-warning-port error-output))
 | ||
|         (catch 'quit
 | ||
|           (lambda ()
 | ||
|             exp
 | ||
|             #f)
 | ||
|           (lambda (key value)
 | ||
|             (list value
 | ||
|                   (let ((message (get-output-string error-output)))
 | ||
|                     (->bool (string-match error-rx message))))))))))
 | ||
| 
 | ||
| (define %public-key
 | ||
|   ;; This key is known to be in the ACL by default.
 | ||
|   (call-with-input-file (string-append %config-directory "/signing-key.pub")
 | ||
|     (compose string->canonical-sexp get-string-all)))
 | ||
| 
 | ||
| (define %private-key
 | ||
|   (call-with-input-file (string-append %config-directory "/signing-key.sec")
 | ||
|     (compose string->canonical-sexp get-string-all)))
 | ||
| 
 | ||
| (define* (signature-body bv #:key (public-key %public-key))
 | ||
|   "Return the signature of BV as the base64-encoded body of a narinfo's
 | ||
| 'Signature' field."
 | ||
|   (base64-encode
 | ||
|    (string->utf8
 | ||
|     (canonical-sexp->string
 | ||
|      (signature-sexp (bytevector->hash-data (sha256 bv)
 | ||
|                                             #:key-type 'rsa)
 | ||
|                      %private-key
 | ||
|                      public-key)))))
 | ||
| 
 | ||
| (define %wrong-public-key
 | ||
|   (string->canonical-sexp "(public-key
 | ||
|  (rsa
 | ||
|   (n #00E05873AC2B168760343145918E954EE9AB73C026355693B192E01EE835261AA689E9EF46642E895BCD65C648524059FC450E4BA77A68F4C52D0E39EF0CC9359709AB6AAB153B63782201871325B0FDA19CB401CD99FD0C31A91CA9000AA90A77E82B89E036FB63BC1D3961207469B3B12468977148D376F8012BB12A4B11A8F1#)
 | ||
|   (e #010001#)
 | ||
|   )
 | ||
|  )"))
 | ||
| 
 | ||
| (define* (signature-field bv-or-str
 | ||
|                           #:key (version "1") (public-key %public-key))
 | ||
|   "Return the 'Signature' field value of bytevector/string BV-OR-STR, using
 | ||
| PUBLIC-KEY as the signature's principal, and using VERSION as the signature
 | ||
| version identifier.."
 | ||
|   (string-append version ";example.gnu.org;"
 | ||
|                  (signature-body (if (string? bv-or-str)
 | ||
|                                      (string->utf8 bv-or-str)
 | ||
|                                      bv-or-str)
 | ||
|                                  #:public-key public-key)))
 | ||
| 
 | ||
| 
 | ||
| 
 | ||
| (test-begin "substitute-binary")
 | ||
| 
 | ||
| (test-quit "not a number"
 | ||
|     "signature version"
 | ||
|   (narinfo-signature->canonical-sexp
 | ||
|    (signature-field "foo" #:version "not a number")))
 | ||
| 
 | ||
| (test-quit "wrong version number"
 | ||
|     "unsupported.*version"
 | ||
|   (narinfo-signature->canonical-sexp
 | ||
|    (signature-field "foo" #:version "2")))
 | ||
| 
 | ||
| (test-assert "valid narinfo-signature->canonical-sexp"
 | ||
|   (canonical-sexp? (narinfo-signature->canonical-sexp (signature-field "foo"))))
 | ||
| 
 | ||
| 
 | ||
| 
 | ||
| (define %narinfo
 | ||
|   ;; Skeleton of the narinfo used below.
 | ||
|   (string-append "StorePath: " (%store-prefix)
 | ||
|                  "/aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa-foo
 | ||
| URL: example.nar
 | ||
| Compression: none
 | ||
| NarHash: sha256:" (bytevector->nix-base32-string
 | ||
|                    (sha256 (string->utf8 "Substitutable data."))) "
 | ||
| NarSize: 42
 | ||
| References: bar baz
 | ||
| Deriver: " (%store-prefix) "/foo.drv
 | ||
| System: mips64el-linux\n"))
 | ||
| 
 | ||
| (define (call-with-narinfo narinfo thunk)
 | ||
|   "Call THUNK in a context where $GUIX_BINARY_SUBSTITUTE_URL is populated with
 | ||
| a file for NARINFO."
 | ||
|   (let ((narinfo-directory (and=> (string->uri (getenv
 | ||
|                                                 "GUIX_BINARY_SUBSTITUTE_URL"))
 | ||
|                                   uri-path))
 | ||
|         (cache-directory   (string-append (getenv "XDG_CACHE_HOME")
 | ||
|                                           "/guix/substitute-binary/")))
 | ||
|     (dynamic-wind
 | ||
|       (lambda ()
 | ||
|         (when (file-exists? cache-directory)
 | ||
|           (delete-file-recursively cache-directory))
 | ||
|         (call-with-output-file (string-append narinfo-directory
 | ||
|                                               "/nix-cache-info")
 | ||
|           (lambda (port)
 | ||
|             (format port "StoreDir: ~a\nWantMassQuery: 0\n"
 | ||
|                     (%store-prefix))))
 | ||
|         (call-with-output-file (string-append narinfo-directory "/"
 | ||
|                                               "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa"
 | ||
|                                               ".narinfo")
 | ||
|           (cut display narinfo <>))
 | ||
| 
 | ||
|         ;; Prepare the nar.
 | ||
|         (call-with-output-file
 | ||
|             (string-append narinfo-directory "/example.out")
 | ||
|           (cut display "Substitutable data." <>))
 | ||
|         (call-with-output-file
 | ||
|             (string-append narinfo-directory "/example.nar")
 | ||
|           (cute write-file
 | ||
|                 (string-append narinfo-directory "/example.out") <>))
 | ||
| 
 | ||
|         (set! (@@ (guix scripts substitute-binary)
 | ||
|                   %allow-unauthenticated-substitutes?)
 | ||
|               #f))
 | ||
|       thunk
 | ||
|       (lambda ()
 | ||
|         (delete-file-recursively cache-directory)))))
 | ||
| 
 | ||
| (define-syntax-rule (with-narinfo narinfo body ...)
 | ||
|   (call-with-narinfo narinfo (lambda () body ...)))
 | ||
| 
 | ||
| ;; Transmit these options to 'guix substitute-binary'.
 | ||
| (set! (@@ (guix scripts substitute-binary) %cache-url)
 | ||
|       (getenv "GUIX_BINARY_SUBSTITUTE_URL"))
 | ||
| 
 | ||
| (test-equal "query narinfo without signature"
 | ||
|   ""                                              ; not substitutable
 | ||
| 
 | ||
|   (with-narinfo %narinfo
 | ||
|     (string-trim-both
 | ||
|      (with-output-to-string
 | ||
|        (lambda ()
 | ||
|          (with-input-from-string (string-append "have " (%store-prefix)
 | ||
|                                                 "/aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa-foo")
 | ||
|            (lambda ()
 | ||
|              (guix-substitute-binary "--query"))))))))
 | ||
| 
 | ||
| (test-equal "query narinfo with invalid hash"
 | ||
|   ;; The hash in the signature differs from the hash of %NARINFO.
 | ||
|   ""
 | ||
| 
 | ||
|   (with-narinfo (string-append %narinfo "Signature: "
 | ||
|                                (signature-field "different body")
 | ||
|                                "\n")
 | ||
|     (string-trim-both
 | ||
|      (with-output-to-string
 | ||
|        (lambda ()
 | ||
|          (with-input-from-string (string-append "have " (%store-prefix)
 | ||
|                                                 "/aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa-foo")
 | ||
|            (lambda ()
 | ||
|              (guix-substitute-binary "--query"))))))))
 | ||
| 
 | ||
| (test-equal "query narinfo signed with authorized key"
 | ||
|   (string-append (%store-prefix) "/aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa-foo")
 | ||
| 
 | ||
|   (with-narinfo (string-append %narinfo "Signature: "
 | ||
|                                (signature-field %narinfo)
 | ||
|                                "\n")
 | ||
|     (string-trim-both
 | ||
|      (with-output-to-string
 | ||
|        (lambda ()
 | ||
|          (with-input-from-string (string-append "have " (%store-prefix)
 | ||
|                                                 "/aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa-foo")
 | ||
|            (lambda ()
 | ||
|              (guix-substitute-binary "--query"))))))))
 | ||
| 
 | ||
| (test-equal "query narinfo signed with unauthorized key"
 | ||
|   ""                                              ; not substitutable
 | ||
| 
 | ||
|   (with-narinfo (string-append %narinfo "Signature: "
 | ||
|                                (signature-field
 | ||
|                                 %narinfo
 | ||
|                                 #:public-key %wrong-public-key)
 | ||
|                                "\n")
 | ||
|     (string-trim-both
 | ||
|      (with-output-to-string
 | ||
|        (lambda ()
 | ||
|          (with-input-from-string (string-append "have " (%store-prefix)
 | ||
|                                                 "/aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa-foo")
 | ||
|            (lambda ()
 | ||
|              (guix-substitute-binary "--query"))))))))
 | ||
| 
 | ||
| (test-quit "substitute, no signature"
 | ||
|     "lacks a signature"
 | ||
|   (with-narinfo %narinfo
 | ||
|     (guix-substitute-binary "--substitute"
 | ||
|                             (string-append (%store-prefix)
 | ||
|                                            "/aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa-foo")
 | ||
|                             "foo")))
 | ||
| 
 | ||
| (test-quit "substitute, invalid hash"
 | ||
|     "hash"
 | ||
|   ;; The hash in the signature differs from the hash of %NARINFO.
 | ||
|   (with-narinfo (string-append %narinfo "Signature: "
 | ||
|                                (signature-field "different body")
 | ||
|                                "\n")
 | ||
|     (guix-substitute-binary "--substitute"
 | ||
|                             (string-append (%store-prefix)
 | ||
|                                            "/aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa-foo")
 | ||
|                             "foo")))
 | ||
| 
 | ||
| (test-quit "substitute, unauthorized key"
 | ||
|     "unauthorized"
 | ||
|   (with-narinfo (string-append %narinfo "Signature: "
 | ||
|                                (signature-field
 | ||
|                                 %narinfo
 | ||
|                                 #:public-key %wrong-public-key)
 | ||
|                                "\n")
 | ||
|     (guix-substitute-binary "--substitute"
 | ||
|                             (string-append (%store-prefix)
 | ||
|                                            "/aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa-foo")
 | ||
|                             "foo")))
 | ||
| 
 | ||
| (test-equal "substitute, authorized key"
 | ||
|   "Substitutable data."
 | ||
|   (with-narinfo (string-append %narinfo "Signature: "
 | ||
|                                (signature-field %narinfo))
 | ||
|     (dynamic-wind
 | ||
|       (const #t)
 | ||
|       (lambda ()
 | ||
|         (guix-substitute-binary "--substitute"
 | ||
|                                 (string-append (%store-prefix)
 | ||
|                                                "/aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa-foo")
 | ||
|                                 "substitute-retrieved")
 | ||
|         (call-with-input-file "substitute-retrieved" get-string-all))
 | ||
|       (lambda ()
 | ||
|         (false-if-exception (delete-file "substitute-retrieved"))))))
 | ||
| 
 | ||
| (test-end "substitute-binary")
 | ||
| 
 | ||
| 
 | ||
| (exit (= (test-runner-fail-count (test-runner-current)) 0))
 | ||
| 
 | ||
| ;;; Local Variables:
 | ||
| ;;; eval: (put 'with-narinfo 'scheme-indent-function 1)
 | ||
| ;;; eval: (put 'test-quit 'scheme-indent-function 2)
 | ||
| ;;; End:
 |