Project

General

Profile

提案 #1294

photo_support: ニコニコ静画への対応

Added by Shibafu Midorino 9 months ago. Updated 8 months ago.

Status:
終了
Priority:
通常
Assignee:
-
Target version:
プラグイン名:
photo_support
ブランチ:
topic/1294-nico-seiga

Description

ニコニコ静画のOGPから画像を展開するパッチを書きました。


Files

0001-nicoseiga-support.patch (942 Bytes) 0001-nicoseiga-support.patch Shibafu Midorino, 2018-09-09 17:31

Associated revisions

Revision baf36e70 (diff)
Added by Shibafu Midorino 8 months ago

ニコニコ静画のOGPから画像を展開する refs #1294

History

#1

Updated by Izumi Tsutsui 8 months ago

ニコニコ静画は実は https でもアクセスできるっぽいので、
そちらの対応も入れておくというのはどうですかね
https://seiga.nicovideo.jp/seiga/im5273604

#2

Updated by Shibafu Midorino 8 months ago

ニコニコ静画、https対応していたんですね……。知らなかったです。
httpsも対象とするよう、正規表現を修正しました。

また、作業中に短縮URL nico.ms が上手く処理できていなかったことに気づいたため、そちらに関しても変更を入れました。
HTTPClientのリダイレクト追跡オプションに任せるとhttps→http移動の際にプロトコル不一致で弾かれてしまうため、手作業で解決しています。
(現時点では、https://nico.ms/im* にアクセスすると決め打ちでhttpのURLを返すようです)

#3

Updated by Shibafu Midorino 8 months ago

  • File deleted (0001-nicoseiga-support.patch)
#4

Updated by Izumi Tsutsui 8 months ago

  • Status changed from 分類待ち to パッチ適用待ち

master d441aa5a に対してパッチ適用して
http, https × seiga.nicovideo.jp/seiga/, nico.ms/ のそれぞれ4通りでプレビューOKでした。
問題ないと思います。
https://social.mikutter.hachune.net/@tsutsuii/100696056922576297

#5

Updated by あひる 家鴨 8 months ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • ブランチ set to topic/1294-nico-seiga

ブランチにプッシュしました。
確認をお願いします。

#6

Updated by Shibafu Midorino 8 months ago

  • Status changed from レビュー待ち to マージ待ち

確認できました。問題ありません。

#7

Updated by cob odo 8 months ago

  • Status changed from マージ待ち to 終了

マージしてpushしました

Also available in: Atom PDF