プロジェクト

全般

プロフィール

Bug(バグ) #2231

[api] 取得したアクティビティのフィールドに含まれるURLがおかしい

Youichi Kimura12年以上前に追加. 12年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2011-06-21
期日:
進捗率:

100%

3.6 で発生するか:
[QA]バグ通知済:
いいえ
3.8 で発生するか:
Unknown (未調査)

説明

Overview (現象)

下記のように activity_data.uri や activity_image.uri にabsolute URIが含まれる場合に、RESTful API や osapi などでアクティビティを取得した際にURLがおかしな値になる。

sqlite> select * from activity_data;
id|member_id|in_reply_to_activity_id|body|uri|public_flag|is_pc|is_mobile|source|source_uri|foreign_table|foreign_id|template|template_param|created_at|updated_at
1|1||test|http://sns.localhost/|1|1|1|||||||2011-06-02 21:29:57|2011-06-02 21:29:57

sqlite> select * from activity_image;
id|activity_data_id|mime_type|uri|file_id|created_at|updated_at
1|1|image/png|http://www.example.com/image.png||2011-06-02 21:29:58|2011-06-02 21:29:58

APIで取得されるアクティビティ:

{
  "id": "1",
  "userId": "1",
  "title": "test",
  "postedTime": "2011-06-02T21:29:57+09:00",
  "streamUrl": "http:\/\/sns.localhost\/sqlite\/http%3A\/",
  "mediaItems": [
    {
      "url": "http:\/\/sns.localhost\/sqlite\/http%3A\/",
      "type": "image\/png" 
    }
  ]
}

Causes (原因)

lib/util/opJsonDbOpensocialService.class.php にて、各 URL のフィールドに値を設定している箇所でURLがabsoluteであるかないかに関わらず app_url_for を通しているため間違った値に変換されている。

if ($activity->getUri())
{
  $a['streamUrl'] = app_url_for('pc_frontend', $activity->getUri(), true);
}
if ($image->getFileId())
{
  // (snip)
}
else
{
  $mediaItem['url'] = app_url_for('pc_frontend', $image->getUri(), true);
  $mediaItem['type'] = $image->getMimeType();
}

Way to fix (修正内容)

下記のいずれかの方法でこの問題を解決することが可能である。

  • absolute URIの場合には app_url_for を使わないようにopOpenSocialPlugin側で修正する
  • app_url_for にabsolute URIが渡された場合は何もせずにreturnするようにCore側で修正する

関連するチケット

関連している opOpenSocialPlugin - Backport(バックポート) #2247: [api] 取得したアクティビティのフィールドに含まれるURLがおかしい Fixed(完了) 2011-06-21
関連している opOpenSocialPlugin - Backport(バックポート) #2246: [api] 取得したアクティビティのフィールドに含まれるURLがおかしい Fixed(完了) 2011-06-21

関係しているリビジョン

リビジョン db256ed9 (差分)
Youichi Kimura12年以上前に追加

fix the weird url of a Activity on APIs (fixes #2231)

履歴

#1 Youichi Kimura12年以上前に更新

  • 対象バージョン1.3.3 にセット

#2 Youichi Kimura12年以上前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

更新履歴 db256ed978e7c02d233a8a7dbfa420b9020b605f で適用されました。

#3 Youichi Kimura12年以上前に更新

  • ステータスPending Review(レビュー待ち) から Fixed(完了) に変更
  • 進捗率50 から 100 に変更

他の形式にエクスポート: Atom PDF