Skip to content
This repository has been archived by the owner on Aug 3, 2023. It is now read-only.

Use local IP not random IP (Fix timeout error & Eventloop error). #634

Merged
merged 7 commits into from
Aug 2, 2023
Merged

Conversation

JE-Chen
Copy link
Contributor

@JE-Chen JE-Chen commented Aug 1, 2023

Use local IP not random IP. (I think they blocked some random IP rnage?)

  • On my windows still need cookie file but fix timeout. (Ubuntu too)
  • Add pull_request_target on github action yml file. (make fork repo can run github action CI/CD test)
  • Ignore PyCharm config.

What actually is this project license? GitHub says unlicensed, PYPI say GPL V2 license.

  • I think 0.13.0 code will make "RuntimeError: Event loop is closed", when reuse same bot instance.
  • So i move the aiohttp session to stream function not on ChatBot init.

Fix:

#621
#627
#606

Maybe fix:

#631
#603
#636

Use local IP not random IP
@JE-Chen JE-Chen marked this pull request as ready for review August 1, 2023 03:26
Add pull_request_target
Add edited state to test not only test on open pull request
@JE-Chen JE-Chen changed the title Use local IP not random IP Use local IP not random IP (Fix timeout error). Aug 1, 2023
@JE-Chen JE-Chen closed this Aug 1, 2023
@JE-Chen JE-Chen reopened this Aug 1, 2023
Move aiohttp session to stream
* Avoid Runtime error
@JE-Chen JE-Chen changed the title Use local IP not random IP (Fix timeout error). Use local IP not random IP (Fix timeout error & Eventloop error). Aug 1, 2023
Refactor
Close aio session on stream
@JE-Chen
Copy link
Contributor Author

JE-Chen commented Aug 1, 2023

image

Not need to catch take IP exception
@acheong08 acheong08 merged commit 1da9d27 into acheong08:main Aug 2, 2023
@acheong08
Copy link
Owner

What actually is this project license? GitHub says unlicensed, PYPI say GPL V2 license.

Thanks for catching that. It's The Unlicense.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants