Skip to content

Conversation

@Paillat-dev
Copy link
Member

Ignore this

FrostByte266 and others added 30 commits September 1, 2021 22:53
- Added middlewareplugin.py, this contains the proxy.py plugin that will
  intercept requests for you
- Added tests.py, this contains the base test case and one "test" test
- Added a makefile to help with cert generation and environment vars
- Added a readme file
- Created a custom request dataclass
- Updated .gitignore to ignore env files and cert files
- Made some changes to discord/http.py to facilitate functionality
  during testing
- Fixed the request blocking system in proxy
- Fixed the middleware dumping requests into queue as two different
  requests
- Changed the middleware plugin to selectively dump only requests that
  are explicitely made through the `proxied_request` test method
- Added logging to the middleware and test file
Fixed a spelling error
Corrected types from HttpParser to the new CombinedHttpRequest
Added method docstrings
Added comments and ignore some faulty type checker errors
# Conflicts:
#	.gitignore
#	discord/http.py
@Paillat-dev Paillat-dev requested review from a team and Lulalaby as code owners December 29, 2025 17:52
@Paillat-dev Paillat-dev added bypass-review Bypasses reviews for urgent pull requests involving critical code or documentation fixes. Test This is a test pull request, ignore labels Dec 29, 2025
@pycord-app
Copy link

pycord-app bot commented Dec 29, 2025

Thanks for opening this pull request!
Please make sure you have read the Contributing Guidelines and Code of Conduct.

This pull request can be checked-out with:

git fetch origin pull/3052/head:pr-3052
git checkout pr-3052

This pull request can be installed with:

pip install git+https://github.com/Pycord-Development/pycord@refs/pull/3052/head

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bypass-review Bypasses reviews for urgent pull requests involving critical code or documentation fixes. Test This is a test pull request, ignore

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants