search

traPtitech / traQ

traQ - traP Internal Messenger Application Backend
MIT License
426 stars 29 forks source link

OGP情報取得のリクエストはtraP認証をバイパスして欲しい #2528

Open cp-20 opened 5 months ago

cp-20 commented 5 months ago

部内の各種サービスでよりリッチな表示をするため

関連issue: traPtitech/traQ_S-UI#4191

nokhnaton commented 1 month ago

フロント->サーバー->ogpを提供するサイト みたいな感じで取得してるので、サーバー側で処理したい気がする。 どこの部分で実装するにせよ特定のurlに特化したコードは書きたくないからどう実装するか難しそう

hijiki51 commented 1 month ago

https://github.com/traPtitech/traQ/blob/21ba43c61c40fb0158dcdcdb64bbbf48a11ae0ff/service/ogp/parser/domain.go#L21

見かけたので失礼します、ここら辺みたいなことすればよさそうかもしれないです

nokhnaton commented 1 month ago

やることとしては↑みたいな感じで良さそうなんですが、traPで使用している以外のインスタンス(例えばex-traQなど)で使えなくなってしまうので、ハードコーディングしちゃうのはできれば避けたいです。 URLをコンフィグや環境変数等で出し分けるのは最低限やるべきかなと考えていて、さらに柔軟性をもたせる実装にすると応用の幅が広がるのかなと思ってます

hijiki51 commented 1 month ago

確かにそうですね