rep2 part72

※■公式p2はこのスレでは扱っていません■※

ブラウザから 2ちゃんねる、まちBBS、JBBS@したらばBBS 等の
掲示板を快適に閲覧するためのPHPスクリプトです。
Mac OS X、UNIX、Linux、Windows などで動くマルチプラットフォーム設計。
ソースは公開されています。

※このスクリプトを動作させるためには、サーバを動かしてPHPを走らせる必要があります。
※荒れる元なのでレンタル鯖やVPSにp2を設置する話題はしない＆スルーしてください。
※不具合は怒らず煽らず落ち着いて報告しましょう。

■公式ページ
rep2 expack 全部入り for PHP 8.x by （´・ω・） ｽ
https://github.com/mikoim/p2-php/
rep2 expack 全部入り by open774
https://github.com/open774/p2-php
p2 - PHPでつくった2ch Viewer
http://akid.s17.xrea.com/
p2Wiki
http://akid.s17.xrea.com/p2puki/

■前スレ
rep2 part71
http://egg.5ch.net/test/read.cgi/software/1710391399/

なんでわざわざ非対応環境で動かす苦行をしてるのか知らんけど数年後の私のために頑張って直しといてね

UbuntuアプグレしたらPHP 8.5になるから今のうちに人柱してくれると助かる

fukumenさんのdockerのphpは8.5やん

自分は8.5でログに出たら都度直してるけど、基本的にNull合体演算子でやってて
でもfukumenさんはarray_key_existsでやる方針っぽいな

そもそも変数定義しろってことだよな。めんどくせぇから対症療法になっちゃうけど。

Docker版だとIP表示スレはIPV6じゃなくてIPV4のが表示される

PHP初心者だしNull合体演算子とか知らんかった、勉強になる
人のコードだし周りに合わせるぐらいでポリシーとか諦めてる
関数のヘッダですら元々ばらばらだしもうどうしようもない

それにしてもこのshift jisのコードはなんとかならないものか
geminiだとGEMINI.mdで指示しておいてもよくutf-8になって出てくる

>674
ipv6はテストしてないから予想だけど
配布しているcompose.ymlだとnetwork_mode: bridgeにしてるので
ipv6で受けたらコンテナ内にはipv4にproxyされるような動作になるはず
でコンテナから外に出るときはipv4になるはず

DATがSJISだからソースをUTF-8に変えたら変換して表示しないといけなくなる

PostgreSQLはこれあてないと多分使えない
https://pastebin.com/5sRmcbNV

Warning: Trying to access array offset on null in rep2/data/compile/iv2/_iv2_popup.tpl.html.ja.php on line 8
iv2.phpでテーブルの空白埋めのセルの数だけ$item['exif']がないぞって怒られるんだけど、
本来修正するべきlib/expack/ImageCache2/templates/_iv2_popup.tpl.htmlはテンプレートの記法がよくわからん
とりあえずiv2.php 920行目をコメントアウトで空白埋め自体をなくして対処したけど

git diff で見るとき SJIS と UTF-8 が混ざるんだよな

サムネ処理の画像フォーマットにwebpとavifを追加 PHP8.2以降
https://pastebin.com/XTJJN5YL

>>678
他のテンプレートでitemのnullチェックをやってたので同様の処理を追加した

PostgreSQLの方はAIさんに12移行だと必要だって言われたわ
古すぎワロタ
地味にメンテされているようで使われてないのかなぁ

MariaDB使ってる
Nextcloudも稼働してるから

>>680
サンクス
AIにレビューしてもらったら追加で3個修正しろって言われたので
合計6箇所を目をつぶってマージした
既存の処理には影響なさそうに見えるしへーきへーき

>681
Gemini曰く
> 確認してみましたが、結論から言うと「この修正を適用済みで、かつ現在も広く使われている（Packagistなどで公開されている）roojsのフォークは存在しない」ようです。

フォークするかプルリクするしか無さそうと思ったら
https://github.com/cweagans/composer-patches
を使えばいいじゃねとも提案してくれたので検討してみるけど、
めんどくさそうだったらそこだけフォークかな・・・

