g1331 / xiaomai-bot

以Graia Ariadne框架为基础的战地1QQ机器人
GNU General Public License v3.0
44 stars 7 forks source link

Feat: Add proxy feature #36

Closed Ra2-IFV closed 4 weeks ago

Ra2-IFV commented 1 month ago

Now you can set proxy in configuration.

Summary by Sourcery

Add support for configuring a proxy for HTTP requests and update relevant functions to utilize the proxy setting.

New Features:

Enhancements:

sourcery-ai[bot] commented 1 month ago

Reviewer's Guide by Sourcery

This pull request introduces a new proxy feature that allows setting a proxy in the configuration. The changes involve retrieving the proxy configuration from GlobalConfig and updating all aiohttp.ClientSession and httpx.AsyncClient calls to include the proxy parameter.

File-Level Changes

Files Changes
utils/bf1/bf_utils.py
utils/bf1/gateway_api.py
utils/bf1/draw/__init__.py
Introduced proxy support by adding proxy configuration retrieval and updating aiohttp.ClientSession and httpx.AsyncClient calls to include the proxy parameter.

Tips - Trigger a new Sourcery review by commenting `@sourcery-ai review` on the pull request. - Continue your discussion with Sourcery by replying directly to review comments. - You can change your review settings at any time by accessing your [dashboard](https://app.sourcery.ai): - Enable or disable the Sourcery-generated pull request summary or reviewer's guide; - Change the review language; - You can always [contact us](mailto:support@sourcery.ai) if you have any questions or feedback.
Ra2-IFV commented 1 month ago

stfu stupid ai words are useless

Ra2-IFV commented 4 weeks ago

write a contributing guide bro

On August 30, 2024 1:33:39 AM UTC, "十三" @.***> wrote:

@g1331 approved this pull request.

If all global variable names are prefixed with g_, it would be better.

-- Reply to this email directly or view it on GitHub: https://github.com/g1331/xiaomai-bot/pull/36#pullrequestreview-2270760585 You are receiving this because you authored the thread.

Message ID: @.***>