Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

記事著者のアイコンを表示する #10

Open
genya0407 opened this issue Mar 21, 2019 · 6 comments
Open

記事著者のアイコンを表示する #10

genya0407 opened this issue Mar 21, 2019 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@genya0407
Copy link
Contributor

genya0407 commented Mar 21, 2019

やりたいこと

著者のアイコンを表示する機能を実装したい.feedsからアイコン画像を配信し,feeds.jsonからそのURLを参照することでこれを実現する.

背景

以前 @tomoyat1 と議論したときは,画像をどこから配信するべきかということに関して合意が取れなかった.

初め @genya0407 は,外部サービスにアップロードした画像のURL(twitterのアイコン画像のURLなど)を指定するということを提案していた.しかし @tomoyat1 は,そのURLが永続的に使えるという保証がなくwebsiteの表示が崩れてしまう可能性があるという問題点を指摘した.
このissueは, 画像の配信もfeedsから行うようにすれば解決するのではないかという提案.

具体的には,レポジトリ直下にimages/ みたいなディレクトリを作って画像を入れていき,デプロイが走るたびにcal.camph.net/public以下にファイルをコピーし,feeds.jsonにそのURLを指定するということを考えている.

@tomoyat1 と方針について合意が取れれば,このissueは閉じてPRを出します.

@genya0407 genya0407 added the enhancement New feature or request label Mar 21, 2019
@genya0407 genya0407 changed the title 記事著者のアイコンとニックネーム(id)を表示する 記事著者のアイコンを表示する Mar 21, 2019
@tomoyat1
Copy link
Contributor

tomoyat1 commented Mar 21, 2019

代替案は今調査・検討していますが,画像をホスティングすることに関しては反対とさせてください.
アイコン画像にアニメのキャラなどの著作物が用いられていてそれをcamph.net以下でホスティングすると法的責任が伴うのでやめたいです.

@genya0407
Copy link
Contributor Author

genya0407 commented Mar 21, 2019

「アイコン」という言葉が悪かったかもしれませんが,他者の著作物ではない画像を指定するような運用にすればよいのではないかと思います(自分の写真など).

そもそも直リンクで他のサービスから画像を持ってきた場合でも,その画像が著作権法に違反している場合は問題になる可能性がありそうです(というか著作権法違反の直リンク画像に関してはまだ判例がない? https://www.derive-ip.com/2018/01/inline-link-and-copyright/ ).
なので本質的には直リンクか自前ホスティングかという区別が重要というよりは,「著作権法違反のアイコン画像を使わない」ということが重要だと思います.

従って,アイコン画像を自前でホスティングし,PRをレビューするときに著作権法違反の画像を使っていないことをチェックすればよいのではないかと思います.

@tomoyat1
Copy link
Contributor

↑なるほど...
ならひとまず実装コストが低い camph.net のサブドメインからの配信としましょうか.

@genya0407
Copy link
Contributor Author

合意が取れたのでclose

@tomoyat1
Copy link
Contributor

↑実装してmergeしたらcloseじゃないですか? (組織によって文化違いそう)

@genya0407
Copy link
Contributor Author

確かにそうかも

@genya0407 genya0407 reopened this Mar 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants