Project

General

Profile

Actions

提案 #1294

closed

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

Added by Shibafu Midorino almost 4 years ago. Updated almost 4 years 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
Actions #1

Updated by Izumi Tsutsui almost 4 years ago

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

Actions #2

Updated by Shibafu Midorino almost 4 years ago

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

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

Actions #3

Updated by Shibafu Midorino almost 4 years ago

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

Updated by Izumi Tsutsui almost 4 years ago

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

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

Actions #5

Updated by あひる 家鴨 almost 4 years ago

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

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

Actions #6

Updated by Shibafu Midorino almost 4 years ago

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

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

Actions #7

Updated by cob odo almost 4 years ago

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

マージしてpushしました

Actions

Also available in: Atom PDF