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

Fork Sync: Update from parent repository #4

Open
wants to merge 489 commits into
base: master
Choose a base branch
from

Conversation

github-actions[bot]
Copy link

No description provided.

@github-actions github-actions bot requested a review from a team as a code owner March 17, 2023 18:00
phatboyg and others added 29 commits October 21, 2023 12:28
* Updated SQS queue access policy to be less permissive in granting access to appropriate SNS topic

* Updated to set queue access policy statement condition to use the topic's ARN without a wild-card

* Removed option of wild-carded topic ARN

* Revert "Removed option of wild-carded topic ARN"

This reverts commit 96324a8.
#4737)

* Bugfix to make rabbitmq_delayed_message_exchange works with other exchange types

* Including unit tests

* Changing the tests to generate different names for each exchange type

---------

Co-authored-by: paulohumberto <paulo@acerto.com.br>
…SagaDbContex

Simplify using a protected constructor
…for SQS, no other transports do this validation though.
…s value despite allowing it to be set (#4788)

* Fixed failure status setting for healthcheck

* Extended health check documentation: added information about options

* Fixed getters and setters in health check options and also fixed docs about tags

* Fixed getter again
* Add net8.0 TFM

* Update to .NET 8 for unit tests

* Fix unit tests

* Disable CS9191

* Obsolete formatter-based serialization

* Disabled more CS9191

* Remove out of support netcoreapp3.1 TFM from benchmark project
lahma and others added 30 commits August 3, 2024 12:36
* remove promotion of Quartz's Newtonsoft based Quartz.Serialization.Json
* remove unused Quartz.Plugins.TimeZoneConverter
* fix XML doc comments
* remove unnecessary compiler warnings suppression
* remove usage of obsolete UseMicrosoftDependencyInjectionJobFactory
* SqlTransport -> Use configured schema when listening to notifications

Hi @phatboyg
I found an issue with the Listen (postgres) notifications part where the schema is hardcoded to be "transport"

Since it is allowed to have a different schema that is passed via options and the PostgresDatabaseMigrator uses it for creating the trigger script, we should have it aligned in the C# code.

* Sanitize schema name to fit postgres channel requirements

* Surround schema name with double quotes to prevent dbMigrator failures
…ransport with job saga endpoints. Also added inline endpoint configuration ability.
…ed from the batch. Empty batches should be "ignored."
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.