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.

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

Converted WinHTTP to Async. #4015

Merged
merged 50 commits into from
Oct 24, 2022
Merged

Converted WinHTTP to Async. #4015

Changes from 1 commit
Commits
Show all changes
50 commits
Select commit Hold shift + click to select a range
de0409b
Async WinHTTP
LarryOsterman Oct 10, 2022
8da47ee
Added cancellation test; added support for request cancellation.
LarryOsterman Oct 11, 2022
e48f66f
Added header for requests
LarryOsterman Oct 11, 2022
6bff722
Fixed deadlock when TLS error happens
LarryOsterman Oct 11, 2022
4a76919
FIxed accidentally changed timeout
LarryOsterman Oct 11, 2022
a37968f
Cleaned up some comments and headers
LarryOsterman Oct 11, 2022
5c5c63a
CSpell and fixed UWP build issue
LarryOsterman Oct 11, 2022
1539336
Update sdk/core/azure-core/CMakeLists.txt
LarryOsterman Oct 12, 2022
b85b2b8
Update sdk/core/azure-core/src/http/winhttp/win_http_transport.cpp
LarryOsterman Oct 12, 2022
25b2ff5
Update sdk/core/azure-core/src/http/winhttp/win_http_transport.cpp
LarryOsterman Oct 12, 2022
e1eaa35
Update sdk/core/azure-core/src/http/winhttp/win_http_transport.cpp
LarryOsterman Oct 12, 2022
8e14296
Update sdk/core/azure-core/src/http/winhttp/win_http_transport.cpp
LarryOsterman Oct 12, 2022
7a217e2
Update sdk/core/azure-core/src/http/winhttp/win_http_request.hpp
LarryOsterman Oct 12, 2022
14e0c86
Update sdk/core/azure-core/src/http/winhttp/win_http_request.hpp
LarryOsterman Oct 12, 2022
8d41533
Update sdk/core/azure-core/src/http/winhttp/win_http_transport.cpp
LarryOsterman Oct 12, 2022
768bfc6
Improved discoverability of tests; handle exceptions thrown during te…
LarryOsterman Oct 12, 2022
c695b02
Merge branch 'larryo/asyncwinhttp' of https://github.com/LarryOsterma…
LarryOsterman Oct 12, 2022
01d7904
clang-format
LarryOsterman Oct 13, 2022
cba73d3
clang-format
LarryOsterman Oct 13, 2022
2b67738
Merge branch 'main' into larryo/asyncwinhttp
LarryOsterman Oct 13, 2022
3b40e8b
Undid bad merge
LarryOsterman Oct 13, 2022
3f23d74
Use sigabrt handler to report any thrown exceptions
LarryOsterman Oct 13, 2022
9dc8514
clang-format
LarryOsterman Oct 13, 2022
ec7577f
Update sdk/core/azure-core/src/http/winhttp/win_http_request.hpp
LarryOsterman Oct 17, 2022
015c846
Update sdk/core/azure-core/src/http/winhttp/win_http_request.hpp
LarryOsterman Oct 17, 2022
684c013
Update sdk/core/azure-core/src/http/winhttp/win_http_request.hpp
LarryOsterman Oct 17, 2022
9b45291
pull request feedback
LarryOsterman Oct 17, 2022
aea5284
Merge branch 'larryo/asyncwinhttp' of https://github.com/LarryOsterma…
LarryOsterman Oct 17, 2022
12dd753
Pull request feedback
LarryOsterman Oct 17, 2022
57c50fb
More pull request feedback
LarryOsterman Oct 18, 2022
807a3cf
clang-format
LarryOsterman Oct 18, 2022
ae10a3f
Added more control over what events cause requests to complete
LarryOsterman Oct 18, 2022
68530e8
Pull request feedback
LarryOsterman Oct 19, 2022
2084eab
Update sdk/core/azure-core/inc/azure/core/internal/diagnostics/global…
LarryOsterman Oct 19, 2022
1a349d9
clang-format
LarryOsterman Oct 19, 2022
7068ca0
Merge branch 'larryo/asyncwinhttp' of https://github.com/LarryOsterma…
LarryOsterman Oct 19, 2022
e3e4cae
clang-format, again
LarryOsterman Oct 19, 2022
2e270fb
Don't set the complete event until after the mutex is released
LarryOsterman Oct 19, 2022
6c8ac5b
Added a bit of comments
LarryOsterman Oct 20, 2022
d431631
Swap out twiter.com for wikipedia.org
LarryOsterman Oct 20, 2022
d673a49
Cannot use wikipedia on CI pipeline
LarryOsterman Oct 20, 2022
2dfa186
Fixed aws url
LarryOsterman Oct 20, 2022
b013c05
Comment changes from pull request feedback
LarryOsterman Oct 21, 2022
10148f8
Update sdk/core/azure-core/src/http/winhttp/win_http_request.hpp
LarryOsterman Oct 21, 2022
a8573e0
pull request feedback
LarryOsterman Oct 21, 2022
f29c866
Merge branch 'larryo/asyncwinhttp' of https://github.com/LarryOsterma…
LarryOsterman Oct 21, 2022
53af4ee
Bumped test discovery timeout
LarryOsterman Oct 21, 2022
8d00d34
Update sdk/core/azure-core/src/http/winhttp/win_http_transport.cpp
LarryOsterman Oct 21, 2022
beac2fb
More pull request feedback
LarryOsterman Oct 21, 2022
1086aae
Merge branch 'larryo/asyncwinhttp' of https://github.com/LarryOsterma…
LarryOsterman Oct 21, 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
3 changes: 2 additions & 1 deletion cmake-modules/AddGoogleTest.cmake
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,8 @@ macro(add_gtest TESTNAME)
else()
gtest_discover_tests(${TESTNAME}
TEST_PREFIX "${TESTNAME}."
PROPERTIES FOLDER "Tests")
PROPERTIES FOLDER "Tests"
DISCOVERY_TIMEOUT 600)
LarryOsterman marked this conversation as resolved.
Show resolved Hide resolved
endif()
else()
add_test(${TESTNAME} ${TESTNAME})
Expand Down