今dat直接読めるからいっそ一から作り直そうと思ったけど自分用には手間がかかりすぎるし公開するにはメンテとデバッグ面倒だしどうすっかな。

PHP8.5に変えたらかなりサクサクになった

cweagans/composer-patchesを使ってPostgreSQL 12用のパッチを当てるように出来たのでコミット
composer.jsonを更新したのでcomposer.phar installが必要

docker-rep2でPostgreSQLをデバッグするのに必要だったのでrep2-extraイメージを用意した
rep2-extraはimagemagickとかmysqlとか何でもあり
README.mdに説明を追加した
rep2-imagickは廃止
なんか他にも入れたほうが良さそうなのを探して追加した

PHP8以降で自分で直したところで、まだ残ってるっぽい所
何年も前に直したっきりの所が多いから要レビュー
https://pastebin.com/NU8hW2z0

lib/P2CurlMulti.php 修正
https://pastebin.com/mx8Arb5y

>>688 追記
curl_multi_select周りのところ、自分の環境(PHP8.5.4, curl 8.18.0)ではこれで動いてるけど
古いバージョンではわからないから触らない方がいいかもしれない

「curl による並列ダウンロードを有効にする」を1にすると板一覧を更新したときに新スレ数が表示されないのは仕様ですか

それ>>688のパッチでなおるんじゃないの
知らんけど

単なるテキストなんだし100Mbpsくらい出てれば並列ダウンロードなんかにしなくてもサクサク読み込める
ナローバンド時代の遺物だな

並列ありとなしじゃ最近読んだスレで更新確認する板が多数ある場合にぜんぜん違うぞ
だからバグり散らかしてるP2CurlMulti.phpを修正してでも使う勝ちがある

>>693
どれくらい差がある？計測したのか？

回線速度は関係ない
リクエスト単発で連続に投げるとと並列でなげるのじゃ全然違う
変わらんていうなら見てる板数の違いじゃね

更新板数61で計測したら、並列なしで15秒くらい、並列ありで0.5秒くらい

>>694
手元の環境だとオフで最近読んだスレの更新がおよそ5秒のところが
1秒ぐらいになる

ひとつのファイルを並列で落とす並列ダウンロードと勘違いしている感じか
並列なしだとsubject.txtを取りに行く板の数に比例して更新時間が伸びるが
並列ありなら一番応答が遅い鯖の速度が支配的になる

パッチの日本語部分が文字化けしちゃてるように見えるけどなんでだろ

ShiftJISとUTF8の違いでしょ、たぶん。

downloadで落としたけどshift jisで見てもutf-8でも化けてるんだよね
Pastebinが余計なことやってるみたいな話なんかな

>688
2件とも修正しつつ反映した
修正したところはコミットメッセージに記載
次上げてくれる機会があれば文字化け(というか破壊に見える)しないようにしてくれると助かる
shift jisのパッチを上げて化けるならutf-8にして上げてくれればいいかも

机上では散々確認したけど、動作はノーチェックです😀

非docker環境のDebian13でPHP8.4.16とcurl8.14.1ですが最新のfukumen rep2で
expack.use_curl_multi = 1で正常にお気にスレ更新できるようになりました
パッチ作者様fukumen様ありがとうございますm(_ _)m

fukumen版なんだけど、●(Uplift)ログインしていても書き込みフォームに●チェックが出なくなったのは仕様？
ログインしていれば自動的に●オンで書き込まれるってこと？
同じくどんぐりも左フレームにどんぐりログインしていることは表示されるけど、
スレッド表示や書き込みフォームにどんぐり絡みが何も出てこないのも仕様？

5chログインとどんぐりはdocの下のマニュアル読んでから質問してくれ
upliftログイン中の書き込みは全部●オンになる仕様
スレッド表示時はどんぐりは送信してない、大砲は常時表示
書き込み時は持ってるどんぐりを常に送信、持ってない(=送信していない)なら書き込みに後に自動ログインのどんぐりを受信

