kurosawa.takuya

PHP phar拡張モジュールの素敵な仕様

久しぶりにやってくれたPHP。
気づかず同じところに引っかかる人が出ると思ったので書いておきます。

今回PHPで.tar.gz圧縮/解凍を実装することになりまして、
phar拡張モジュールが対応しているということなので、素直にこれで実装したところ動作確認中にエラーが発生しました。

PHP Fatal error:  Uncaught BadMethodCallException: Unable to add newly converted phar "/path/to/hogehoge.tar.gz" to the list of phars, a phar with that name already exists in C:\Users\Kurosawa\Desktop\phar.php:18
Stack trace:
#0 /path/to/phar.php(18): PharData->compress()
#1 {main}
 thrown in /path/to/phar.php on line 18

PHP Fatal error: Uncaught BadMethodCallException: Unable to add newly converted phar "/path/to/hogehoge.tar.gz" to the list of phars, a phar with that name already exists in C:\Users\Kurosawa\Desktop\phar.php:18

Stack trace:

#0 /path/to/phar.php(18): PharData->compress()

#1 {main}

thrown in /path/to/phar.php on line 18

再現のための最小コードがこちらになります。

<?php

// 適当に.tar.gzに入れるファイル
file_put_contents('hogehoge.txt', 'このままでは俺の寿命がストレスでﾏｯﾊなんだが・・');

// .tarと.tar.gzファイルを作成
$phar = new \PharData('hogehoge.tar');
$phar->addFile('hogehoge.txt', 'hogehoge.txt');
$phar->compress(\Phar::GZ);

// .tarと.tar.gzファイルを削除
unlink('hogehoge.tar');
unlink('hogehoge.tar.gz');

// 同じ名前の.tarと.tar.gzファイルを作成したい
$phar = new \PharData('hogehoge.tar');
$phar->addFile('hogehoge.txt', 'hogehoge.txt');
$phar->compress(\Phar::GZ);

<?php

// 適当に.tar.gzに入れるファイル

file_put_contents('hogehoge.txt', 'このままでは俺の寿命がストレスでﾏｯﾊなんだが・・');

// .tarと.tar.gzファイルを作成

$phar = new \PharData('hogehoge.tar');

$phar->addFile('hogehoge.txt', 'hogehoge.txt');

$phar->compress(\Phar::GZ);

// .tarと.tar.gzファイルを削除

unlink('hogehoge.tar');

unlink('hogehoge.tar.gz');

// 同じ名前の.tarと.tar.gzファイルを作成したい

$phar = new \PharData('hogehoge.tar');

$phar->addFile('hogehoge.txt', 'hogehoge.txt');

$phar->compress(\Phar::GZ);

PharDataは内部でファイルパスをキーとして静的な領域にデータをキャッシュする実装になっているようで、上記のように同じファイルパスだと、複数PharDataを作成しても同じデータを指すことになり、2回目のcompressを実行しようとしたタイミングで「それはもう有るよエラー」ということです。

逆に以下のようなコードで1つのファイルを複数のPharDataで操作出来ます(意味は無い)

$phar1 = new \PharData('hogehoge.tar');
$phar2 = new \PharData('hogehoge.tar');
$phar1->addFile('hogehoge.txt', 'hogehoge1.txt');
$phar2->addFile('hogehoge.txt', 'hogehoge2.txt');
$phar1->compress(\Phar::GZ, '.tar.gz');

$phar1 = new \PharData('hogehoge.tar');

$phar2 = new \PharData('hogehoge.tar');

$phar1->addFile('hogehoge.txt', 'hogehoge1.txt');

$phar2->addFile('hogehoge.txt', 'hogehoge2.txt');

$phar1->compress(\Phar::GZ, '.tar.gz');

この現象については報告されていたのですが、ソースとかディスカッションを見るに
意図して実装されており、良い悪いは別として仕様な気がします。

https://bugs.php.net/bug.php?id=75101

1処理中に同じファイルパスで複数回ファイルを作り直すみたいなのはレアケースだと思うので遭遇する確率は低いと思いますが、記憶の片隅に留めておくとよいと思います。

WordPressのデフォルトアバターが表示されない(404エラーになる)問題

WordPressでデフォルトのアバターを表示させると、以下のようなGravatarのURLを生成します。

https://secure.gravatar.com/avatar/?s=60&d=mm&r=g

このURLが2023年8月9日現在404エラーとなっています。

次のようにパラメータの順番を変更すると正常に表示されます。

https://secure.gravatar.com/avatar/?s=60&r=g&d=mm

さすがにこれが仕様ということは無いと思うので、何かしら不具合が発生していると思われます。

Gravatarにお知らせしたいのですが、不具合を報告したり、お問い合わせするようなフォームが見つけられなかったので諦めました。

Gravatarの中の人ー！見てたら直してくれー！

Workaroundとしては、WordPressが生成するURLのパラメータを並べ替えることが考えられます。

以下のようにURLの生成箇所に用意されているフィルターでパラメータを並べ替えればよいでしょう。

add_filter(
	'get_avatar_url',
	function( $url, $id_or_email, $args ) {
		return preg_replace( '|\A(https?://.+\.gravatar\.com/avatar/.*\?)(.*)(&d=[^&]+)(.*)\z|', '$1$2$4$3', $url );
	},
	10,
	3
);

				
					
				1
2
3
4
5
6
7
8

						add_filter(
    'get_avatar_url',
    function( $url, $id_or_email, $args ) {
        return preg_replace( '|\A(https?://.+\.gravatar\.com/avatar/.*\?)(.*)(&d=[^&]+)(.*)\z|', '$1$2$4$3', $url );
    },
    10,
    3
);

					

			

