Skip to content

Conversation

@StefanAustin
Copy link

Notes for Reviewers

  • The commit history must be preserved - please use the rebase-merge or standard merge option instead of squash-merge
  • Sync up with the author before merging

@StefanAustin StefanAustin linked an issue Oct 13, 2025 that may be closed by this pull request
@StefanAustin StefanAustin changed the base branch from master to 7.0 October 13, 2025 14:09
</li>
<li>The message field will be replaced by a shorter message to reduce license utilization. Activate the <code class="linecode">full_message</code> option in the input if needed.</li>
<section class="infoBox">
<div class="content"><b>Hint:</b> This pack rewrites the message field to reduce license utilization. To get the full message, set <i>Store full message</i> in the input settings to true.</div>
Copy link
Contributor

Choose a reason for hiding this comment

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

Should there be a space after >?

  • </b> This pack...

I don't have a system w/ the input running but verify store full message is actually a true or false setting. Update wording if necessary.

Copy link
Author

Choose a reason for hiding this comment

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

I copied it from an existing pack 1 by 1.... to have a consistent structure. I can of course mixes styles ... .
In terms of input, I do not have that input running either and the configuration is is step two behind the API connection test.
@CG3827 do you still have the input running?

Copy link
Contributor

Choose a reason for hiding this comment

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

Same (see previous comment).

Copy link
Contributor

@gormanbj gormanbj left a comment

Choose a reason for hiding this comment

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

lgtm

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.

Sophos: Update documentation add a note about shorter message

4 participants