更新時間の制限ってどこで変更するんだっけ

https://egg.2ch.net/software/
↑こういう板URLを現在のホストで開くやつ
https://hstorage.io/dl/TWv9Mr5jQFgu.patch
$urlはそのままの方がいいとか、$strはそのままのほうがいいとか、p2じゃなくrep2にすべきとか色々あると思うが

>>707
あ、置換URLの最後にスラッシュ入れるべきだったか

やりたいことは理解できた。
今までのホスト移転であれば302のときにgetCurrentHostを使ってリクエストをやり直す処理で救っていたけど、
5ch.netドメインが不通になったからそんな実装じゃどうしようもないって話だよね。

ただここだけじゃ足りない気もするし、
getCurrentHostを使っていた側(SubjectTxt他とかのコンストラクタあたり)に
処理を用意したほうが網羅的に対応できて良さそうな気はする。
検討します。

なんか最近書き込みに時間がかかる
15秒くらい？
変なとこいじっちゃったのかな…

今は5ch自体がおかしい
ブラウザで直に開いても404になるところあるし

>707
2ch_to_5chが有効のときにread.phpだけ5ch.ioに読み替える処理を作っていたのでそれをsubject.phpとかでもやるようにした
過去ログ系もURL自体は元々置換えはしていないのでURL自体は置換えしない方がいいと思うのでこれでいいかなとおもう

あと、docker-rep2のイメージのcleanup処理のバグでイメージが全部消えていたので再ビルド中です

>>712
設定引き継いで新しいバージョンにして設定しようとすると↓の警告でるね
一回設定すればでなくなるけど
Warning: Undefined array key "use_https" in rep2/edit_conf_user.php on line 1748
Warning: Undefined array key "use_https" in rep2/edit_conf_user.php on line 1904
Warning: Undefined array key "use_http_domains" in rep2/edit_conf_user.php on line 1748

>>713
ユーザー設定ファイルにp2versionが埋め込まれていてバージョン更新を検知する仕組みがあることに気付いてなかった
p2versionってjavascript更新したときだけ上げればいいのかと思ってた
次更新するときは更新するようにする

confってローカル設定用のファイルがおけるようになっていればマージしなくて済みそうだけど
ちょっと見ただけでもic2が特殊だったり、手前で代入した値を使ってるところが結構あるからそう単純な話じゃなさそう
本格的にやるなら大工事になるわりに恩恵が微妙だし、もっと他にやることあるだろって話だよなあ

既存のライブラリとかフレームワークとかありそうなもんだけど
何か良さげなアイデアあったら教えてください

confに関してはずっとこれでやってきたし無理に変える必要はないと思う
マジでやるなら根本から作り直すな俺なら

スマホモードのSPMから"これにレス"を選ぶとこれがでる
PHP Warning: Constant P2_SESSION_CLOSE_AFTER_AUTHENTICATION already defined, this will be an error in PHP 9 in rep2/post_form.php on line 6

>>717
spm_k.phpでrequire init.phpしてからinclude post_form.phpしちゃってるからか
これ他のincludeも同じことだなぁ
とりあえずリダイレクトに変更が一番楽に他に起きているかもしれない問題を含めて直せるけど、うーん微妙すぎる

リダイレクトは操作性悪化するしPC版のやり方に寄せる方向で検討中

今の実装だとSPMから跳んだ先でセッションがクローズされたのにセッション変数を書こうとすると失敗するので
アップローダー使うと失敗するとか、read.phpで読んだ位置とか、
分かりづらい不具合が起きてそう、みたいな話
PHP8の警告とかじゃなくてただの不具合ですね

PHP 9出るまでに直さんといけないな
次は8.6って話もあるがどうなんだろうな

https://greta.5ch.io/test/read.cgi/poverty/1774410659/
このスレ開ける？

>>721
開ける

PHP Warning: Undefined array key "img" in rep2/hissi.php on line 30

