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

Upgrade OADA Helper Packages #110

Merged
merged 1 commit into from
Jan 19, 2021
Merged

Conversation

aaron97neu
Copy link
Member

@aaron97neu aaron97neu commented Jan 11, 2021

Fixes new axios security issue axios/axios#3410.
Updates to client should also fix this: OADA/client#5

Update - Running new version on feeder at ault's farm. Full stack appears to be working fine. Will let it run through the end of the week to detect any intermittent issues and then will merge

Fixes new axios security issue axios/axios#3410.
Updates to client should also fix this: OADA/client#5
@aaron97neu aaron97neu self-assigned this Jan 11, 2021
@aaron97neu aaron97neu added bug Something isn't working dependencies Pull requests that update a dependency file labels Jan 11, 2021
@aaron97neu aaron97neu marked this pull request as ready for review January 14, 2021 20:06
@aaron97neu
Copy link
Member Author

Week of testing suspend cycles in a real environment plus successful testing with the newest versions of OADA should be sufficient. Merging

@aaron97neu aaron97neu merged commit 233cb26 into master Jan 19, 2021
@aaron97neu aaron97neu deleted the bugfix/security_oada_client_axios branch January 19, 2021 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant