Skip to content

Conversation

@mdrohmann
Copy link
Contributor

@mdrohmann mdrohmann commented Jan 15, 2021

@mdrohmann
Copy link
Contributor Author

I don't know if this ready for prime time due to some limitations of the mediator that came up:

Follow-up stories are here:

The first one might be a blocker for putting it on production...

@mdrohmann mdrohmann requested a review from Naatan January 15, 2021 17:45
Copy link
Contributor

@Naatan Naatan left a comment

Choose a reason for hiding this comment

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

Very nice work! Just one super nitpicky change.

@mdrohmann mdrohmann requested a review from Naatan February 12, 2021 21:16
@mdrohmann
Copy link
Contributor Author

I added the changes that were necessary to work with the updated mediator. Of course, this does not work yet, with the production endpoint for the mediator.

Naatan
Naatan previously approved these changes Feb 12, 2021
@mdrohmann mdrohmann requested a review from Naatan February 13, 2021 00:41
@mdrohmann
Copy link
Contributor Author

Took a little while until the mediator was deployed, because kubernetes ran out of some resources, but now everything seems to be working. Just need another approval for the latest additions that I had to do...

@mdrohmann mdrohmann merged commit 9e17d76 into master Feb 13, 2021
@mdrohmann mdrohmann deleted the martind/state-cve-176449951 branch February 13, 2021 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants