Skip to content

Update iOS, Android SDK - #108

Merged
koutaku921 merged 10 commits into
payjp:masterfrom
bu-tokumitsu:feature/update_android_sdk
Jun 15, 2026
Merged

Update iOS, Android SDK#108
koutaku921 merged 10 commits into
payjp:masterfrom
bu-tokumitsu:feature/update_android_sdk

Conversation

@bu-tokumitsu

@bu-tokumitsu bu-tokumitsu commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

概要

  • iOS SDK更新内容
  • Android SDK更新内容
    • Android SDKが内包するOkHttpのバージョンが古く、アプリ側で4.12.0以上のバージョンを使用した場合に logging-interceptor との依存関係が壊れてカードのページを開く際にクラッシュが発生していました
      • 暫定対応としてアプリ側で okhttp-bom を加えることで回避も可能ですが、根本解決ではないのでAndroid SDKのOkHttpの更新を行いました
    • Android SDKのOkHttpバージョンを3.12.13 から 4.12.0 に更新する改修を加えたので、参照するAndroid SDKの更新になります
  • CIの build-test.yml に FLUTTER_CHANNEL: stable が指定されているので現時点のstable(Flutter v3.44.1)に合わせていくつかファイルの変更が入っています

iOS動作確認

  • カード登録ができることを確認
  • キーボード周り、ナビゲーションバーの不具合が解消されていることを確認
  • Podfile.lockでiOS SDKのバージョン更新を確認

Android動作確認、依存関係の確認

前提条件

  • 切り替えの度にexampleで flutter clean && flutter pub get を実行して確認

依存関係の確認方法

  • example/android内で次のコマンドで確認
    • ./gradlew :app:dependencies --configuration releaseRuntimeClasspath | grep -E "jp.pay:payjp-android|com.squareup.okhttp3"

クラッシュ再現( c64539c

クラッシュ発生時の依存関係、PAY Android SDK(v2.2.1)+ OkHttp(v4.12.0)

logging-interceptorのみv3.12.13のままの為、カードViewを開くタイミングでクラッシュします

+--- com.squareup.okhttp3:okhttp:4.12.0
     +--- jp.pay:payjp-android:2.2.1
     |    +--- jp.pay:payjp-android-core:2.2.1
     |    |    +--- com.squareup.okhttp3:okhttp:3.12.13 -> 4.12.0 (*)
     |    |    +--- com.squareup.okhttp3:logging-interceptor:3.12.13
     |    |    |    \--- com.squareup.okhttp3:okhttp:3.12.13 -> 4.12.0 (*)
     |    |    |    \--- com.squareup.okhttp3:okhttp:3.14.9 -> 4.12.0 (*)
     |    \--- jp.pay:payjp-android-cardform:2.2.1
     |         +--- jp.pay:payjp-android-core:2.2.1 (*)
     |         +--- jp.pay:payjp-android-verifier:2.2.1
     |         |    +--- jp.pay:payjp-android-core:2.2.1 (*)

Android SDK v2.2.2へ更新後

logging-interceptorも4.12.0に更新される為、クラッシュは発生しません

+--- com.squareup.okhttp3:okhttp:4.12.0
     +--- jp.pay:payjp-android:2.2.2
     |    +--- jp.pay:payjp-android-core:2.2.2
     |    |    +--- com.squareup.okhttp3:okhttp:4.12.0 (*)
     |    |    +--- com.squareup.okhttp3:logging-interceptor:4.12.0
     |    |    |    +--- com.squareup.okhttp3:okhttp:4.12.0 (*)
     |    |    |    \--- com.squareup.okhttp3:okhttp:3.14.9 -> 4.12.0 (*)
     |    \--- jp.pay:payjp-android-cardform:2.2.2
     |         +--- jp.pay:payjp-android-core:2.2.2 (*)
     |         +--- jp.pay:payjp-android-verifier:2.2.2
     |         |    +--- jp.pay:payjp-android-core:2.2.2 (*)

CI Build and Test

Comment thread ios/payjp_flutter.podspec
s.pod_target_xcconfig = { 'DEFINES_MODULE' => 'YES', 'EXCLUDED_ARCHS[sdk=iphonesimulator*]' => 'i386' }

s.ios.deployment_target = '10.0'
s.ios.deployment_target = '12.0'

@bu-tokumitsu bu-tokumitsu Jun 12, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ここはPAYJP - iOSの IOS_DEPLOYMENT_TARGET に合わせています

@bu-tokumitsu bu-tokumitsu changed the title [WIP] Update iOS, Android SDK Update iOS, Android SDK Jun 12, 2026
@bu-tokumitsu
bu-tokumitsu marked this pull request as ready for review June 12, 2026 02:43

@laiso laiso left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

変更の差分を確認しました。問題なさそうです

@koutaku921 koutaku921 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

有識者レビューが完了したので、masterマージのため改めてapproveします。

@koutaku921
koutaku921 merged commit 9b27fb3 into payjp:master Jun 15, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants