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

Jump to bottom

Enable support for HTTP communication between components ECFLOW-1957 #121

Draft
wants to merge 23 commits into
base: develop
Choose a base branch
from
Draft

Enable support for HTTP communication between components ECFLOW-1957 #121

wants to merge 23 commits into from

Conversation

marcosbento
Copy link
Collaborator

No description provided.

@marcosbento marcosbento force-pushed the feature/support_http_comms branch 3 times, most recently from 5e20a18 to 92016f8 Compare September 23, 2024 11:37
@codecov-commenter
Copy link

codecov-commenter commented Sep 23, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 45.18900% with 319 lines in your changes missing coverage. Please review.

Project coverage is 48.55%. Comparing base (61ec541) to head (5549420).

Files with missing lines Patch % Lines
libs/client/test/harness/SCPort.cpp 32.98% 65 Missing ⚠️
libs/test/harness/TestFixture.cpp 0.00% 45 Missing ⚠️
Viewer/ecflowUI/src/ServerListDialog.cpp 0.00% 27 Missing ⚠️
libs/server/src/ecflow/server/HttpServer.cpp 80.19% 20 Missing ⚠️
libs/core/src/ecflow/core/EcfPortLock.hpp 0.00% 18 Missing ⚠️
libs/base/src/ecflow/base/ServerProtocol.hpp 22.22% 14 Missing ⚠️
libs/server/test/TestServer.cpp 39.13% 14 Missing ⚠️
libs/test/harness/ScratchDir.hpp 0.00% 14 Missing ⚠️
Viewer/ecflowUI/src/ServerHandler.cpp 0.00% 13 Missing ⚠️
Viewer/ecflowUI/src/ServerList.cpp 0.00% 13 Missing ⚠️
... and 23 more
Additional details and impacted files 
@@             Coverage Diff             @@
##           develop     #121      +/-   ##
===========================================
+ Coverage    48.51%   48.55%   +0.03%     
===========================================
  Files         1182     1187       +5     
  Lines        97588    97886     +298     
  Branches     14719    14779      +60     
===========================================
+ Hits         47344    47526     +182     
- Misses       50244    50360     +116

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@marcosbento marcosbento force-pushed the feature/support_http_comms branch 3 times, most recently from 798ae74 to b1ca0fa Compare September 24, 2024 12:03
@marcosbento marcosbento force-pushed the feature/support_http_comms branch 2 times, most recently from df873ea to 6d66c4f Compare October 2, 2024 13:29
@marcosbento marcosbento force-pushed the feature/support_http_comms branch 5 times, most recently from 3a68a7b to a6b987c Compare October 20, 2024 07:47
@marcosbento marcosbento force-pushed the feature/support_http_comms branch 2 times, most recently from 6a27b3e to 3eb760f Compare October 21, 2024 11:34
@marcosbento
Refactor DefaultServer class template
a469786 
Re ECFLOW-1957
@marcosbento
Enable HTTP connection between Client/Server
de3acd4 
Re ECFLOW-1957
@marcosbento
Enable UI to use HTTP connection
25e6695 
Re ECFLOW-1957
@marcosbento
Enable REST/UDP proxies to use HTTP connection
5fc6d31 
Also, fix an eventual race condition accessing:
 - Ecf::state_change_no_ and Ecf::state_modify_no_
Apart from making these global (instead of thread_local),
the Http library is restricted to the use of a single thread

Re ECFLOW-1957
@marcosbento
Enable ecFlow REST API tests using HTTP backend
db7fc93 
Re ECFLOW-1957
@marcosbento
Improve TestFixture by extracting ScratchDir
943e8bf 
Re ECFLOW-1957
@marcosbento
Improve TestFixture by extracting LocalServerLauncher
673bbef 
Re ECFLOW-1957
@marcosbento
Enable s_test with HTTP backend
8c814e0 
Re ECFLOW-1957
@marcosbento
Simplify tests AbortCmd and WaitCmd
77e1665 
Re ECFLOW-1957
@marcosbento
Correct typo in source code comment
d498aeb 
Re ECFLOW-1957
@marcosbento
Replace deprecated use of BOOST_GLOBAL_FIXTURE
07e688e 
BOOST_GLOBAL_FIXTURE has been deprecated with Boost 1.65, to be
replaced by BOOST_TEST_GLOBAL_FIXTURE.

Re ECFLOW-1957
@marcosbento
Allow REST API tests to run in parallel
0427e5a 
Re ECFLOW-1957
@marcosbento
Remove the use of std::filesystem
5d489cc 
Re ECFLOW-1957
@marcosbento
Ensure parallel s_test* do not use same port or directory
923c599 
Re ECFLOW-1957
@marcosbento
Test ecFlow Python using HTTP ecFlow server
3bd2bfc 
Re ECFLOW-1957
@marcosbento
Enable HTTPS on the Client side
597af70 
Re ECFLOW-1957
@marcosbento
Use notion of selected Protocol
f137a9c 
- use Protocol value instead of flags isSsl, isHttp, isHttps in the ecFlowUI implementation

Re ECFLOW-1957
@marcosbento
Move test methods in ClientInvoker to SCPort
5fcd0b0 
- create `harness` libraries to support the tests

Re ECFLOW-1957
@marcosbento
Use notion of selected Protocol
c34e5b2 
- use Protocol value instead of flags isSsl, isHttp, isHttps in the server/client implementation

Re ECFLOW-1957
@marcosbento
Add description to HttpServer
cab2395 
Re ECFLOW-1957
@marcosbento
Correct #include guards
44109f2 
Re ECFLOW-1957
@marcosbento
Correct outstanding compilation warnings
d267ddf 
Re ECFLOW-1957
@marcosbento
Update default HTTP connection timeouts
5549420 
Re ECFLOW-1957
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@marcosbento @codecov-commenter