>> 717
スマホはspm_k.phpを使わないように修正した
携帯は警告はまだ出るが実害ないようにした

>> 723
修正した

docker-rep2とrep2-allinoneで部分的にconfの自動マージをするようにした
詳細はREADME.mdに記載

rep2-allinoneにmacOS(Homebrew)を追加した
Linuxと同様に外部依存無しの静的リンクのPHP-FPMとCaddyを使用

Mac使いの俺としては興味あるけどVPSで10年以上も動かしてるから導入は見送る
Cloudflare使えばMAP-Eとかでポート全閉じになってても外部からアクセスできるんだろうけど

fukumenさんのdocker環境に乗り換えた
docker使うの初めてだったのでホストのnet.ipv4.ip_forwardが0になってるせいで
コンテナが外のネットワークと繋がらず半日悩んだ

fukumen氏版でcrontabにscripts/fetch-dat.phpを仕込む場合
環境変数SECRET_KEYは必要？
どこで設定すればいい？

fukumen氏には申し訳ないけどDockerからファイル抜き出して動かしてる
DockerだとCSSイジれないから
CSSやmenu.phpなどがイジれないと無理十数年自分なりのインターフェイスでやってきたのに今更デフォとか無理だし
Dockerでそれら改変できればDockerが楽でいいんだけどね

cssだけバインドマウントすればいいのでは

>>727
SECRET_KEYは必要
crontabにPATHみたいに定義するのが一番簡単そう
使ったことなかったので動かしてみたら警告は出るけけどそれっぽいログは出たから一応動いていそう

>>728
css変えるだけなら729の言うとおりバインドマウントすればいいと思う
menu.phpとかファイル指定のバインドマウントも可能
変更ファイルが多いならビルドした方が早いかな
rep2のみ変更のビルドは1分あれば終わる

新規でfukumen Docker入れてみたテスト

fukumen dockerでimagecacheをmysqlで使おうとすると

php scripts/ic2.php setup
enabled=3
dsn='mysql://ic2_user:[email protected]:3306/ic2_db'
driver='gd'
Image Driver: OK

In Setup.php line 174:

Extension 'mysql' is not loaded.

で使えない
sqliteなら使えるんだけど

Docker側からMySQLサーバーが見えてない
unix socket使えばいい

Dockerなんか使ったら終わり
横着の極み

はいはい。

外のMySQLに繋ぎたいんだろうけど
まず doc/database.md のようにDocker同士でいけるかを先に試したほうがいいかな
MariaDBはテストしたけどMySQLはテストしてないので何かあってもおかしくはない

あとはcompose.ymlを晒してくれたらあれこれ言えるかもしれない
まずはコンテナ内からping通るんだっけとかだろうか
さすがにrep2-extraを使っていないというオチはないよね

Extension 'mysql' is not loaded. だからDockerのイメージにphpのmysqlのライブラリが入ってないんでしょ。

732です
ご指摘通りrep2-extraが入っていませんでした
docker-compose.extra.ymlをdocker-compose.override.ymlにリネームして
再度docker compose up -dしてみましたが今度は
docker compose exec rep2php8 sh
/var/www # php scripts/ic2.php setup
enabled=3
dsn='mysql://ic2_user:[email protected]:3306/ic2_db'
driver='gd'
Image Driver: OK

In Exception.php line 45:

Connect failed, turn on debugging to 5 see why


In DataObject.php line 628:

SQLSTATE[HY000] [1045] Access denied for user 'ic2_user'@'172.19.0.2' (using password: YES)


setup [--check-only] [--pg-trgm PG-TRGM]
とやはりホスト側のmariadbに繋げられません

