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

Transport updates #15

Merged

Conversation

andrewwhitehead
Copy link
Contributor

  • Move webhook dispatch into AdminServer and use same mechanism for admin websocket connections
  • Use a single outbound queue and limit active transmissions with TaskProcessor
  • Add inbound and outbound transport unit tests, related refactoring
  • Fix demos to run with acagent instead of icatagent
  • Fixes for clean agent shut down

Signed-off-by: Andrew Whitehead <[email protected]>
… support multiple webhook targets; send webhooks over open websockets

Signed-off-by: Andrew Whitehead <[email protected]>
Signed-off-by: Andrew Whitehead <[email protected]>
Signed-off-by: Andrew Whitehead <[email protected]>
Signed-off-by: Andrew Whitehead <[email protected]>
Signed-off-by: Andrew Whitehead <[email protected]>
Signed-off-by: Andrew Whitehead <[email protected]>
Signed-off-by: Andrew Whitehead <[email protected]>
Signed-off-by: Andrew Whitehead <[email protected]>
Signed-off-by: Andrew Whitehead <[email protected]>
@andrewwhitehead andrewwhitehead requested a review from nrempel June 26, 2019 18:04
@codecov
Copy link

codecov bot commented Jun 27, 2019

Codecov Report

Merging #15 into master will decrease coverage by 3.37%.
The diff coverage is 77.41%.

@@            Coverage Diff             @@
##           master      #15      +/-   ##
==========================================
- Coverage   68.95%   65.58%   -3.38%     
==========================================
  Files         211      168      -43     
  Lines        8724     6988    -1736     
==========================================
- Hits         6016     4583    -1433     
+ Misses       2708     2405     -303

andrewwhitehead and others added 2 commits June 27, 2019 18:25
- Change location of test results
- Collect coverage report.

Signed-off-by: Wade Barnes <[email protected]>
@nrempel nrempel merged commit 92d4c2d into openwallet-foundation:master Jun 28, 2019
@andrewwhitehead andrewwhitehead deleted the transport-updates branch February 29, 2020 21:45
olichwiruk pushed a commit to olichwiruk/aries-cloudagent-python that referenced this pull request Jan 4, 2021
Init acknowledgment phase. Update POST settings are now validated.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants