Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@nighttardis
Copy link
Contributor

Describe the change
When a request includes their own ID, backend container are unable to find the event and data entries in Redis. This is because the keys for both the even and data are set with a UUID generated by the frontend, but when submitting the tasks the ID will the the one provided by the request. This update will update the event and data keys before they are submitting to Redis if an ID exists in the request.

Describe testing procedures
Ran the sample python client included in this repo.

Sample output
If this change modifies Strelka's output, then please include a sample of the output here.

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of and tested my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

@phutelmyer
Copy link
Contributor

@nighttardis Great find! Thanks for troubleshooting and writing this PR up. I'll review it locally today. 👍

@phutelmyer
Copy link
Contributor

Tested locally, works as expected.
Thank you for your contribution @nighttardis!

@phutelmyer phutelmyer merged commit d3dc40f into target:master Jun 14, 2023
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.

2 participants