Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:90252
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 74.125.82.43 as permitted sender)
MIME-Version: 1.0
Reply-To: bishop@php.net
Sender: bishop.bettini@gmail.com
In-Reply-To: <CAKws9z1g24rD=t92+9xKtO=3PuwQ3_N=L36gmVMqCC84CoqipQ@mail.gmail.com>
References: <CAKws9z1g24rD=t92+9xKtO=3PuwQ3_N=L36gmVMqCC84CoqipQ@mail.gmail.com>
Date: Thu, 7 Jan 2016 10:20:24 -0500
Message-ID: <CAEYWF=5+vXJPe17XF4Vp04paKoi3fn6VVrCAR7i5RFQ2+a9wFQ@mail.gmail.com>
To: Scott Arciszewski <scott@paragonie.com>
Cc: PHP Internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=001a11443f0a3c5b950528c00487
Subject: Re: [PHP-DEV] [RFC] Libsodium
From: bishop@php.net (Bishop Bettini)

--001a11443f0a3c5b950528c00487
Content-Type: text/plain; charset=UTF-8

On Thu, Jan 7, 2016 at 8:26 AM, Scott Arciszewski <scott@paragonie.com>
wrote:

> I've updated the RFC to make libsodium a core PHP extension in 7.1, to
> include references to the online documentation.
>
> https://wiki.php.net/rfc/libsodium
>
> All new functions and classes would exist in the Sodium namespace. e.g.
>
>     $ciphertext = \Sodium\crypto_box($message, $nonce, $keypair);
>
> This is part of an overall effort to improve PHP's cryptography; up
> next will be the pluggable crypto API that supports multiple backends
> (with a scope limited to openssl and libsodium at the time of release)
> but always provide conservative defaults. Then I'd like to look at
> deprecating ext/mcrypt back to PECL and add more hash functions to
> ext/hash.
>

+1. Only comment is on namespace: how about \Php\Ext\Sodium?  I don't know
if namespacing PHP engine components has been standardized, but I feel we
should limit how often the engine consumes names in the global namespace.
E.g, I could see a user wanting "Sodium" for an app name, but couldn't use
it simultaneously with the libsodium extension.

--001a11443f0a3c5b950528c00487--