-
-
Notifications
You must be signed in to change notification settings - Fork 86
Develop #472
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
Develop #472
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -58,6 +58,16 @@ public static class ServiceBusConfig | |
| public static string RabbitUsername = ""; | ||
| public static string RabbbitPassword = ""; | ||
| public static string RabbbitExchange = ""; | ||
|
|
||
| /// <summary> | ||
| /// Ceiling for a single serialized message body, in bytes. Sits under the broker's | ||
| /// max_message_size, which production reports as 16777216 (16MiB), leaving room to | ||
| /// shed payload before the broker refuses it. A body over max_message_size isn't | ||
| /// rejected cleanly, it closes the channel with a PRECONDITION_FAILED and takes the | ||
| /// connection's in-flight work with it. Keep this below whatever the brokers are | ||
| /// configured with; their config lives outside this repository. | ||
| /// </summary> | ||
| public static int MaxMessageSizeInBytes = 15 * 1024 * 1024; | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Immutability issue in Core/Resgrid.Config/ServiceBusConfig.cs: MaxMessageSizeInBytes is a compile-time constant but is declared as a mutable static field. Declare MaxMessageSizeInBytes as const, or static readonly if runtime assignment is required, to prevent accidental modification. Kody rule violation: Use `readonly` or `const` for Immutable Data public const int MaxMessageSizeInBytes = 15 * 1024 * 1024;Prompt for LLMTalk to Kody by mentioning @kody Was this suggestion helpful? React with 👍 or 👎 to help Kody learn from this interaction. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Immutable configuration data in Kody rule violation: Use `readonly` or `const` for Immutable Data public const int MaxMessageSizeInBytes = 15 * 1024 * 1024;Prompt for LLMTalk to Kody by mentioning @kody Was this suggestion helpful? React with 👍 or 👎 to help Kody learn from this interaction. |
||
| #endregion RabbitMQ Bus Values | ||
| } | ||
|
|
||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.