Skip to content

session-per-request to session-over-time #16

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.

Sign up for GitHub

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

Closed
runawaycoast wants to merge 8 commits into from
Closed

session-per-request to session-over-time #16

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
e661698
change connection from class to module
Mar 30, 2022
24e8845
reuse adapter
Apr 1, 2022
c258098
change initialization flow to allow configurate
runawaycoast Apr 5, 2022
a715192
change module naming, fix test
runawaycoast Apr 5, 2022
d910538
fix lint
runawaycoast Apr 5, 2022
c79d450
remove debug print
runawaycoast Apr 5, 2022
2b0f25d
add test for session reuse
runawaycoast Apr 6, 2022
9bc8dd0
fix lint
runawaycoast Apr 6, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
remove debug print
  • Loading branch information
@runawaycoast
runawaycoast committed Apr 5, 2022
commit c79d4501c6e436a2cfbd84a9669206b20125155d
1 change: 0 additions & 1 deletion nasdaqdatalink/connection.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,6 @@ def get_retries():
def get_session():
global session
if session is None:
print("initialized")
session = requests.Session()
adapter = HTTPAdapter(max_retries=get_retries())
session.mount(ApiConfig.api_protocol, adapter)
Expand Down