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

Sentinel logic-app adding comments containing a HTML table has excessive newlines #10459

Open
twi252 opened this issue May 9, 2024 · 5 comments
Assignees
Labels
Playbook Playbook specialty review needed

Comments

@twi252
Copy link

twi252 commented May 9, 2024

Describe the bug

In a Logic App, using the Sentinel logic-app connector (Add comment to incident (V3)) in a For Each loop and entering a simple HTML table as the comment causes the table to be prefixed by many <br> tags upon presentation inside the browser leading to an unprofessional looking comment.

To Reproduce

Steps to reproduce the behavior:

  1. Make a simple logic app that iterates over some sample data in an array via For Each
  2. Use the "Add comment to incident (V3)" connector function to post a comment to a sample incident inside the For Each
  3. In the comment, add a title describing the table, ie, "This is my test data"
  4. Press enter 3 times
  5. Make a simple HTML table with 10 or so rows in the table from sample data
  6. Execute the logic-app and look at the comments in the Sentinel incident

Expected behavior

A comment with a title, a newline, then the table presented graphically (without excessive newlines between title and table)

Desktop (please complete the following information):

  • OS: Windows 10
  • Browser Chrome
  • Version Latest

Additional context

Using Chromes "inspect" feature to inspect the HTML in the relevant comment, expanding the relevant <div>, Sentinel seems to be adding many <br> tags (sometimes > 20) leading to a huge amount of space from the title in the comment (line 1) after all the <br>'s the table is way down below.

It seems to work normally with other HTML tags, but seems to have a problem with <table>'s specifically. I've been trying to remove the excess <br> tags but haven't had much luck, so am filing this bug report.

@v-rusraut v-rusraut added the Connector Connector specialty review needed label May 10, 2024
@v-rusraut
Copy link
Contributor

Hi @twi252 , Thanks for flagging this issue, we will investigate this issue and get back to you with some updates by 17-05-2024. Thanks!

@twi252
Copy link
Author

twi252 commented May 11, 2024

Thanks. Just to add, i tried putting the table into a string-variable and passed the output to "Add comment to incident (V3)" and it does exactly the same thing (i noticed it passes HTML colors across from the <table> tag, but it strips it out when the html is put into Add comment to incident (V3) directly).

I've tried trim(variable('mystring')) to ensure there's no leading or trailing whitespace, and see the same.

I need to transpose the column headers from the top to the left, hence the reason i'm not using "Create HTML Table".

@v-rusraut v-rusraut added Playbook Playbook specialty review needed Connector Connector specialty review needed and removed Connector Connector specialty review needed labels May 15, 2024
@v-rusraut
Copy link
Contributor

Hi @twi252,
We need more time to check and investigate into this issue. Thanks!

@twi252
Copy link
Author

twi252 commented May 20, 2024

Hi, @v-rusraut, no problem, appreciate the help.

@v-sudkharat v-sudkharat removed the Connector Connector specialty review needed label May 23, 2024
@v-rusraut
Copy link
Contributor

Hi @twi252,
We have replicated the issue, now we are working with respective team to fix it, we will update you.
Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Playbook Playbook specialty review needed
Projects
None yet
Development

No branches or pull requests

3 participants