docker-compose.ymlは
services:
rep2php8:
build:
context: .
dockerfile: docker/Dockerfile
additional_contexts:
p2-rep2: https://github.com/fukumen/p2-php.git#php8-merge-mbstring
2chproxy.pl: https://github.com/fukumen/2chproxy.pl.git#always-https-for-2ch-config
args:
FLAG_EXTRA: "false"
FLAG_LOCAL: "false"
FLAG_DEBUG: "false"
image: ${REP2_IMAGE:-ghcr.io/fukumen/rep2:latest}
volumes:
- ${REP2_DATA:-./rep2-data}:/ext
ports:
- "${REP2_PORT:-10088}:8443"
restart: unless-stopped
extra_hosts:
- "host.docker.internal:host-gateway"
environment:
TZ: "Asia/Tokyo"
SECRET_KEY: （以下略）
です

sqlサーバでそのアドレスを許可してないだろ

確かにsqlサーバー側が蹴ってるだけぽいね
rep2側の問題じゃなさげ

>>740
ホスト側の /etc/mysql/mariadb.conf.d/50-server.conf で
bind-address = 0.0.0.0
にしているのでコンテナからの接続はうけられるはずなのですが……

あとlinuxserver.io版のnextcloudを同じホストのdockerで動かしていまして
そちらはホスト側のmariadbに繋がっています
nextcloudのdocker-compose.ymlはこんな感じ
services:
nextcloud:
image: lscr.io/linuxserver/nextcloud:latest
container_name: nextcloud
environment:
- PUID=1000
- PGID=1000
- TZ=Asia/Tokyo
volumes:
- ./config:/config
- ./data:/data
ports:
- 11000:443
restart: unless-stopped
extra_hosts:
- "host.docker.internal:host-gateway"

Access deniedって応答はしてくれているからDB側の権限の問題じゃないの？
ユーザー毎にIPアドレスの許可とか指定できるだろうからそのあたりの話ではない
nextcloudとはic2_userってのは別ユーザーだろうし同じような設定にすればよいかもね

>>744
そうでした！
mariadbで
GRANT ALL PRIVILEGES ON *.* TO 'ic2_user'@'%' IDENTIFIED BY 'ic2_user' WITH GRANT OPTION;
したところ
# php scripts/ic2.php setup
enabled=3
dsn='mysql://ic2_user:[email protected]:3306/ic2_db'
driver='gd'
Image Driver: OK
Database: OK
Table 'imgcache' already exists
Table 'ic2_errors' already exists
Table 'ic2_blacklist' already exists
Index 'idx_imgcache_uri' already exists
Index 'idx_imgcache_time' already exists
Index 'idx_imgcache_unique' already exists
Index 'idx_ic2_errors_uri' already exists
Index 'idx_ic2_blacklist_uri' already exists
Index 'idx_ic2_blacklist_unique' already exists
とmariadbでimagecacheが使えるようになりました！
これでmikoim時代に集めたbbspinkの画像を掘り出すことができます
ありがとうございましたm(_ _)m

ここで聞いて誰かの回答待つよりClaudeやGeminiに聞いた方が瞬時で答え出るぞ

そうそう単にググるより的確な答えが返ってくる

Geminiは無料枠で足りなくなって金払うようになっても月額料金不要で使っただけな上に安いので趣味用途にはちょうど良いよな。

AIに否定的だったけどGrokを試しに使ってみて考えが変わった
Claude使おうまでとは思わんけどね

質問があります

git clone https://github.com/fukumen/docker-rep2.git
cd docker-rep2
docker compose up -d

公式どおりの手順で rep2 が使えるようになりました
その後なんですが

今後何か更新されたとかでアップデートしたい場合は
単に docker-rep2 ディレクトリで
docker compose up -d するだけでいいのでしょうか

それとも git clone からやり直して
rep2-data ディレクトリを移行すればいいとかでしょうか

コマンドラインは苦にならないのですが
git や docker はそこまで詳しくないので
どなたか教えていただけると助かります

docker compose down
してから
docker compose pull
そして
docker compose up -d

rep2に直接は関係ないけど「あなたの投稿を拒否しております」はCookie削除したら書き込めることが多いから、
これが帰ってきたらCookie削除してリトライするか確認してくれるとありがたいかも

テストがすげえやりづらそうだな
つうかそれ見た覚えないし

