Recommended
More Related Content
Similar to とあるGaroon開発者のお仕事.pptx
Similar to とあるGaroon開発者のお仕事.pptx (20)
More from Cybozu, Inc.
More from Cybozu, Inc. (20)
とあるGaroon開発者のお仕事.pptx
- 2. 2 目次 1. 自己紹介 2. Garoonの紹介 3. Garoon開発チームの紹介 4. Yukimiチームの紹介 5. PHPのアップデートについて 6. 最近行った業務
- 4. 4 自己紹介 名前 所属 出身大学 入社 趣味 一言 ：金丸 眞也 ：開発本部 Garoon Yukimiチーム ：東京工業大学 ：2020年4月サイボウズ新卒入社 ：歌うこと、ゲーム ：丁寧な暮らしがしたい かねまる なおや
- 6. 6
- 7. 7 Copyright © Cybozu PHP4時代から続く、(PHPだけでも) 120万行以上のコード！ そりゃ、課題もみつかってきます！ • 古いコードによる脆弱性への対応 • リリース方法の見直し • パフォーマンスの改善 • インフラ基盤の刷新に合わせた対応 Garoonは21歳！(2002年初代リリース)
- 8. 8 Copyright © Cybozu 直近の新機能例： • PDFプレビュー機能 • ユーザープロフィールのポップアップウィンドウ表示 新機能開発もまだまだ続きます！
- 13. 13 Copyright © Cybozu セキュリティの担保のための活動 • 脆弱性の対応 • XSS、SQLインジェクション等 • ライブラリの更新 • 新機能利用のためもあるが、脆弱性対応のためであることが多い • こまめに更新しないと、最新のセキュリティパッチが取り込めない！ • 毎年行っている、特に大きいものはPHPの更新 より安全なGaroonをユーザーに届ける！
- 15. 17 Copyright © Cybozu PHPアップデートの流れ 影響調査 実装 テスト リリース 更新により機能が使えなくなったり、エラーの種類の変更があったりするので、 単純に実行ファイルを置き換えたり、 `$ brew upgrade php` みたいなことをすればいいというものではない！
- 16. 18 Copyright © Cybozu PHPアップデートの流れ 影響調査 実装 テスト リリース
- 18. 20 Copyright © Cybozu 1.PHPの公式移行マニュアルをもとにした調査 • 後ほど詳しく話します 2.PHPの内部API(C言語で書かれた部分)に対する調査 3.PHPのバージョンだけ上げたブランチを試しに使う調査 4.ライブラリに対する調査 PHPアップデート調査の大まかな流れ
- 19. 21 Copyright © Cybozu 1.PHPの公式移行マニュアルをもとにした調査 • 後ほど詳しく話します 2.PHPの内部API(C言語で書かれた部分)に対する調査 • GaroonはPHPに手を入れて自前でビルドして利用している • コンフリクトが発生しているかも 3.PHPのバージョンだけ上げたブランチを試しに使う調査 4.ライブラリに対する調査 PHPアップデート調査の大まかな流れ
- 20. 22 Copyright © Cybozu 1.PHPの公式移行マニュアルをもとにした調査 • 後ほど詳しく話します 2.PHPの内部API(C言語で書かれた部分)に対する調査 • GaroonはPHPに手を入れて自前でビルドして利用している • コンフリクトが発生しているかも 3.PHPのバージョンだけ上げたブランチを試しに使う調査 • 動かしてみて分かる問題に早めに気づくため 4.ライブラリに対する調査 PHPアップデート調査の大まかな流れ
- 21. 23 Copyright © Cybozu 1.PHPの公式移行マニュアルをもとにした調査 • 後ほど詳しく話します 2.PHPの内部API(C言語で書かれた部分)に対する調査 • GaroonはPHPに手を入れて自前でビルドして利用している • コンフリクトが発生しているかも 3.PHPのバージョンだけ上げたブランチを試しに使う調査 • 動かしてみて分かる問題に早めに気づくため 4.ライブラリに対する調査 • Garoonが利用するライブラリが新しいPHPに対応しているか知るため PHPアップデート調査の大まかな流れ
- 25. 27 Copyright © Cybozu https://www.php.net/manual/en/migration82.php 1. 調査対象の把握 PHPの公式移行マニュアル
- 26. 28 Copyright © Cybozu アップデートによりなんらかの変更があった件数 PHP7.4→PHP8.0：240件 （メジャーアップデートは多め） PHP8.0→PHP8.1：98件 PHP8.1→PHP8.2：66件 1. 調査対象の把握
- 27. 29 Copyright © Cybozu PHPの更新作業(調査、実装、テスト、リリース)が完了するまでに Garoon本体に新機能やバグ修正等がどんどんマージされる！ 何度も調査しなければいけない！ → 調査を繰り返し行えるよう、調査方法をまとめる！ → 自動で調査できるものはスクリプトを作成し簡略化！ 2. 調査方法の調査
- 28. 30 Copyright © Cybozu ■ Garoonに影響があるか(ありそうか)どうかを調べる • PHPの更新によって追加された関数、定数と同じ名前のものが Garoonで既に使われているかも • 比較演算の結果が変わるけど大丈夫？（大丈夫じゃない） ■ Garoonに影響がある(ありそうな)場合 それをどうやって発見するのか、方法をまとめていく • 同じ関数名を使っていないかgrepする • 適切な型の引数が渡されているかを確認する 2. 調査方法の調査
- 29. 31 Copyright © Cybozu 「2. 調査方法の調査」で決めた方法で実際にGaroonのソースコー ドを調査し、問題を発見する。 • PHP8.2から非推奨になる関数をGaroonでも使っていた！ → 代替関数に置き換えるタスクを作成！ • PHP8.2で挙動が変化する関数はGaroonで使われていない！ → 問題なし！ヨシ！ → 調査記録をまとめて、調査完了！ 3. 実調査
- 30. 32 32 32 Copyright © Cybozu まとめ • Garoonの日本チームは課題解決に日々奮闘してます！ • ライブラリ更新はこまめに行うのが大事！ • 大きなプロダクトのライブラリ更新には、 入念な調査が必須！ レガシーなプロダクトに興味がある！ PHPに興味がある！そんな方はぜひGaroonへ！
- 31. 33 33 33 Copyright © Cybozu ■ Garoon開発チームを紹介します！ 2023 https://blog.cybozu.io/entry/2023/10/03/162616 ■ Garoonのセキュリティを維持するYukimiチームの紹介 https://blog.cybozu.io/entry/2023/10/04/101916 ■ 20年ものの巨大レガシープロダクトをPHP 8.0にアップデートした際の対 策と得られた知見 https://fortee.jp/phpcon-2022/proposal/8f29f20e-1275-49eb-89c0- fe684e28d110 もっと詳しく知りたい場合は、以下をご覧ください！