Skip to content
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.

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

Proposal: Issue #344/PR#488 fix tests #501

Draft
wants to merge 68 commits into
base: master
Choose a base branch
from
Draft
Changes from all commits
Commits
Show all changes
68 commits
Select commit Hold shift + click to select a range
72d311d
Update syntax and test
nd4p90x Oct 24, 2023
43162b4
Push to rerun tests
nd4p90x Oct 24, 2023
7b5ce2e
Syntax update
nd4p90x Oct 24, 2023
4ca2160
Push to test
nd4p90x Oct 24, 2023
1246168
made a change to force action
nd4p90x Oct 25, 2023
cd0122d
Update Readme to reflect new workflows
nd4p90x Oct 25, 2023
ede8d5a
Remove CI folder and files
nd4p90x Oct 25, 2023
ed0c587
Update syntax
nd4p90x Oct 25, 2023
5fa073e
Fix issues with run rows
nd4p90x Oct 25, 2023
c236222
Adding workflows for manual execution
nd4p90x Oct 25, 2023
db0cfcb
Update run on push for tests
nd4p90x Oct 25, 2023
bd5c8f3
Removing workflow
nd4p90x Oct 26, 2023
7b9c6de
Remove version suffix
nd4p90x Oct 26, 2023
29eafc9
Trying a suggestion from user
nd4p90x Oct 26, 2023
99791df
removed
nd4p90x Oct 26, 2023
df04c48
Reset
nd4p90x Oct 26, 2023
5111757
Update syntax
nd4p90x Oct 26, 2023
c5935f8
Update per docs
nd4p90x Oct 26, 2023
bd0d5b6
Update make statement
nd4p90x Oct 26, 2023
3ef7a6a
Trying to fix some errors
nd4p90x Oct 26, 2023
467e1ac
Forgot only one run per statement
nd4p90x Oct 26, 2023
a8944be
Update pylint
nd4p90x Oct 27, 2023
7968319
enable e2e ENV variable
nd4p90x Nov 11, 2023
eb6e684
Updating test script
nd4p90x Jan 31, 2024
43f4311
Update node version requirements
nd4p90x Jan 31, 2024
742ca44
Remove e2e test for now and update test command
nd4p90x Jan 31, 2024
46d999d
adding dateutil to test
nd4p90x Jan 31, 2024
67e30b1
set up required modules
nd4p90x Jan 31, 2024
9484d07
Update scripts for testing
nd4p90x Jan 31, 2024
869d342
Update syntax and test
nd4p90x Oct 24, 2023
348c64b
Push to rerun tests
nd4p90x Oct 24, 2023
111b35a
Syntax update
nd4p90x Oct 24, 2023
b7fdd97
Push to test
nd4p90x Oct 24, 2023
c03212e
made a change to force action
nd4p90x Oct 25, 2023
2be9756
Update Readme to reflect new workflows
nd4p90x Oct 25, 2023
3cb2f8f
Remove CI folder and files
nd4p90x Oct 25, 2023
0ac337a
Update syntax
nd4p90x Oct 25, 2023
6d13cd3
Fix issues with run rows
nd4p90x Oct 25, 2023
4cc007c
Adding workflows for manual execution
nd4p90x Oct 25, 2023
1622bf7
Update run on push for tests
nd4p90x Oct 25, 2023
6bbffbd
Removing workflow
nd4p90x Oct 26, 2023
db09b2e
Remove version suffix
nd4p90x Oct 26, 2023
3b48754
Trying a suggestion from user
nd4p90x Oct 26, 2023
f789435
removed
nd4p90x Oct 26, 2023
b7126bf
Reset
nd4p90x Oct 26, 2023
232ed89
Update syntax
nd4p90x Oct 26, 2023
2ed85f4
Update per docs
nd4p90x Oct 26, 2023
e136673
Update make statement
nd4p90x Oct 26, 2023
a935834
Trying to fix some errors
nd4p90x Oct 26, 2023
9494146
Forgot only one run per statement
nd4p90x Oct 26, 2023
32a8e52
Update pylint
nd4p90x Oct 27, 2023
5cca99e
enable e2e ENV variable
nd4p90x Nov 11, 2023
b696e0e
Updating test script
nd4p90x Jan 31, 2024
ec6669a
Update node version requirements
nd4p90x Jan 31, 2024
7fae017
Remove e2e test for now and update test command
nd4p90x Jan 31, 2024
5cd013e
adding dateutil to test
nd4p90x Jan 31, 2024
c471d9d
set up required modules
nd4p90x Jan 31, 2024
c8fbe89
Update scripts for testing
nd4p90x Jan 31, 2024
3806f1c
Merge branch 'Issue-#344-Fix-Tests' of github.com:North-Two-Five/anal…
nd4p90x Jan 31, 2024
334518b
Moving tests to new file
nd4p90x Feb 1, 2024
018b15f
Update test Suite for python 3 requirements
nd4p90x Feb 1, 2024
d5892ad
Merge branch 'segmentio:master' into Issue-#344-Fix-Tests
nd4p90x Feb 8, 2024
1116532
Updating requirements.txt to include only necessary packages
nd4p90x Feb 20, 2024
b60b109
Update label
nd4p90x Feb 20, 2024
f39b3cf
Adding in requirements
nd4p90x Feb 20, 2024
3245c60
Remove dephell from requirements
nd4p90x Feb 21, 2024
b01701e
Proposal for update to tests failing in PR #344
nd4p90x Jan 29, 2025
310d7ef
Merge in master branch
nd4p90x Jan 29, 2025
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
11 changes: 7 additions & 4 deletions segment/analytics/request.py
Original file line number Diff line number Diff line change
Expand Up @@ -45,14 +45,17 @@ def post(write_key, host=None, gzip=False, timeout=15, proxies=None, oauth_manag
kwargs = {
"data": data,
"headers": headers,
"timeout": 15,
"timeout": timeout,
}

if proxies:
kwargs['proxies'] = proxies
res = None
kwargs['proxies'] = {
'http': proxies,
'https': proxies,
}

try:
res = _session.post(url, data=data, headers=headers, timeout=timeout)
res = _session.post(url, **kwargs)
except Exception as e:
log.error(e)
raise e
Expand Down
Loading