Skip to content

Conversation

@realbigsean
Copy link
Member

Issue Addressed

Closes #1669

Proposed Changes

Remove cargo audit ignore for ws server related vuln now that the ws server has been removed

Additional Info

N/A

@realbigsean realbigsean added low-hanging-fruit Easy to resolve, get it before someone else does! ready-for-review The code is ready for review labels Dec 4, 2020
Copy link
Member

@paulhauner paulhauner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! The test failure looks like the non-deterministic one that we've seen recently, not a fault of this PR.

bors r+

@paulhauner paulhauner added ready-for-merge This PR is ready to merge. and removed ready-for-review The code is ready for review labels Dec 6, 2020
bors bot pushed a commit that referenced this pull request Dec 6, 2020
## Issue Addressed

Closes #1669

## Proposed Changes

Remove cargo audit ignore for ws server related vuln now that the ws server has been removed

## Additional Info

N/A


Co-authored-by: realbigsean <seananderson33@gmail.com>
@bors
Copy link

bors bot commented Dec 7, 2020

@bors bors bot changed the title Remove audit ignore ws server [Merged by Bors] - Remove audit ignore ws server Dec 7, 2020
@bors bors bot closed this Dec 7, 2020
@realbigsean realbigsean deleted the remove-audit-ignore-ws-server branch November 21, 2023 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

low-hanging-fruit Easy to resolve, get it before someone else does! ready-for-merge This PR is ready to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants