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
ID generator for celery extension #51
Merged
Merged
Commits on Sep 9, 2022
-
Added customizable generator options to celery extension to better ma…
…tch capabilities of the correlation id middleware * Added optional "header_key" param to asgi_correlation_id.extensions.celery.load_correlation_id * Added optional "generator" param to asgi_correlation_id.extensions.celery.load_correlation_id * Added optional "generator" param to asgi_correlation_id.extensions.celery.load_celery_current_and_parent_ids
David Pryor (dapryor) committedSep 9, 2022 Configuration menu - View commit details
-
Copy full SHA for 75d6587 - Browse repository at this point
Copy the full SHA 75d6587View commit details
Commits on Sep 28, 2022
-
Standardized the signatures for CorrelationIdFilter/CeleryTracingIdsF…
…ilter and standardized the string trimming process for the filtering logic * changed type from int to Optional[int] for optional parameter 'uuid_length' in CeleryTracingIdsFilter * changed default value from 32 to None for optional parameter 'uuid_length' in CeleryTracingIdsFilter * created and utilized function `_trim_string` to standardize string trimming logic in filters * added test to test string trimming for CeleryTracingIdsFilter * added test to ensure the default behavior of the new filter matched the behavior of the old filter. This assumes default generators are used
David Pryor (dapryor) committedSep 28, 2022 Configuration menu - View commit details
-
Copy full SHA for ec96801 - Browse repository at this point
Copy the full SHA ec96801View commit details -
expanded
load_celery_current_and_parent_ids
to be able to utilize t……he internally generated celery_id * Added optional argument `use_internal_celery_task_id` to `load_celery_current_and_parent_ids` * When `use_internal_celery_task_id` is set to True, the internal celery task ID will be used and the generator function will be ignored * Updated README with information on this new argument
David Pryor (dapryor) committedSep 28, 2022 Configuration menu - View commit details
-
Copy full SHA for 6fe14ef - Browse repository at this point
Copy the full SHA 6fe14efView commit details
Commits on Sep 29, 2022
-
Configuration menu - View commit details
-
Copy full SHA for c3685a2 - Browse repository at this point
Copy the full SHA c3685a2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 57f4e40 - Browse repository at this point
Copy the full SHA 57f4e40View commit details -
Configuration menu - View commit details
-
Copy full SHA for 202cbf6 - Browse repository at this point
Copy the full SHA 202cbf6View commit details -
Configuration menu - View commit details
-
Copy full SHA for 237e183 - Browse repository at this point
Copy the full SHA 237e183View commit details -
Configuration menu - View commit details
-
Copy full SHA for a239fab - Browse repository at this point
Copy the full SHA a239fabView commit details -
Configuration menu - View commit details
-
Copy full SHA for 808122c - Browse repository at this point
Copy the full SHA 808122cView commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.