Gravatarが修正されるまでの一時的な物なのでプラグイン化するほどではないかなと思います。

お使いのテーマのfunctions.phpに入れればよいのではないでしょうか。

httpd 2.4.33のmod_sslでLibreSSL 2.7以上を使うときのパッチ

httpdのバージョンが上がったらそのうち対応されて直ると思いますが
それまでは使えると思うのでメモっておきます。

具体的には次のようなエラーがでた場合の対処です。

ssl_engine_init.c:54:12: error: static declaration of 'DH_set0_pqg' follows non-static declaration
 static int DH_set0_pqg(DH *dh, BIGNUM *p, BIGNUM *q, BIGNUM *g)
            ^
In file included from /usr/local/libressl/include/openssl/dsa.h:83:0,
                 from /usr/local/libressl/include/openssl/x509.h:99,
                 from /usr/local/libressl/include/openssl/pem.h:71,
                 from /usr/local/libressl/include/openssl/ssl.h:150,
                 from ssl_private.h:90,
                 from ssl_engine_init.c:29:
/usr/local/libressl/include/openssl/dh.h:195:5: note: previous declaration of 'DH_set0_pqg' was here

ssl_engine_init.c:54:12: error: static declaration of 'DH_set0_pqg' follows non-static declaration

static int DH_set0_pqg(DH *dh, BIGNUM *p, BIGNUM *q, BIGNUM *g)

In file included from /usr/local/libressl/include/openssl/dsa.h:83:0,

from /usr/local/libressl/include/openssl/x509.h:99,

from /usr/local/libressl/include/openssl/pem.h:71,

from /usr/local/libressl/include/openssl/ssl.h:150,

from ssl_private.h:90,

from ssl_engine_init.c:29:

/usr/local/libressl/include/openssl/dh.h:195:5: note: previous declaration of 'DH_set0_pqg' was here

LibreSSLのopenssl/dh.hでDH_set0_pqgが宣言されているのに
mod_sslのssl_engine_init.cでDH_set0_pqgがstaticで実装されていて合っていないというエラーです。

このDH_set0_pqgという関数はOpenSSL1.1未満では実装されていないため、
mod_sslはOpenSSLが1.1未満の場合に自前の実装を使うようになっています。

プリプロセッサ的にはMODSSL_USE_OPENSSL_PRE_1_1_APIがONになっていれば
mod_ssl側でDH_set0_pqgを実装する形です。

このMODSSL_USE_OPENSSL_PRE_1_1_APIはLibreSSLを使うとONになるようになっているため
DH_set0_pqgが実装されたLibreSSL 2.7以降と合わせると不整合という状態になってしまいます。
(LibreSSLは基本的にOpenSSL 1.0.1ベースだが、LibreSSL 2.7でOpenSSL 1.0.2や1.1.0の機能がいくらか追加された)

対処としてはmod_ssl側の実装を削除してしまうだけです。

--- modules/ssl/ssl_engine_init.c       2018-02-14 08:43:36.000000000 +0900
+++ modules/ssl/ssl_engine_init.c.mod   2018-04-05 13:20:58.117148437 +0900
@@ -48,29 +48,6 @@
 #define KEYTYPES "RSA or DSA"
 #endif
 
-#if MODSSL_USE_OPENSSL_PRE_1_1_API
-/* OpenSSL Pre-1.1.0 compatibility */
-/* Taken from OpenSSL 1.1.0 snapshot 20160410 */
-static int DH_set0_pqg(DH *dh, BIGNUM *p, BIGNUM *q, BIGNUM *g)
-{
-    /* q is optional */
-    if (p == NULL || g == NULL)
-        return 0;
-    BN_free(dh->p);
-    BN_free(dh->q);
-    BN_free(dh->g);
-    dh->p = p;
-    dh->q = q;
-    dh->g = g;
-
-    if (q != NULL) {
-        dh->length = BN_num_bits(q);
-    }
-
-    return 1;
-}
-#endif
-
 /*
  * Grab well-defined DH parameters from OpenSSL, see the BN_get_rfc*
  * functions in  for all available primes.

--- modules/ssl/ssl_engine_init.c 2018-02-14 08:43:36.000000000 +0900

+++ modules/ssl/ssl_engine_init.c.mod 2018-04-05 13:20:58.117148437 +0900

@@ -48,29 +48,6 @@

#define KEYTYPES "RSA or DSA"

#endif

-#if MODSSL_USE_OPENSSL_PRE_1_1_API

-/* OpenSSL Pre-1.1.0 compatibility */

-/* Taken from OpenSSL 1.1.0 snapshot 20160410 */

-static int DH_set0_pqg(DH *dh, BIGNUM *p, BIGNUM *q, BIGNUM *g)

- /* q is optional */

- if (p == NULL || g == NULL)

- return 0;

- BN_free(dh->p);

- BN_free(dh->q);

- BN_free(dh->g);

- dh->p = p;

- dh->q = q;

- dh->g = g;

- if (q != NULL) {

- dh->length = BN_num_bits(q);

- }

- return 1;

-#endif

* Grab well-defined DH parameters from OpenSSL, see the BN_get_rfc*

* functions in for all available primes.

mod_sslとLibreSSLでは微妙に実装内容が違うのですが、きっとLibreSSL側の実装の方が信用できる。
mod_sslの方が信用できるよ派の人はssl_engine_init.c内のDH_set0_pqgを適当にリネームでもすればよいと思います。

あと私はApacheよりNginx派です。

2024年7月
月	火	水	木	金	土	日
1	2	3	4	5	6	7
8	9	10	11	12	13	14
15	16	17	18	19	20	21
22	23	24	25	26	27	28
29	30	31