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

several fixes to increase reliability on staging #594

Merged
merged 13 commits into from
Mar 30, 2022
Merged

several fixes to increase reliability on staging #594

merged 13 commits into from
Mar 30, 2022

Conversation

druiz0992
Copy link
Contributor

@druiz0992 druiz0992 commented Mar 25, 2022 •
edited
Loading

Several minor fixes to increase reliability on testnet.
closes #569

  • Checks WS state before sending data
  • Reopens WS if they close
  • Add keep alive function to Web3 web socket
  • Reopens Web3 socket in case it closes
  • Removes documentDb TLS requirement as it is not needed. It simplifies how to read documentDb from a lambda function in AWS
  • Reduces keepAliveInterval in web3 socket to 15 seconds
  • Increases ERC20 limit restriction. It was set to 2000 wei, and its gave problems to some of the tests that assumed Eth.
  • Adds offchain transactions to ping pong test. It was deleted in a previous PR by mistake
  • Add token amount restrictions

@druiz0992 druiz0992 temporarily deployed to AWS March 25, 2022 13:53 Inactive
@druiz0992 druiz0992 temporarily deployed to AWS March 25, 2022 14:11 Inactive
@druiz0992 druiz0992 temporarily deployed to AWS March 26, 2022 11:23 Inactive
@druiz0992 druiz0992 temporarily deployed to AWS March 28, 2022 08:26 Inactive
@druiz0992 druiz0992 temporarily deployed to AWS March 29, 2022 13:06 Inactive
@ChaitanyaKonda ChaitanyaKonda self-requested a review March 30, 2022 09:32
@Westlad Westlad self-requested a review March 30, 2022 09:33
@Westlad Westlad merged commit 24dcc84 into master Mar 30, 2022
@Westlad Westlad deleted the david/staging4 branch March 30, 2022 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ChaitanyaKonda ChaitanyaKonda ChaitanyaKonda approved these changes

@Westlad Westlad Westlad approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Check WS state before sending data
3 participants
@druiz0992 @Westlad @ChaitanyaKonda