>>751
DockerHubにイメージ上げといてもらえるとwatchtowerで自動更新できて助かります

>>754
watchtowerは知らなかったのでaiに聞いたらパブリックならghcr.ioでも動くって言ってるけどダメそうなん？

フォント設定の見直し、アクティブモナーのスマホ対応、アクティブモナー用のWebフォント(Textar)に対応しました

* 個々のスキンに持っていたフォント設定を廃止
* conf_user_style.inc.phpのフォント設定をスキンの設定とし、OS毎に変更したいときに有効化するように変更
* フォント設定編集やユーザー設定(ActiveMona)の項目を修正
* safari周りのもういらないだろみたいなのを廃止
* 色々と古いバグの修正やゴミの掃除

注意点:
pref/p2_user_font.txtの構造を変更しているため、初回アクセス時にデフォルトに戻しています
mobile.res_sizeがデフォルト値だと小さめのAA以外は省略されます
iPadのSafariは「デスクトップ用Webサイトを表示」がデフォルトでONになっておりmacOSと判定されます
expack.am.autodetect_iとexpack.am.autong_kを同時にONにすることは想定していません

なお、フォント設定の説明を見ればだいたい分かるようには記載したつもりです
気がついているやり残しは以下の通り

* スマホモードでCSSにフォントが埋め込まれているがまだそのまま
* スマホでフォント設定できない

あと、docker-rep2とrep2-allioneでユーザースキン(user_skin)をrep2-dataに持てるようにしました

お疲れさん
フォント変わっちゃったけど再設定で戻せた
あの画面壊れててたけど長年スルーして使ってたんだよな…

ん？
元々AAがずれてるのか？
アクティブモナー機能のバグか？

もうAAなんて見かけなくなったな
気にしなくていい

>>758は自動判定でAAではないと判定されてるだけ

https://imgur.com/tPt2j9G.png

>>758のAAは正常に表示されています
バグではありません
失礼しました

> * スマホモードでCSSにフォントが埋め込まれているがまだそのまま

スマホモードはフォントサイズもCSSに埋め込みになっているので今回のフォント対応は意味ない状態とわかりました
メインメニューやUI関係はツールキットのCSSが埋め込みなので
それ以外のところ（スレ一覧・スレ本体）にフォント名とかサイズを反映するよう準備中です

AA判定ロジックは何も触ってない
モナーが誤判定するのはどうかと思うけど、でもまぁ放置かな

スマホのフォント名とフォントサイズ変更の対応をコミットしました
スレ一覧とスレ表示が以前より小さく表示されるので16pxに設定するか
（スマホのOSのデフォルトは16pxに変更したので）デフォルトに戻すを実行してください

16pxに設定してもスマホ本体側のフォントサイズ設定によっては
以前より大きくなったり小さくなったりするので適当に変更してください

あと、スマホでもフォント設定に行けるようにしましたが、
雑な対応のためスマホだと画面右にはみ出ます
まぁ使えりゃいいよね・・・

いまさらだけど、スキン毎のフォント設定を削除したのは正しい方向性なの？
スキンを変えるとそれに合わせてフォントも変わるってのが当たり前のような
フォントファミリーが変わると当然サイズも調整が必要だし
blackboardならcomic sansの〜ポイントみたいな
もし頻繁にスキン変える人がいたら使いにくくなってるんじゃないかと

edit_user_font.phpが開けなくなったので
conf_user_style.inc.phpでフォントを修正した

>>765
正直わからん
元の実装はスキンの中にもUSER_AGENTの判定してフォント設定してあったりとても残す気にはなれなかった

今回スマホの設定画面のボタンのテキストが下にズレてたのとか直したけど、
フォントやフォントサイズに依存した設計になっているのがそもそも駄目だと思うわ

どうしてもスキンでフォントを指定したい人はuser_skinに自分専用のスキンを入れて
そこでフォントを指定すればいいんじゃないかな

>>766
ログに何か出てる？

>>767
mac特有の問題っぽい
https://github.com/php/php-src/issues/11818