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

import-custom-emojisでプロキシ除外の設定が効かない #8766

Open
CyberRex0 opened this issue May 30, 2022 · 7 comments · May be fixed by #15431
Open

import-custom-emojisでプロキシ除外の設定が効かない #8766

CyberRex0 opened this issue May 30, 2022 · 7 comments · May be fixed by #15431
Assignees
Labels
🐛Bug Unexpected behavior [Feat] Emoji/EmojiPicker Emoji or EmojiPicker related issue packages/backend Server side specific issue/PR

Comments

@CyberRex0
Copy link
Contributor

💡 Summary

絵文字の一括インポートにて。
config.ymlのproxyBypassHostsでプロキシを通さないホストを指定できるが、import-custom-emojisでは反映されない。
※一括インポートでのみ確認されたもの。個別インポートでは除外設定が効いていた

🥰 Expected Behavior

プロキシを設定しているとき、proxyBypassHostsに指定したホストと通信する場合はプロキシを経由しない。

🤬 Actual Behavior

プロキシを設定しているとき、proxyBypassHostsの設定に関係なくプロキシを経由してしまう。

📝 Steps to Reproduce

※絵文字がまとまったファイルが example.com にあるとする

  1. config.ymlでプロキシサーバへのアドレスとして通信できない適当なところを指定する (http://localhost:1000など)
  2. config.ymlのproxyBypassHostsにexample.comを追加する
  3. 絵文字一括インポートを試行する

📌 Environment

Misskey version: 12.110.1
Your OS: Android
Your browser: Chrome 102

@Johann150
Copy link
Contributor

I wonder if we could merge the implementations of getResponse (packages/backend/src/misc/fetch.ts) and downloadUrl (packages/backend/src/misc/download-url.ts).

@Johann150
Copy link
Contributor

maybe related: #8522

@CyberRex0
Copy link
Contributor Author

これどうなったんだっけ

@syuilo
Copy link
Member

syuilo commented Feb 12, 2023

直す

@Sayamame-beans
Copy link
Member

triage: @syuilo 直っていますか?

@Sayamame-beans Sayamame-beans added 🐛Bug Unexpected behavior and removed ⚠️bug? This might be a bug labels Jun 16, 2024
@syuilo
Copy link
Member

syuilo commented Jun 16, 2024

@samunohito samunohito added the packages/backend Server side specific issue/PR label Nov 27, 2024
@samunohito
Copy link
Member

なおってないです(さっき別件の調査の時にそれっぽい現象に遭遇した)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛Bug Unexpected behavior [Feat] Emoji/EmojiPicker Emoji or EmojiPicker related issue packages/backend Server side specific issue/PR
Projects
Status: No status
5 participants