Skip to content
This repository was archived by the owner on Feb 7, 2024. It is now read-only.

Added persistent cookie storage to avoid having to renew the session … - #1

Open
sco01 wants to merge 1 commit into
Kenzim:mainfrom
sco01:main
Open

Added persistent cookie storage to avoid having to renew the session …#1
sco01 wants to merge 1 commit into
Kenzim:mainfrom
sco01:main

Conversation

@sco01

@sco01 sco01 commented Apr 9, 2023

Copy link
Copy Markdown

…for each new session.

The PR will try to store the session cookies after a successful renew_session() into a file. Each following get_resp() will try to load the cookies from the file instead of renewing the session cookies. This saves about 4.5 seconds when establishing a new session and will do so until the cookies expire and a new renew_session() is called.

Please let me know if there is anything you would like me to improve and a big thanks for making this available!

Regards,
Stefan

@Kenzim

Kenzim commented Apr 26, 2023

Copy link
Copy Markdown
Owner

Hi, thanks so much for contributing.

Maybe you could make the cookie file path an argument when instantiating a new Rarbg object, so we dont always have to use the cookies file.

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